Changeset 6698 for trunk/admin/cat_list.php

Timestamp:

Jul 23, 2010, 11:33:56 PM (14 years ago)

Author:

plg

Message:

feature 408: ability to automatically sort sub-categories. The number of SQL
queries is constant, no matter the deepth of your tree.

+ refactor to avoid duplicate code (ascending/desceding should be only a
single parameter in a function, not 20 lines of duplicated code)

File:

• trunk/admin/cat_list.php (modified) (2 diffs)

trunk/admin/cat_list.php

@@ -56 +56 @@
 function save_categories_order($categories)
 {
+  $current_rank_for_id_uppercat = array();
   $current_rank = 0;
+  
   $datas = array();
-  foreach ($categories as $id)
-  {
-    array_push($datas, array('id' => $id, 'rank' => ++$current_rank));
+  foreach ($categories as $category)
+  {
+    if (is_array($category))
+    {
+      $id = $category['id'];
+      $id_uppercat = $category['id_uppercat'];
+
+      if (!isset($current_rank_for_id_uppercat[$id_uppercat]))
+      {
+        $current_rank_for_id_uppercat[$id_uppercat] = 0;
+      }
+      $current_rank = ++$current_rank_for_id_uppercat[$id_uppercat];
+    }
+    else
+    {
+      $id = $category;
+      $current_rank++;
+    }
+    
+    array_push($datas, array('id' => $id, 'rank' => $current_rank));
   }
   $fields = array('primary' => array('id'), 'update' => array('rank'));
@@ -112 +131 @@
 else if (isset($_POST['submitOrder']))
 {
-  asort($_POST['catOrd'], SORT_NUMERIC);
-  save_categories_order(array_keys($_POST['catOrd']));
-
-  array_push(
-    $page['infos'],
-    l10n('Categories manual order was saved')
-    );
-}
-// sort categories alpha-numerically
-else if (isset($_POST['submitOrderAlphaNum']))
-{
-  $query = '
-SELECT id, name
+  if ('manual' == $_POST['order_type'])
+  {
+    asort($_POST['catOrd'], SORT_NUMERIC);
+    save_categories_order(array_keys($_POST['catOrd']));
+
+    array_push(
+      $page['infos'],
+      l10n('Categories manual order was saved')
+      );
+  }
+  else
+  {
+    $query = '
+SELECT id
   FROM '.CATEGORIES_TABLE.'
   WHERE id_uppercat '.
     (!isset($_GET['parent_id']) ? 'IS NULL' : '= '.$_GET['parent_id']).'
 ;';
-  $result = pwg_query($query);
-  while ($row = pwg_db_fetch_assoc($result))
-  {
-    $categories[ $row['id'] ] = strtolower($row['name']);
-  }
-
-  asort($categories, SORT_REGULAR);
-  save_categories_order(array_keys($categories));
-
-  array_push(
-    $page['infos'],
-    l10n('Categories ordered alphanumerically')
-    );
-}
-// sort categories alpha-numerically reverse
-else if (isset($_POST['submitOrderAlphaNumReverse']))
-{
-  $query = '
-SELECT id, name
+    $category_ids = array_from_query($query, 'id');
+
+    if (isset($_POST['recursive']))
+    {
+      $category_ids = get_subcat_ids($category_ids);
+    }
+
+    $categories = array();
+    $names = array();
+    $id_uppercats = array();
+  
+    $query = '
+SELECT id, name, id_uppercat
   FROM '.CATEGORIES_TABLE.'
-  WHERE id_uppercat '.
-    (!isset($_GET['parent_id']) ? 'IS NULL' : '= '.$_GET['parent_id']).'
+  WHERE id IN ('.implode(',', $category_ids).')
 ;';
-  $result = pwg_query($query);
-  while ($row = pwg_db_fetch_assoc($result))
-  {
-    $categories[ $row['id'] ] = strtolower($row['name']);
-  }
-
-  arsort($categories, SORT_REGULAR);
-  save_categories_order(array_keys($categories));
-
-  array_push(
-    $page['infos'],
-    l10n('Categories ordered alphanumerically reverse')
-    );
+    $result = pwg_query($query);
+    while ($row = pwg_db_fetch_assoc($result))
+    {
+      array_push(
+        $categories,
+        array(
+          'id' => $row['id'],
+          'id_uppercat' => $row['id_uppercat'],
+          )
+        );
+      array_push(
+        $names,
+        $row['name']
+        );
+    }
+
+    array_multisort(
+      $names,
+      SORT_REGULAR,
+      'asc' == $_POST['ascdesc'] ? SORT_ASC : SORT_DESC,
+      $categories
+      );
+    save_categories_order($categories);
+
+    array_push(
+      $page['infos'],
+      l10n('Categories automatically sorted')
+      );
+  }
 }