[
https://issues.apache.org/jira/browse/OFBIZ-11796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17128256#comment-17128256 ]
ASF subversion and git services commented on OFBIZ-11796:
---------------------------------------------------------
Commit f9142039568b93f31eea1f09358ef564d2696cc6 in ofbiz-framework's branch refs/heads/release17.12 from Suraj Khurana
[
https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=f914203 ]
Fixed: Unnecessary iterations and setting inventory for all productFacilities in setLastInventoryCount.
(OFBIZ-11796)
Instead of updating all product facilities records on one inventory item change, we should be changing only specific product facility record for that the inventory item belongs to. Thanks Jacques, Pierre, Pawan and Arun for the review.
> 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)
Locked
