[ http://issues.apache.org/jira/browse/OFBIZ-523?page=comments#action_12457075 ] Jacopo Cappellato commented on OFBIZ-523: -----------------------------------------
I like this patch and I think we should commit it but since I don't know exactly how this service is supposed to work, I'd like to get the feedback from others before committing it. > Small change that can greatly reduce the amount of processing that > autoCancelOrderItems does... > ----------------------------------------------------------------------------------------------- > > Key: OFBIZ-523 > URL: http://issues.apache.org/jira/browse/OFBIZ-523 > Project: OFBiz (The Open for Business Project) > Issue Type: Improvement > Components: order > Affects Versions: SVN trunk > Environment: N/A > Reporter: Ray Barlow > Attachments: autocancelorderitems_v2.patch > > > This patch only changes two files: > services.xml - I've just re-written the comment based on reading and > understanding the code in the method. Hopefully people will find this clearer > and more precise about how it functions in different ways for orders at the > CREATED and APPROVED status. > OrderServices.java - This is the real perfomance change, I've just inserted a > Map filter so the sales order items code block will only check for cancel > dates against line items that have a CREATED or APPROVED status. It only > wastes time and resource to check COMPLETED, CANCELLED and REJECTED order > items. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
