Skip to content

Commit

Permalink
refactor: remove unnecssary vars
Browse files Browse the repository at this point in the history
also remove misleading docstring
  • Loading branch information
ankush committed Apr 27, 2022
1 parent 23cb0d6 commit 25485ed
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions erpnext/stock/doctype/pick_list/pick_list.py
Original file line number Diff line number Diff line change
Expand Up @@ -224,8 +224,6 @@ def group_similar_items(self):
item.idx = idx

def update_bundle_picked_qty(self):
"""Ensure that picked quantity is sufficient for fulfilling a whole number of."""

product_bundles = self._get_product_bundles()
product_bundle_qty_map = self._get_product_bundle_qty_map(product_bundles.values())

Expand All @@ -246,10 +244,11 @@ def _get_product_bundles(self) -> Dict[str, str]:
for item in self.locations:
if not item.product_bundle_item:
continue
bundle_item_code = frappe.db.get_value(
"Sales Order Item", item.product_bundle_item, "item_code"
product_bundles[item.product_bundle_item] = frappe.db.get_value(
"Sales Order Item",
item.product_bundle_item,
"item_code",
)
product_bundles[item.product_bundle_item] = bundle_item_code
return product_bundles

def _get_product_bundle_qty_map(self, bundles: List[str]) -> Dict[str, Dict[str, float]]:
Expand Down

0 comments on commit 25485ed

Please sign in to comment.