Hi Deepak,
It's minor, but instead of hiding a possible RuntimeException by catching
Exception here I'd rather follow this FindBugs advice
--
This
The Buildbot has detected a restored build on builder
ofbiz-trunk-framework-plugins while building . Full details are available at:
https://ci.apache.org/builders/ofbiz-trunk-framework-plugins/builds/656
Buildbot URL: https://ci.apache.org/
Buildslave for this Build: lares_ubuntu
Build
The Buildbot has detected a restored build on builder ofbiz-trunk-framework
while building . Full details are available at:
https://ci.apache.org/builders/ofbiz-trunk-framework/builds/645
Buildbot URL: https://ci.apache.org/
Buildslave for this Build: silvanus_ubuntu
Build Reason: The
Author: deepak
Date: Wed Nov 1 10:43:14 2017
New Revision: 1813964
URL: http://svn.apache.org/viewvc?rev=1813964=rev
Log:
Fixed: doDecrypt method may return ClassNotFoundException,
BadPaddingException,so instead of handling GeneralException used Exception
class to handle all exception
Author: deepak
Date: Wed Nov 1 10:37:51 2017
New Revision: 1813961
URL: http://svn.apache.org/viewvc?rev=1813961=rev
Log:
Fixed: Added check in checkoutValidation method, as it should validate only for
Sales order, as shipping method, and payment mehtod not set in purchase order
(OFBIZ-9755)
5 matches
Mail list logo