[
https://issues.apache.org/jira/browse/OFBIZ-2882?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jacques Le Roux closed OFBIZ-2882.
----------------------------------
Resolution: Invalid
I close this issue because the fixing patch is now invalid. But I applied the
test patch and it works. I believe it's incomplete because of OFBIZ-5192. Also
I'm not sure of it's path, there is already EntityTestSuite.java where it could
fit. It's minor but can be discussed before being commited
> EntityList cache clearing issues when removing generic entity via
> DelegatorImpl
> -------------------------------------------------------------------------------
>
> Key: OFBIZ-2882
> URL: https://issues.apache.org/jira/browse/OFBIZ-2882
> Project: OFBiz
> Issue Type: Bug
> Components: framework
> Affects Versions: SVN trunk
> Reporter: Bob Morley
> Assignee: Jacques Le Roux
> Priority: Critical
> Attachments: OFBIZ-2882_EntityCacheListFix.patch,
> OFBIZ-2882_EntityCacheListFix_V2.patch, OFBIZ-2882_EntityCacheListTest.patch,
> OFBIZ-2882_EntityCacheListTest_V2.patch
>
>
> For more information refer to
> http://ofbiz.135035.n4.nabble.com/EntityList-cache-issues-td206907.html
> Ran into some trouble when we turned out caching and started to dependent on
> the EntityList cache. The two problems were:
> 1) When attempting to storeHook to the entityListCache or entityObjectCache
> via the Cache.remove method, the NEW entity was being passed into the OLD
> entity. This caused condition matching (in the list cache) to sometimes fail
> if a matched entity no longer matches do to it being modified.
> 2) During the matching logic the EntityJoinOperator was incorrectly short
> circuiting. It was always checking if the lhs/rhs condition was true and if
> so returning the short-circuit value. A concrete example is as such -- (A is
> funny) && (B is funny). The short-circuit value for this expression is
> "FALSE" which means that if the first expression is FALSE we short-circuit
> and return FALSE. What was happening was "if (A is funny) then return FALSE"
> rather then "if (A is funny == FALSE) then return FALSE".
> I have a patch in the works for both of these issues and will include a unit
> tester that illustrates the problems (pre-patch) and passes successfully
> (post-patch).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
