[OFBIZ-11823] Refactor PickListServices#isBinComplete method to remove unnecessary iterations

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Implemented
• Affects Version/s: Trunk
• Component/s: product/facility
• Assignee: Pawan Verma
• Reporter: Pawan Verma
• Created: 16/Jun/20 10:02
• Updated: 11/Jan/22 09:36
• Resolved: 27/Jun/20 05:58

Description

PickListServices#isBinComplete() is used to check if picklistBin is complete.

This method reduces system performance when a picklistBin contains heavy no of items in it, this method iterator all the items to check the status of the item to validate if the bin is complete or not.

Instead of iteration, we can add itemStatusId != PICKITEM_COMPLETED condition while fetching records from the entity using queryCount() method and if we found count then picklistBin is not complete.

It can be a major performance factor on a production system working on heavy orders.