[jira] [Commented] (OFBIZ-6798) Broken filter links if using ecommerce Layered Navigation

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

[jira] [Commented] (OFBIZ-6798) Broken filter links if using ecommerce Layered Navigation

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-6798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15096273#comment-15096273 ]

Jacques Le Roux commented on OFBIZ-6798:
----------------------------------------

I just tried but had also to change
{code}
Index: specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/catalog/LayeredNavigation.groovy
===================================================================
--- specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/catalog/LayeredNavigation.groovy (revision 1723830)
+++ specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/catalog/LayeredNavigation.groovy (working copy)
@@ -59,7 +59,7 @@
     productCategoryRollups = from("ProductCategoryRollup").where("productCategoryId", searchCategoryId).filterByDate().queryList();
     previousCategoryId = null;
     if (productCategoryRollups) {
-        for (GenericValue categoryRollup : productCategoryRollups) {
+        for (categoryRollup in productCategoryRollups) {
             categoryConstraint = new ProductSearch.CategoryConstraint(categoryRollup.parentProductCategoryId, true, false);
             if (searchConstraintList.contains(categoryConstraint)) {
                 previousCategoryId = categoryRollup.parentProductCategoryId;
{code}
Now, what does this add? I don't see much differences, were you just curious?

> Broken filter links if using ecommerce Layered Navigation
> ---------------------------------------------------------
>
>                 Key: OFBIZ-6798
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-6798
>             Project: OFBiz
>          Issue Type: Bug
>          Components: specialpurpose/ecommerce
>    Affects Versions: Trunk
>         Environment: Any
>            Reporter: Jason Reast-Jones
>            Assignee: Jacques Le Roux
>            Priority: Minor
>              Labels: newbie, patch
>         Attachments: LayeredNavBar.ftl.patch
>
>
> If you enable Layered Navigation in the ecommerce (via CatalogScreens.xml) the Sub-Category and color filter links have no closing </a> tag.
> Patch attached.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)