Bug 703: Authorization system in Cat management works now with well o…

…rdered lists (Minor: Admin theme changes) git-svn-id: http://piwigo.org/svn/trunk@2349 68402e56-0260-453c-a942-63ccdbb3a9ee
Piwigo · May 17, 2008 · 069748a · 069748a
1 parent 6ee69cf
commit 069748a
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 3 deletions.
diff --git a/admin/cat_perm.php b/admin/cat_perm.php
@@ -231,12 +231,13 @@
   WHERE cat_id = '.$page['cat'].'
 ;';
 $group_granted_ids = array_from_query($query, 'group_id');
+$group_granted_ids = order_by_name($group_granted_ids, $groups);
 $template->assign('group_granted_ids', $group_granted_ids);
 
 
 // groups denied
 $template->assign('group_denied_ids',
-      array_diff(array_keys($groups), $group_granted_ids)
+    order_by_name(array_diff(array_keys($groups), $group_granted_ids), $groups)
   );
 
 // users...
@@ -257,7 +258,7 @@
   WHERE cat_id = '.$page['cat'].'
 ;';
 $user_granted_direct_ids = array_from_query($query, 'user_id');
-
+$user_granted_direct_ids = order_by_name($user_granted_direct_ids, $users);
 $template->assign('user_granted_direct_ids', $user_granted_direct_ids);
 
 
@@ -294,7 +295,8 @@
 
   $user_granted_indirect_ids = array_diff($user_granted_by_group_ids,
                                           $user_granted_direct_ids);
-
+  $user_granted_indirect_ids = 
+    order_by_name($user_granted_indirect_ids, $users);  
   foreach ($user_granted_indirect_ids as $user_id)
   {
     foreach ($granted_groups as $group_id => $group_users)
@@ -317,6 +319,7 @@
 $user_denied_ids = array_diff(array_keys($users),
                               $user_granted_indirect_ids,
                               $user_granted_direct_ids);
+$user_denied_ids = order_by_name($user_denied_ids, $users);
 $template->assign('user_denied_ids', $user_denied_ids);
 
 

diff --git a/include/functions_html.inc.php b/include/functions_html.inc.php
@@ -731,4 +731,21 @@ function render_category_literal_description($desc)
 {
   return strip_tags($desc, '<span><p><a><br><b><i><small><big><strong><em>');
 }
+
+/** returns the argument_ids array with new sequenced keys based on related 
+ * names. Sequence is not case sensitive.
+ * Warning: By definition, this function breaks original keys
+ */
+function order_by_name($element_ids,$name)
+{
+  $ordered_element_ids = array();
+  foreach ($element_ids as $k_id => $element_id)
+  {
+    $key = strtolower($name[$element_id]) .'-'. $name[$element_id] .'-'. $k_id;
+    $ordered_element_ids[$key] = $element_id;
+  }
+  ksort($ordered_element_ids);
+  return $ordered_element_ids;
+}
+
 ?>
diff --git a/template/yoga/theme/admin/images/header_bottom.png b/template/yoga/theme/admin/images/header_bottom.png
diff --git a/template/yoga/theme/admin/images/logo23.jpg b/template/yoga/theme/admin/images/logo23.jpg
diff --git a/template/yoga/theme/admin/images/select_bt.png b/template/yoga/theme/admin/images/select_bt.png