Changeset 13037 for branches/2.3/admin/configuration.php

Timestamp:

Feb 6, 2012, 9:22:13 PM (12 years ago)

Author:

mistic100

Message:

merge r12872 from trunk
bug:2540 When default photos order is only by rank, failure outside albums

File:

• branches/2.3/admin/configuration.php (modified) (2 diffs)

branches/2.3/admin/configuration.php

@@ -135 +135 @@
           $order_by = array();
           $order_by_inside_category = array();
+          
           for ($i=0; $i<count($_POST['order_by_field']); $i++)
           {
-            if ($i>5) continue;
-            if ($_POST['order_by_field'][$i] == '')
+            if ( $i >= (count($sort_fields)-1) ) break; // limit to the number of available parameters
+            if ( empty($_POST['order_by_field'][$i]) )
             {
               array_push($page['errors'], l10n('No field selected'));
+              break;
             }
             else
             {
+              // there is no rank outside categories
               if ($_POST['order_by_field'][$i] != 'rank')
               {
@@ -151 +154 @@
             }
           }
+          // must define a default order_by if user want to order by rank only
+          if ( count($order_by) == 0 )
+          {
+            $order_by = array('id ASC');
+          }
+          
           $_POST['order_by'] = 'ORDER BY '.implode(', ', $order_by);
           $_POST['order_by_inside_category'] = 'ORDER BY '.implode(', ', $order_by_inside_category);
+          unset($_POST['order_by_field']);
         }
       }