[ofbiz-framework] branch trunk updated: Improved: Move page-specific script links to html template (OFBIZ-11799)

[James Yong-2]

This is an automated email from the ASF dual-hosted git repository.

jamesyong 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 135c7f5  Improved: Move page-specific script links to html template (OFBIZ-11799)
135c7f5 is described below

commit 135c7f5158d4c0b0eb548821d78bfbbd1aef625d
Author: James Yong <[hidden email]>
AuthorDate: Sun Aug 23 13:57:03 2020 +0800

    Improved: Move page-specific script links to html template (OFBIZ-11799)
   
    Use importLibrary js function to load jstree js at CategoryTree.ftl
    Tested at
    https://localhost:8443/catalog/control/main
---
 .../product/template/category/CategoryTree.ftl     | 81 ++++++++++++----------
 1 file changed, 43 insertions(+), 38 deletions(-)

diff --git a/applications/product/template/category/CategoryTree.ftl b/applications/product/template/category/CategoryTree.ftl
index dcb6a47..3a995c6 100644
--- a/applications/product/template/category/CategoryTree.ftl
+++ b/applications/product/template/category/CategoryTree.ftl
@@ -17,8 +17,6 @@ specific language governing permissions and limitations
 under the License.
 -->
 
-<script src="/common/js/jquery/plugins/jsTree/jquery.jstree.js" type="application/javascript" data-import="head"></script>
-
 <script type="application/javascript">
 <#-- some labels are not unescaped in the JSON object so we have to do this manualy -->
 function unescapeHtmlText(text) {
@@ -55,46 +53,53 @@ var rawdata = [
  <#-- create Tree-->
   function createTree() {
     jQuery(function () {
-        <#-- reset the tree when user browsing out of scope of catalog manager -->
-        <#if stillInCatalogManager>
+        importLibrary(["/common/js/jquery/plugins/jsTree/jquery.jstree.js"], function() {
+            <#-- reset the tree when user browsing out of scope of catalog manager -->
+            <#if stillInCatalogManager>
             $.cookie('jstree_select', null);
             $.cookie('jstree_open', null);
-        <#else>
-        <#-- Coloring the category when type the product categoryId manualy at the url bar -->
+            <#else>
+            <#-- Coloring the category when type the product categoryId manualy at the url bar -->
             $.cookie('jstree_select', "<#if productCategoryId??>${productCategoryId}<#elseif prodCatalogId??>${prodCatalogId}<#elseif showProductCategoryId??>${showProductCategoryId}</#if>");
-        </#if>
-        jQuery("#tree").jstree({
-            "plugins" : [ "themes", "json_data","ui" ,"cookies", "types"],
-            "json_data" : {
-                "data" : rawdata,
-                "ajax" : { "url" : "getChild",
-                           "type" : "POST",
-                           "data" : function (n) {
-                                        return {
-                                            "isCategoryType" :  n.attr ? n.attr("isCatalog").replace("node_","") : 1 ,
-                                            "isCatalog" :  n.attr ? n.attr("isCatalog").replace("node_","") : 1 ,
-                                            "productCategoryId" : n.attr ? n.attr("id").replace("node_","") : 1 ,
-                                            "additionParam" : "','category" ,
-                                            "hrefString" : "EditCategory?productCategoryId=" ,
-                                            "onclickFunction" : "callDocument"
-                                        };
-                                    },
-                           success : function(data) {
-                               return data.treeData;
-                           }
+            </#if>
+            jQuery("#tree").jstree({
+                "plugins": ["themes", "json_data", "ui", "cookies", "types"],
+                "json_data": {
+                    "data": rawdata,
+                    "ajax": {
+                        "url": "getChild",
+                        "type": "POST",
+                        "data": function (n) {
+                            return {
+                                "isCategoryType": n.attr ? n.attr("isCatalog").replace("node_", "") : 1,
+                                "isCatalog": n.attr ? n.attr("isCatalog").replace("node_", "") : 1,
+                                "productCategoryId": n.attr ? n.attr("id").replace("node_", "") : 1,
+                                "additionParam": "','category",
+                                "hrefString": "EditCategory?productCategoryId=",
+                                "onclickFunction": "callDocument"
+                            };
+                        },
+                        success: function (data) {
+                            return data.treeData;
+                        }
+                    }
+                },
+                "types": {
+                    "valid_children": ["root"],
+                    "types": {
+                        "CATEGORY": {
+                            "icon": {
+                                "image": "/common/js/jquery/plugins/jsTree/themes/apple/d.png",
+                                "position": "10px40px"
+                            }
+                        }
+                    }
+                },
+                "themes": {
+                    "theme": "default",
+                    "url": "/common/js/jquery/plugins/jsTree/themes/default/style.css"
                 }
-            },
-            "types" : {
-             "valid_children" : [ "root" ],
-             "types" : {
-                 "CATEGORY" : {
-                     "icon" : {
-                         "image" : "/common/js/jquery/plugins/jsTree/themes/apple/d.png",
-                         "position" : "10px40px"
-                     }
-                 }
-             }
-         }
+            });
         });
     });
   }