[
https://issues.apache.org/jira/browse/OFBIZ-11796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17128075#comment-17128075
]
Pawan Verma commented on OFBIZ-11796:
-------------------------------------
+1 Suraj, looks good.
> Unnecessary iterations for all productFacilities in setLastInventoryCount
> -------------------------------------------------------------------------
>
> Key: OFBIZ-11796
> URL: https://issues.apache.org/jira/browse/OFBIZ-11796
> Project: OFBiz
> Issue Type: Bug
> Components: product
> Affects Versions: Release Branch 18.12, Release Branch 17.12, Trunk
> Reporter: Suraj Khurana
> Assignee: Suraj Khurana
> Priority: Major
> Attachments: OFBIZ-11796.patch
>
>
> In the service setLastInventoryCount, which updates lastInventoryCount for
> product facility records, it works on inventoryItemDetail changes on
> availableToPromiseDiff.
> In parameter: InventoryItemId
> So it should be updating productFacilities entertaining both productId and
> facilityId of same inventoryItemId.
> Currently, it it unnecessary updating all the facilities inventory on a
> single inventory item update.
> It can be major performance factor on a production system working highly on
> inventory updates.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
