[ofbiz-framework] branch trunk updated: Improved: multi-block attribute for html-template tag (OFBIZ-11686)

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

[ofbiz-framework] branch trunk updated: Improved: multi-block attribute for html-template tag (OFBIZ-11686)

jleroux@apache.org
This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 7647685  Improved: multi-block attribute for html-template tag (OFBIZ-11686)
7647685 is described below

commit 76476856214a5891ade406c1fc2fbd7cd8c90cb5
Author: Jacques Le Roux <[hidden email]>
AuthorDate: Mon Aug 10 11:05:39 2020 +0200

    Improved: multi-block attribute for html-template tag (OFBIZ-11686)
   
    Fixes JavaDoc issues
---
 build.gradle                                             |  2 +-
 .../java/org/apache/ofbiz/widget/model/HtmlWidget.java   | 16 +++++++---------
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/build.gradle b/build.gradle
index dc6d26c..1d85a90 100644
--- a/build.gradle
+++ b/build.gradle
@@ -287,7 +287,7 @@ checkstyle {
     // the sum of errors found last time it was changed after using the
     // ‘checkstyle’ tool present in the framework and in the official
     // plugins.
-    tasks.checkstyleMain.maxErrors = 10184
+    tasks.checkstyleMain.maxErrors = 10185
     // Currently there are a lot of errors so we need to temporarily
     // hide them to avoid polluting the terminal output.
     showViolations = false
diff --git a/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java b/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java
index a3ec19f..00d83d8 100644
--- a/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java
+++ b/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java
@@ -30,6 +30,8 @@ import java.util.Map;
 import java.util.Set;
 import java.util.Stack;
 
+import javax.servlet.http.HttpServletRequest;
+
 import org.apache.ofbiz.base.util.Debug;
 import org.apache.ofbiz.base.util.GeneralException;
 import org.apache.ofbiz.base.util.UtilCodec;
@@ -59,8 +61,6 @@ import freemarker.template.TemplateModel;
 import freemarker.template.TemplateModelException;
 import freemarker.template.Version;
 
-import javax.servlet.http.HttpServletRequest;
-
 /**
  * Widget Library - Screen model HTML class.
  */
@@ -188,13 +188,11 @@ public class HtmlWidget extends ModelScreenWidget {
     }
 
     /**
-     * Render html template when multi-block=true.
-     * We use stack to store the string writer because a freemarker template may also render a sub screen widget
-     * by using ${screens.render(<link to the screen>)}. So before rendering the sub screen widget, ScreenRenderer
-     * class will check for the existence of the stack and retrieve the correct string writer.
-     * The following tags are removed from the final rendering:
-     * 1. External and inline javascript tags
-     * 2. Css link tags
+     * Render html template when multi-block=true. We use stack to store the string writer because a freemarker template may also render a sub screen
+     * widget by using ${screens.render(link to the screen)}. So before rendering the sub screen widget, ScreenRenderer class will check for the
+     * existence of the stack and retrieve the correct string writer. The following tags are removed from the final rendering: 1. External and inline
+     * javascript tags 2. Css link tags
+     *
      * @param writer
      * @param locationExdr
      * @param context