svn commit: r1750610 - /ofbiz/trunk/applications/content/src/org/ofbiz/content/cms/CmsEvents.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1750610 - /ofbiz/trunk/applications/content/src/org/ofbiz/content/cms/CmsEvents.java

pranayp
Author: pranayp
Date: Wed Jun 29 08:36:48 2016
New Revision: 1750610

URL: http://svn.apache.org/viewvc?rev=1750610&view=rev
Log:
No functional changes. Corrected formatting for better readability.

Modified:
    ofbiz/trunk/applications/content/src/org/ofbiz/content/cms/CmsEvents.java

Modified: ofbiz/trunk/applications/content/src/org/ofbiz/content/cms/CmsEvents.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/content/src/org/ofbiz/content/cms/CmsEvents.java?rev=1750610&r1=1750609&r2=1750610&view=diff
==============================================================================
--- ofbiz/trunk/applications/content/src/org/ofbiz/content/cms/CmsEvents.java (original)
+++ ofbiz/trunk/applications/content/src/org/ofbiz/content/cms/CmsEvents.java Wed Jun 29 08:36:48 2016
@@ -152,7 +152,7 @@ public class CmsEvents {
                 mapKey = pathAlias.getString("mapKey");
                 if (contentId == null && UtilValidate.isNotEmpty(alias)) {
                     if (!alias.startsWith("/")) {
-                       alias = "/" + alias;
+                        alias = "/" + alias;
                     }
                     RequestDispatcher rd = request.getRequestDispatcher(request.getServletPath() + alias);
                     try {
@@ -189,14 +189,14 @@ public class CmsEvents {
             boolean hasErrorPage = false;
 
             if (contentId != null) {
-            try {
-                statusCode = verifyContentToWebSite(delegator, webSiteId, contentId);
-            } catch (GeneralException e) {
-                Debug.logError(e, module);
-                throw new GeneralRuntimeException(e.getMessage(), e);
-            }
+                try {
+                    statusCode = verifyContentToWebSite(delegator, webSiteId, contentId);
+                } catch (GeneralException e) {
+                    Debug.logError(e, module);
+                    throw new GeneralRuntimeException(e.getMessage(), e);
+                }
             } else {
-             statusCode = HttpServletResponse.SC_NOT_FOUND;
+                statusCode = HttpServletResponse.SC_NOT_FOUND;
             }
 
             // We try to find a specific Error page for this website concerning the status code
@@ -225,14 +225,16 @@ public class CmsEvents {
                         Debug.logError(e, module);
                     }
                     if (errorPage != null) {
-                        if (Debug.verboseOn()) Debug.logVerbose("Found error pages " + statusCode + " : " + errorPage, module);
+                        if (Debug.verboseOn())
+                            Debug.logVerbose("Found error pages " + statusCode + " : " + errorPage, module);
                         contentId = errorPage.getString("contentId");
                     } else {
-                        if (Debug.verboseOn()) Debug.logVerbose("No specific error page, falling back to the Error Container for " + statusCode, module);
+                        if (Debug.verboseOn())
+                            Debug.logVerbose("No specific error page, falling back to the Error Container for " + statusCode, module);
                         contentId = errorContainer.getString("contentId");
                     }
                     mapKey = null;
-                    hasErrorPage=true;
+                    hasErrorPage = true;
                 }
                 // We try to find a generic content Error page concerning the status code
                 if (!hasErrorPage) {
@@ -273,7 +275,7 @@ public class CmsEvents {
                     // TODO: replace "screen" to support dynamic rendering of different output
                     FormStringRenderer formStringRenderer = new MacroFormRenderer(EntityUtilProperties.getPropertyValue("widget", "screen.formrenderer", delegator), request, response);
                     templateMap.put("formStringRenderer", formStringRenderer);
-                    
+
                     // if use web analytics
                     List<GenericValue> webAnalytics = EntityQuery.use(delegator).from("WebAnalyticsConfig").where("webSiteId", webSiteId).queryList();
                     // render
@@ -286,11 +288,11 @@ public class CmsEvents {
                     }
 
                 } catch (TemplateException e) {
-                 throw new GeneralRuntimeException(String.format("Error creating form renderer while rendering content [%s] with path alias [%s]", contentId, pathInfo), e);
+                    throw new GeneralRuntimeException(String.format("Error creating form renderer while rendering content [%s] with path alias [%s]", contentId, pathInfo), e);
                 } catch (IOException e) {
-                 throw new GeneralRuntimeException(String.format("Error in the response writer/output stream while rendering content [%s] with path alias [%s]", contentId, pathInfo), e);
+                    throw new GeneralRuntimeException(String.format("Error in the response writer/output stream while rendering content [%s] with path alias [%s]", contentId, pathInfo), e);
                 } catch (GeneralException e) {
-                 throw new GeneralRuntimeException(String.format("Error rendering content [%s] with path alias [%s]", contentId, pathInfo), e);
+                    throw new GeneralRuntimeException(String.format("Error rendering content [%s] with path alias [%s]", contentId, pathInfo), e);
                 }
 
                 return "success";
@@ -318,18 +320,18 @@ public class CmsEvents {
         try {
             webSite = EntityQuery.use(delegator).from("WebSite").where("webSiteId", webSiteId).cache().queryOne();
             if (webSite != null) {
-             siteName = webSite.getString("siteName");
+                siteName = webSite.getString("siteName");
             }
             if (siteName == null) {
-             siteName = "Not specified";
+                siteName = "Not specified";
             }
         } catch (GenericEntityException e) {
             Debug.logError(e, module);
         }
         if (webSite != null) {
-         request.setAttribute("_ERROR_MESSAGE_", "Not able to find a page to display for website: " + siteName + " [" + webSiteId + "] not even a default page!");
+            request.setAttribute("_ERROR_MESSAGE_", "Not able to find a page to display for website: " + siteName + " [" + webSiteId + "] not even a default page!");
         } else {
-         request.setAttribute("_ERROR_MESSAGE_", "Not able to find a page to display, not even a default page AND the website entity record for WebSiteId:" + webSiteId + " could not be found");
+            request.setAttribute("_ERROR_MESSAGE_", "Not able to find a page to display, not even a default page AND the website entity record for WebSiteId:" + webSiteId + " could not be found");
         }
         return "error";
     }
@@ -360,7 +362,7 @@ public class CmsEvents {
                     .orderBy("-fromDate").cache().filterByDate().queryList();
 
             if (topLevelContentValues != null) {
-                for (GenericValue point: topLevelContentValues) {
+                for (GenericValue point : topLevelContentValues) {
                     int subContentStatusCode = verifySubContent(delegator, contentId, point.getString("contentId"));
                     if (subContentStatusCode == HttpServletResponse.SC_OK) {
                         return HttpServletResponse.SC_OK;