Author: mthl
Date: Sat Oct 19 13:20:01 2019 New Revision: 1868621 URL: http://svn.apache.org/viewvc?rev=1868621&view=rev Log: Fixed: Format ‘XmlRpcEventHandler’ (OFBIZ-10284) This removes coding style errors introduced in revision 1868611. Modified: ofbiz/ofbiz-framework/trunk/build.gradle ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/event/XmlRpcEventHandler.java Modified: ofbiz/ofbiz-framework/trunk/build.gradle URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/build.gradle?rev=1868621&r1=1868620&r2=1868621&view=diff ============================================================================== --- ofbiz/ofbiz-framework/trunk/build.gradle (original) +++ ofbiz/ofbiz-framework/trunk/build.gradle Sat Oct 19 13:20:01 2019 @@ -307,7 +307,7 @@ checkstyle { // the sum of errors that were present before introducing the // âcheckstyleâ tool present in the framework and in the official // plugins. - maxErrors = 37947 + maxErrors = 37930 // Currently there are a lot of errors so we need to temporarily // hide them to avoid polluting the terminal output. showViolations = false Modified: ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/event/XmlRpcEventHandler.java URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/event/XmlRpcEventHandler.java?rev=1868621&r1=1868620&r2=1868621&view=diff ============================================================================== --- ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/event/XmlRpcEventHandler.java (original) +++ ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/event/XmlRpcEventHandler.java Sat Oct 19 13:20:01 2019 @@ -16,7 +16,6 @@ specific language governing permissions and limitations under the License. */ - package org.apache.ofbiz.webapp.event; import static org.apache.ofbiz.base.util.UtilGenerics.checkMap; @@ -71,14 +70,18 @@ import org.xml.sax.XMLReader; /** * XmlRpcEventHandler */ -public class XmlRpcEventHandler extends XmlRpcHttpServer implements EventHandler { +public final class XmlRpcEventHandler extends XmlRpcHttpServer implements EventHandler { - public static final String module = XmlRpcEventHandler.class.getName(); - protected LocalDispatcher dispatcher; + public static final String MODULE = XmlRpcEventHandler.class.getName(); + private LocalDispatcher dispatcher; private Boolean enabledForExtensions = null; private Boolean enabledForExceptions = null; + LocalDispatcher getDispatcher() { + return dispatcher; + } + @Override public void init(ServletContext context) throws EventHandlerException { String delegatorName = context.getInitParameter("entityDelegatorName"); @@ -97,15 +100,16 @@ public class XmlRpcEventHandler extends } @Override - public String invoke(Event event, RequestMap requestMap, HttpServletRequest request, HttpServletResponse response) throws EventHandlerException { - String report = request.getParameter("echo"); + public String invoke(Event event, RequestMap requestMap, HttpServletRequest req, HttpServletResponse res) + throws EventHandlerException { + String report = req.getParameter("echo"); if (report != null) { BufferedReader reader = null; StringBuilder buf = new StringBuilder(); try { // read the inputstream buffer String line; - reader = new BufferedReader(new InputStreamReader(request.getInputStream())); + reader = new BufferedReader(new InputStreamReader(req.getInputStream())); while ((line = reader.readLine()) != null) { buf.append(line).append("\n"); } @@ -120,12 +124,12 @@ public class XmlRpcEventHandler extends } } } - Debug.logInfo("Echo: " + buf.toString(), module); + Debug.logInfo("Echo: " + buf.toString(), MODULE); // echo back the request try { - response.setContentType("text/xml"); - Writer out = response.getWriter(); + res.setContentType("text/xml"); + Writer out = res.getWriter(); out.write("<?xml version=\"1.0\" encoding=\"UTF-8\"?>"); out.write("<methodResponse>"); out.write("<params><param>"); @@ -140,9 +144,9 @@ public class XmlRpcEventHandler extends } } else { try { - this.execute(this.getXmlRpcConfig(request), new HttpStreamConnection(request, response)); + this.execute(this.getXmlRpcConfig(req), new HttpStreamConnection(req, res)); } catch (XmlRpcException e) { - Debug.logError(e, module); + Debug.logError(e, MODULE); throw new EventHandlerException(e.getMessage(), e); } } @@ -152,10 +156,11 @@ public class XmlRpcEventHandler extends @Override protected void setResponseHeader(ServerStreamConnection con, String header, String value) { - ((HttpStreamConnection) con).getResponse().setHeader(header, value); + HttpStreamConnection hscon = (HttpStreamConnection) con; + hscon.getResponse().setHeader(header, value); } - protected XmlRpcHttpRequestConfig getXmlRpcConfig(HttpServletRequest req) { + private XmlRpcHttpRequestConfig getXmlRpcConfig(HttpServletRequest req) { OFBizXmlRpcHttpRequestConfigImpl result = new OFBizXmlRpcHttpRequestConfigImpl(req); XmlRpcHttpServerConfig serverConfig = (XmlRpcHttpServerConfig) getConfig(); @@ -178,12 +183,12 @@ public class XmlRpcEventHandler extends return result; } - class OfbizRpcAuthHandler implements AbstractReflectiveHandlerMapping.AuthenticationHandler { + final class OfbizRpcAuthHandler implements AbstractReflectiveHandlerMapping.AuthenticationHandler { @Override public boolean isAuthorized(XmlRpcRequest xmlRpcReq) throws XmlRpcException { - OFBizXmlRpcHttpRequestConfigImpl config = (OFBizXmlRpcHttpRequestConfigImpl) xmlRpcReq.getConfig(); - LocalDispatcher dispatcher = config.getDispatcher(); + OFBizXmlRpcHttpRequestConfigImpl config = (OFBizXmlRpcHttpRequestConfigImpl) xmlRpcReq.getConfig(); + LocalDispatcher dispatcher = config.getDispatcher(); ModelService model; try { @@ -218,8 +223,7 @@ public class XmlRpcEventHandler extends } @Override - public void execute(XmlRpcStreamRequestConfig pConfig, - ServerStreamConnection pConnection) throws XmlRpcException { + public void execute(XmlRpcStreamRequestConfig pConfig, ServerStreamConnection pConnection) throws XmlRpcException { try { Object result = null; boolean foundError = false; @@ -228,7 +232,7 @@ public class XmlRpcEventHandler extends XmlRpcRequest request = getRequest(pConfig, istream); result = execute(request); } catch (Exception e) { - Debug.logError(e, module); + Debug.logError(e, MODULE); foundError = true; } @@ -246,7 +250,8 @@ public class XmlRpcEventHandler extends if (!foundError) { writeResponse(pConfig, ostream, result); } else { - writeError(pConfig, ostream, new Exception("Failed to read XML-RPC request. Please check logs for more information")); + writeError(pConfig, ostream, + new Exception("Failed to read XML-RPC request. Please check logs for more information")); } } @@ -309,7 +314,7 @@ public class XmlRpcEventHandler extends class ServiceRpcHandler extends AbstractReflectiveHandlerMapping implements XmlRpcHandler { - public ServiceRpcHandler() { + ServiceRpcHandler() { this.setAuthenticationHandler(new OfbizRpcAuthHandler()); } @@ -319,7 +324,7 @@ public class XmlRpcEventHandler extends try { model = dispatcher.getDispatchContext().getModelService(method); } catch (GenericServiceException e) { - Debug.logWarning(e, module); + Debug.logWarning(e, MODULE); } if (model == null) { throw new XmlRpcNoSuchHandlerException("No such service [" + method + "]"); @@ -330,9 +335,9 @@ public class XmlRpcEventHandler extends @Override public Object execute(XmlRpcRequest xmlRpcReq) throws XmlRpcException { - OFBizXmlRpcHttpRequestConfigImpl requestConfig = (OFBizXmlRpcHttpRequestConfigImpl) xmlRpcReq.getConfig(); - LocalDispatcher dispatcher = requestConfig.getDispatcher(); - + OFBizXmlRpcHttpRequestConfigImpl requestConfig = (OFBizXmlRpcHttpRequestConfigImpl) xmlRpcReq.getConfig(); + LocalDispatcher dispatcher = requestConfig.getDispatcher(); + DispatchContext dctx = dispatcher.getDispatchContext(); String serviceName = xmlRpcReq.getMethodName(); ModelService model = null; @@ -372,7 +377,7 @@ public class XmlRpcEventHandler extends throw new XmlRpcException(e.getMessage(), e); } if (ServiceUtil.isError(resp)) { - Debug.logError(ServiceUtil.getErrorMessage(resp), module); + Debug.logError(ServiceUtil.getErrorMessage(resp), MODULE); throw new XmlRpcException(ServiceUtil.getErrorMessage(resp)); } @@ -382,8 +387,8 @@ public class XmlRpcEventHandler extends protected Map<String, Object> getContext(XmlRpcRequest xmlRpcReq, String serviceName) throws XmlRpcException { ModelService model; - OFBizXmlRpcHttpRequestConfigImpl requestConfig = (OFBizXmlRpcHttpRequestConfigImpl) xmlRpcReq.getConfig(); - LocalDispatcher dispatcher = requestConfig.getDispatcher(); + OFBizXmlRpcHttpRequestConfigImpl requestConfig = (OFBizXmlRpcHttpRequestConfigImpl) xmlRpcReq.getConfig(); + LocalDispatcher dispatcher = requestConfig.getDispatcher(); try { model = dispatcher.getDispatchContext().getModelService(serviceName); } catch (GenericServiceException e) { @@ -408,18 +413,18 @@ public class XmlRpcEventHandler extends } } - // only one parameter; if its a map use it as the context; otherwise make sure the service takes one param + // only one parameter; if its a map use it as the context; + // otherwise make sure the service takes one param. } else if (parameterCount == 1) { Object param = xmlRpcReq.getParameter(0); if (param instanceof Map<?, ?>) { context = checkMap(param, String.class, Object.class); + } else if (model.getDefinedInCount() == 1) { + String paramName = model.getInParamNames().iterator().next(); + context.put(paramName, xmlRpcReq.getParameter(0)); } else { - if (model.getDefinedInCount() == 1) { - String paramName = model.getInParamNames().iterator().next(); - context.put(paramName, xmlRpcReq.getParameter(0)); - } else { - throw new XmlRpcException("More than one parameter defined on service; cannot call via RPC with parameter list"); - } + throw new XmlRpcException( + "More than one parameter defined on service; cannot call via RPC with parameter list"); } } @@ -433,8 +438,8 @@ public class XmlRpcEventHandler extends class HttpStreamConnection implements ServerStreamConnection { - protected HttpServletRequest request; - protected HttpServletResponse response; + private HttpServletRequest request; + private HttpServletResponse response; protected HttpStreamConnection(HttpServletRequest req, HttpServletResponse res) { this.request = req; @@ -466,17 +471,15 @@ public class XmlRpcEventHandler extends } } - class OFBizXmlRpcHttpRequestConfigImpl extends XmlRpcHttpRequestConfigImpl { + class OFBizXmlRpcHttpRequestConfigImpl extends XmlRpcHttpRequestConfigImpl { private LocalDispatcher dispatcher; - public OFBizXmlRpcHttpRequestConfigImpl (HttpServletRequest request) { - dispatcher = (LocalDispatcher) request.getAttribute("dispatcher"); + OFBizXmlRpcHttpRequestConfigImpl(HttpServletRequest request) { + dispatcher = (LocalDispatcher) request.getAttribute("dispatcher"); } - + public LocalDispatcher getDispatcher() { - return dispatcher; + return dispatcher; } } - - } |
Free forum by Nabble | Edit this page |