[email protected] wrote: > Author: lektran > Date: Fri May 28 03:30:47 2010 > New Revision: 949088 > > URL: http://svn.apache.org/viewvc?rev=949088&view=rev > Log: > Be a little more explicit about what didn't return success when a > preprocessor event fails
Should this be backported to some of the release branches? > > Modified: > > ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java > > Modified: > ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java?rev=949088&r1=949087&r2=949088&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java > (original) > +++ > ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java > Fri May 28 03:30:47 2010 > @@ -301,7 +301,7 @@ public class RequestHandler { > String returnString = this.runEvent(request, response, > event, null, "preprocessor"); > if (returnString != null && > !returnString.equalsIgnoreCase("success")) { > if (!returnString.contains(":_protect_:")) { > - throw new EventHandlerException("Pre-Processor > event did not return 'success'."); > + throw new EventHandlerException("Pre-Processor > event [" + event.invoke + "] did not return 'success'."); > } else { // protect the view normally rendered and > redirect to error response view > returnString = > returnString.replace(":_protect_:", ""); > if (returnString.length() > 0) { > >
