Re: svn commit: r1647522 - /ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/thirdparty/authorizedotnet/AIMPaymentServices.java

2014-12-23 Thread Scott Gray
and as much as I want to have yet another rant about you not testing changes before back porting them to the release branches I'm just going to sigh and consider stable branches a failed experiment. On Tue, Dec 23, 2014 at 11:42 PM, Scott Gray wrote: > I know it looks like this makes sense

Re: svn commit: r1647522 - /ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/thirdparty/authorizedotnet/AIMPaymentServices.java

2014-12-23 Thread Scott Gray
I know it looks like this makes sense at first glance but sometimes it's useful to look at the history of changes. It should be clear that something is amiss because of the useless try/catch block directly below your change. It points out that perhaps this code previously did something similar to