This has been reverted at r1857095 Kind Regards, Deepak Dixit
On Mon, Apr 8, 2019 at 10:48 AM Deepak Dixit <deepak.di...@hotwax.co> wrote: > Getting the following exception on FindGeneric screen, It seems its due to > above change. Let me check.. > > ERROR MESSAGE > > java.lang.IllegalArgumentException: Error running script at location > [component://webtools/groovyScripts/entity/FindGeneric.groovy]: > org.apache.ofbiz.base.util.GeneralRuntimeException: Error getting the next > result (ResultSet not open. Operation 'next' not permitted. Verify that > autocommit is off.) > Kind Regards, > Deepak Dixit > > > On Mon, Apr 8, 2019 at 5:44 AM Scott Gray <scott.g...@hotwaxsystems.com> > wrote: > >> You'll need to be careful with this one, based on the limited usage of the >> iterator here it seems likely that it will be used again later in the >> process. >> >> Regards >> Scott >> >> On Mon, 8 Apr 2019 at 07:38, <m...@apache.org> wrote: >> >> > Author: mthl >> > Date: Sun Apr 7 19:38:09 2019 >> > New Revision: 1857084 >> > >> > URL: http://svn.apache.org/viewvc?rev=1857084&view=rev >> > Log: >> > Fixed: Use try with resource for closing ‘EntityListIterator’ >> > >> > Modified: >> > >> > >> ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/FormRenderer.java >> > >> > Modified: >> > >> ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/FormRenderer.java >> > URL: >> > >> http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/FormRenderer.java?rev=1857084&r1=1857083&r2=1857084&view=diff >> > >> > >> ============================================================================== >> > --- >> > >> ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/FormRenderer.java >> > (original) >> > +++ >> > >> ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/FormRenderer.java >> > Sun Apr 7 19:38:09 2019 >> > @@ -1215,8 +1215,7 @@ public class FormRenderer { >> > } >> > int itemIndex = -1; >> > if (iter instanceof EntityListIterator) { >> > - EntityListIterator eli = (EntityListIterator) iter; >> > - try { >> > + try (EntityListIterator eli = (EntityListIterator) iter) { >> > if(eli.getResultsSizeAfterPartialList() > 0){ >> > itemIndex++; >> > } >> > >> > >> > >> >