Changeset 23813 for trunk/admin

Timestamp:

Jul 6, 2013, 8:31:14 PM (11 years ago)

Author:

mistic100

Message:

bug 2931: revert r23675 & r23721, make element_set_ranks.php uses ASC keyword (+ migration task)

Location:

trunk/admin

Files:

• configuration.php (modified) (4 diffs)
• element_set_ranks.php (modified) (2 diffs)

trunk/admin/configuration.php

@@ -113 +113 @@
 $sort_fields = array(
   ''                    => '',
-  'file'                => l10n('File name, A → Z'),
+  'file ASC'            => l10n('File name, A → Z'),
   'file DESC'           => l10n('File name, Z → A'),
-  'name'                => l10n('Photo title, A → Z'),
+  'name ASC'            => l10n('Photo title, A → Z'),
   'name DESC'           => l10n('Photo title, Z → A'),
   'date_creation DESC'  => l10n('Date created, new → old'),
-  'date_creation'       => l10n('Date created, old → new'),
+  'date_creation ASC'   => l10n('Date created, old → new'),
   'date_available DESC' => l10n('Date posted, new → old'),
-  'date_available'      => l10n('Date posted, old → new'),
+  'date_available ASC'  => l10n('Date posted, old → new'),
   'rating_score DESC'   => l10n('Rating score, high → low'),
-  'rating_score'        => l10n('Rating score, low → high'),
+  'rating_score ASC'    => l10n('Rating score, low → high'),
   'hit DESC'            => l10n('Visits, high → low'),
-  'hit'                 => l10n('Visits, low → high'),
-  'id'                  => l10n('Numeric identifier, 1 → 9'),
+  'hit ASC'             => l10n('Visits, low → high'),
+  'id ASC'              => l10n('Numeric identifier, 1 → 9'),
   'id DESC'             => l10n('Numeric identifier, 9 → 1'),
-  'rank'                => l10n('Manual sort order'),
+  'rank ASC'            => l10n('Manual sort order'),
   );
 
@@ -163 +163 @@
 
             // there is no rank outside categories
-            if ( ($i = array_search('rank', $order_by)) !== false)
+            if ( ($i = array_search('rank ASC', $order_by)) !== false)
             {
               unset($order_by[$i]);
@@ -171 +171 @@
             if ( count($order_by) == 0 )
             {
-              $order_by = array('id');
+              $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);
+            $_POST['order_by'] = 'ORDER BY '.implode(', ', $order_by);
+            $_POST['order_by_inside_category'] = 'ORDER BY '.implode(', ', $order_by_inside_category);
           }
         }
@@ -335 +335 @@
     else
     {
+      $out = array();
       $order_by = trim($conf['order_by_inside_category']);
       $order_by = str_replace('ORDER BY ', null, $order_by);
-      $order_by = explode(',', $order_by);
-      foreach($order_by as &$order)
-      {
-        $order = trim($order);
-        if (substr_compare($order, ' ASC', -4)==0)
-        {
-          $order = substr($order, 0, -4);
-        }
-      }
-      unset($order);
+      $order_by = explode(', ', $order_by);
     }
 

trunk/admin/element_set_ranks.php

@@ -125 +125 @@
   elseif ($image_order_choice=='rank')
   {
-    $image_order = 'rank';
+    $image_order = 'rank ASC';
   }
   $query = '
@@ -238 +238 @@
 $sort_fields = array(
   ''                    => '',
-  'file'                => l10n('File name, A → Z'),
+  'file ASC'            => l10n('File name, A → Z'),
   'file DESC'           => l10n('File name, Z → A'),
-  'name'                => l10n('Photo title, A → Z'),
+  'name ASC'            => l10n('Photo title, A → Z'),
   'name DESC'           => l10n('Photo title, Z → A'),
   'date_creation DESC'  => l10n('Date created, new → old'),
-  'date_creation'       => l10n('Date created, old → new'),
+  'date_creation ASC'   => l10n('Date created, old → new'),
   'date_available DESC' => l10n('Date posted, new → old'),
-  'date_available'      => l10n('Date posted, old → new'),
+  'date_available ASC'  => l10n('Date posted, old → new'),
   'rating_score DESC'   => l10n('Rating score, high → low'),
-  'rating_score'        => l10n('Rating score, low → high'),
+  'rating_score ASC'    => l10n('Rating score, low → high'),
   'hit DESC'            => l10n('Visits, high → low'),
-  'hit'                 => l10n('Visits, low → high'),
-  'id'                  => l10n('Numeric identifier, 1 → 9'),
+  'hit ASC'             => l10n('Visits, low → high'),
+  'id ASC'              => l10n('Numeric identifier, 1 → 9'),
   'id DESC'             => l10n('Numeric identifier, 9 → 1'),
-  'rank'                => l10n('Manual sort order'),
+  'rank ASC'            => l10n('Manual sort order'),
   );