Changeset 25489

Timestamp:

Nov 15, 2013, 4:41:11 PM (10 years ago)

Author:

mistic100

Message:

bug 2883: filter order_by for duplicated values

Location:

trunk/admin

Files:

• configuration.php (modified) (1 diff)
• themes/default/template/configuration.tpl (modified) (1 diff)

trunk/admin/configuration.php

@@ -153 +153 @@
         if ( !empty($_POST['order_by']) )
         {
+          $used = array();
           foreach ($_POST['order_by'] as $i => $val)
           {
-            if (empty($val)) unset($_POST['order_by'][$i]);
+            if (empty($val) or isset($used[$val]))
+            {
+              unset($_POST['order_by'][$i]);
+            }
+            else
+            {
+              $used[$val] = true;
+            }
           }
           if ( !count($_POST['order_by']) )

trunk/admin/themes/default/template/configuration.tpl

@@ -69 +69 @@
 {footer_script require='jquery'}
 (function(){
-// counters for displaying of addFilter link
-var fields = {$main.order_by|@count},
-    max_fields = Math.ceil({$main.order_by_options|@count}/2);
-
-function updateAddFilterLink() {
-  if (fields >= max_fields) {
-    jQuery('.addFilter').css('display', 'none');
-  } else {
-    jQuery('.addFilter').css('display', '');
-  }
+var max_fields = Math.ceil({$main.order_by_options|@count}/2);
+
+function updateFilters() {
+  var $selects = jQuery('#order_filters select');
+
+  jQuery('#order_filters .addFilter').toggle($selects.length <= max_fields);
+  jQuery('#order_filters .removeFilter').css('display', '').filter(':first').css('display', 'none');
+  
+  $selects.find('option').removeAttr('disabled');
+  $selects.each(function() {
+    $selects.not(this).find('option[value="'+ jQuery(this).val() +'"]').attr('disabled', 'disabled');
+  });
 }
 
-function updateRemoveFilterTrigger() {
-  jQuery(".removeFilter").click(function() {
-    jQuery(this).parent('span.filter').remove();
-    fields--;
-    updateAddFilterLink();
-  });
-
-  jQuery(".removeFilter").css('display', '');
-  jQuery(".filter:first .removeFilter").css('display', 'none');
-}
-
-
-jQuery('.addFilter').click(function() {
-  jQuery(this).prev('span.filter').clone().insertBefore($(this));
-  jQuery(this).prev('span.filter').children('select[name="order_by[]"]').val('');
-
-  fields++;
-  updateRemoveFilterTrigger();
-  updateAddFilterLink();
+jQuery('#order_filters').on('click', '.removeFilter', function() {
+  jQuery(this).parent('span.filter').remove();
+  updateFilters();
 });
 
-updateRemoveFilterTrigger();
-updateAddFilterLink();
+jQuery('#order_filters').on('change', 'select', updateFilters);
+
+jQuery('#order_filters .addFilter').click(function() {
+  jQuery(this).prev('span.filter').clone().insertBefore(jQuery(this));
+  jQuery(this).prev('span.filter').children('select').val('');
+  updateFilters();
+});
+
+updateFilters();
 }());
 {/footer_script}