Merge pull request #1313 from macmastermike/fix_frontend_view_dropdowns

Fixes the category items-per-page dropdown in the frontend (didn't wo…
coreshop · Mar 5, 2020 · dfa48b8 · dfa48b8
2 parents e02653d + fa67577
commit dfa48b8
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/src/CoreShop/Bundle/FrontendBundle/Resources/public/static/js/shop.js b/src/CoreShop/Bundle/FrontendBundle/Resources/public/static/js/shop.js
@@ -8,6 +8,7 @@ $(document).ready(function () {
         shop.initChangeAddress();
         shop.initCartShipmentCalculator();
         shop.initQuantityValidator();
+        shop.initCategorySelect();
 
         $('#paymentProvider').handlePrototypes({
             'prototypePrefix': 'paymentProvider',
@@ -16,6 +17,22 @@ $(document).ready(function () {
         });
     };
 
+    shop.initCategorySelect = function () {
+        function updateQueryStringParameter(uri, key, value) {
+            var re = new RegExp("([?&])" + key + "=.*?(&|$)", "i");
+            var separator = uri.indexOf('?') !== -1 ? "&" : "?";
+            if (uri.match(re)) {
+                return uri.replace(re, '$1' + key + "=" + value + '$2');
+            }
+            else {
+                return uri + separator + key + "=" + value;
+            }
+        }
+        $(".site-reload").change(function() {
+            location.href= updateQueryStringParameter( window.location.href, this.name, this.value );
+        });
+    };
+
     shop.initQuantityValidator = function () {
         $.coreshopQuantitySelector({
             buttondown_class: 'btn btn-secondary',

diff --git a/src/CoreShop/Bundle/FrontendBundle/Resources/views/Category/index.html.twig b/src/CoreShop/Bundle/FrontendBundle/Resources/views/Category/index.html.twig
@@ -47,8 +47,8 @@
             </div>
             <div class="col-md-2 text-right">
                 <select class="form-control site-reload" name="perPage">
-                    {% for perPage in perPageAllowed %}
-                        <option value="{{ perPage }}" {% if perPage == perPage %} selected="selected" {% endif %}>{{ perPage }}</option>
+                    {% for perPageList in perPageAllowed %}
+                        <option value="{{ perPageList }}" {% if perPageList == perPage %} selected="selected" {% endif %}>{{ perPageList }}</option>
                     {% endfor %}
                 </select>
             </div>