Hi Kyra, It seems that these changes are intentional, you can also check the comment inside the code itself:
"NOTE DEJ20080609 commenting out this code because the old OFBiz Workflow Engine is being deprecated and this was only for that get the delegator from the dispatcher" For more information, please have a look at the following: https://svn.apache.org/viewvc?view=revision&revision=665981 Thanks & Regards -- Deepak Nigam Technical Consultant/Team Lead HotWax Systems www.hotwaxsystems.com On Thu, Aug 3, 2017 at 2:08 PM, Kyra Pritzel-Hentley < [email protected]> wrote: > Hello OFBiz Enthusiasts, > > I am working to rid OFBiz of Bugs using the help of a code analysis tool > called FindBugs. With this, I find all kinds of peculiar code in the OFBiz > project. This JIRA ticket describes further details about this effort: > https://issues.apache.org/jira/browse/OFBIZ-9450 > > FindBugs pointed out to me that inside PayPalEvents#payPalIPN the method > OrderChangeHelper.releaseInitialOrderHold is called. But looking at the > implementation of the method, it only returns true even though the > parameters LocalDispatcher dispatcher and String orderId are handed over to > the method. The same problem arises with OrderChangeHelper# > abortOrderProcessing. > > There doesn’t seem to be a ticket working on this issue. I am fairly > certain, looking at the names of the methods, that something more should > happen other than just returning true. Does somebody have an idea what is > supposed to happen with these methods? > > Thank you, > Kyra Pritzel-Hentley > > ecomify GmbH > Bielefeld, Germany
