You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2018/06/04 08:57:48 UTC

svn commit: r1832802 - in /ofbiz/ofbiz-plugins/branches/release17.12: ./ ecommerce/groovyScripts/catalog/AjaxBreadcrumbs.groovy ecommerce/widget/CommonScreens.xml

Author: jleroux
Date: Mon Jun  4 08:57:48 2018
New Revision: 1832802

URL: http://svn.apache.org/viewvc?rev=1832802&view=rev
Log:
"Applied fix from plugins for revision: 1832800  " 
------------------------------------------------------------------------
r1832800 | jleroux | 2018-06-04 10:34:35 +0200 (lun. 04 juin 2018) | 11 lignes

Fixed: Click on category the filter function doesn't work
(OFBIZ-10423)

Click on the left category, the page doesn't refresh, there are two issues.

productCategoryId not set in 
   ecommerce/groovyScripts/catalog/AjaxBreadcrumbs.groovy
updated data will fill in #div3, but the div id is missing in 
   ecommerce/widget/CommonScreens.xml
 
Thanks: Jason Hao
------------------------------------------------------------------------

Modified:
    ofbiz/ofbiz-plugins/branches/release17.12/   (props changed)
    ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/groovyScripts/catalog/AjaxBreadcrumbs.groovy
    ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/widget/CommonScreens.xml

Propchange: ofbiz/ofbiz-plugins/branches/release17.12/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jun  4 08:57:48 2018
@@ -10,4 +10,4 @@
 /ofbiz/branches/json-integration-refactoring/plugins:1634077-1635900
 /ofbiz/branches/multitenant20100310/plugins:921280-927264
 /ofbiz/branches/release13.07/plugins:1547657
-/ofbiz/ofbiz-plugins/trunk:1819576,1819580,1819582,1819589,1820962,1820973,1821045,1821049,1821051,1822129,1822133,1822139,1822143,1822145,1822389,1822391,1822395,1824646,1824648,1824650,1824654,1828569,1830856,1831084,1832727
+/ofbiz/ofbiz-plugins/trunk:1819576,1819580,1819582,1819589,1820962,1820973,1821045,1821049,1821051,1822129,1822133,1822139,1822143,1822145,1822389,1822391,1822395,1824646,1824648,1824650,1824654,1828569,1830856,1831084,1832727,1832800

Modified: ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/groovyScripts/catalog/AjaxBreadcrumbs.groovy
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/groovyScripts/catalog/AjaxBreadcrumbs.groovy?rev=1832802&r1=1832801&r2=1832802&view=diff
==============================================================================
--- ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/groovyScripts/catalog/AjaxBreadcrumbs.groovy (original)
+++ ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/groovyScripts/catalog/AjaxBreadcrumbs.groovy Mon Jun  4 08:57:48 2018
@@ -21,6 +21,7 @@ import org.apache.ofbiz.product.category
 import org.apache.ofbiz.base.util.UtilValidate
 
 parentCategoryStr = parameters.parentCategoryStr
+productCategoryId=parameters.category_id
 if(!UtilValidate.isEmpty(parentCategoryStr)) {
     pathList = parentCategoryStr.split('/')
     cateList = []

Modified: ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/widget/CommonScreens.xml
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/widget/CommonScreens.xml?rev=1832802&r1=1832801&r2=1832802&view=diff
==============================================================================
--- ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/widget/CommonScreens.xml (original)
+++ ofbiz/ofbiz-plugins/branches/release17.12/ecommerce/widget/CommonScreens.xml Mon Jun  4 08:57:48 2018
@@ -86,13 +86,13 @@ under the License.
 
                      <container id="ecom-mainarea" style="row mx-2">
 
-                         <container style="col-lg-2 order-lg-1 order-2"><include-screen name="${leftbarScreenName}" location="${leftbarScreenLocation}"/></container>
-                         <container style="col-lg-8 order-lg-2 order-1">
+                         <container id="div1" style="col-lg-2 order-lg-1 order-2"><include-screen name="${leftbarScreenName}" location="${leftbarScreenLocation}"/></container>
+                         <container id="div3" style="col-lg-8 order-lg-2 order-1">
                              <!-- render messages -->
                              <platform-specific><html><html-template location="component://common-theme/template/includes/Messages.ftl"/></html></platform-specific>
                              <decorator-section-include name="body"/>
                          </container>
-                             <container style="col-lg-2 order-lg-3 order-3"><include-screen name="${rightbarScreenName}" location="${rightbarScreenLocation}"/></container>
+                             <container id="div2" style="col-lg-2 order-lg-3 order-3"><include-screen name="${rightbarScreenName}" location="${rightbarScreenLocation}"/></container>
                          <container id="div4" style="endcolumns"/>
                     </container>