Changeset 23428

Timestamp:

Jun 21, 2013, 4:09:02 PM (11 years ago)

Author:

flop25

Message:

bug:2855
take care if no parent album, and propagate the permissions
ToDo: check "Apply to sub" checkbox if $conf inheritance_by_default is true, optimizations

File:

• trunk/admin/site_update.php (modified) (4 diffs)

trunk/admin/site_update.php

@@ -267 +267 @@
         array(
           'id' => $insert['id'],
-          'parent' => $parent,
+          'parent' => (isset($parent)) ? $parent : Null,
           'status' => $insert['status'],
           'visible' => $insert['visible'],
@@ -304 +304 @@
       {
         $category_ids[] = $category['id'];
-        $category_up[] = $category['id_uppercat'];
+        if (!empty($category['id_uppercat']))
+        {
+          $category_up[] = $category['id_uppercat'];
+        }
       }
       $category_up=implode(',',array_unique($category_up));
@@ -315 +318 @@
         ;';
         $result = pwg_query($query);
-        $granted_grps = array();
-        while ($row = pwg_db_fetch_assoc($result))
+        if (!empty($result))
         {
-          if (is_null($granted_grps[$row['cat_id']]))
+          $granted_grps = array();
+          while ($row = pwg_db_fetch_assoc($result))
           {
-            $granted_grps[$row['cat_id']]=array();
-          }
-          array_push(
-            $granted_grps,
-            array(
-              $row['cat_id'] => array_push($granted_grps[$row['cat_id']],$row['group_id'])
-            )
-          );
-        }
-        $insert_granted_grps=array();
-        foreach ($category_ids as $ids)
-        {
-          $parent=$db_categories[$ids]['parent'];
-          foreach ($granted_grps[$parent] as $granted_grp)
-          {
+            if (!is_array ($granted_grps[$row['cat_id']]))
+            {
+              $granted_grps[$row['cat_id']]=array();
+            }
             array_push(
-              $insert_granted_grps,
+              $granted_grps,
               array(
-                'group_id' => $granted_grp,
-                'cat_id' => $ids
+                $row['cat_id'] => array_push($granted_grps[$row['cat_id']],$row['group_id'])
               )
             );
-           
           }
+          $insert_granted_grps=array();
+          foreach ($category_ids as $ids)
+          {
+            $parent_id=$db_categories[$ids]['parent'];
+            while (in_array($parent_id, $category_ids))
+            {
+              $parent_id= $db_categories[$parent_id]['parent'];
+            }
+            if ($db_categories[$ids]['status']=='private' and !is_null($parent_id))
+            {
+              foreach ($granted_grps[$parent_id] as $granted_grp)
+              {
+                array_push(
+                  $insert_granted_grps,
+                  array(
+                    'group_id' => $granted_grp,
+                    'cat_id' => $ids
+                  )
+                );
+               
+              }
+            }
+          }
+  
+          mass_inserts(GROUP_ACCESS_TABLE, array('group_id','cat_id'), $insert_granted_grps);
         }
-
-        mass_inserts(GROUP_ACCESS_TABLE, array('group_id','cat_id'), $insert_granted_grps);
 
         $query = '
@@ -354 +367 @@
         ;';
         $result = pwg_query($query);
-        $granted_users = array();
-        while ($row = pwg_db_fetch_assoc($result))
+        if (!empty($result))
         {
-          if (is_null($granted_users[$row['cat_id']]))
+
+          $granted_users = array();
+          while ($row = pwg_db_fetch_assoc($result))
           {
-            $granted_users[$row['cat_id']]=array();
-          }
-          array_push(
-            $granted_users,
-            array(
-              $row['cat_id'] => array_push($granted_users[$row['cat_id']],$row['user_id'])
-            )
-          );
-        }
-        $insert_granted_users=array();
-        foreach ($category_ids as $ids)
-        {
-          $parent=$db_categories[$ids]['parent'];
-          foreach ($granted_users[$parent] as $granted_user)
-          {
+            if (!is_array ($granted_users[$row['cat_id']]))
+            {
+              $granted_users[$row['cat_id']]=array();
+            }
             array_push(
-              $insert_granted_users,
+              $granted_users,
               array(
-                'user_id' => $granted_user,
-                'cat_id' => $ids
+                $row['cat_id'] => array_push($granted_users[$row['cat_id']],$row['user_id'])
               )
             );
-           
           }
+          $insert_granted_users=array();
+          foreach ($category_ids as $ids)
+          {
+            $parent_id=$db_categories[$ids]['parent'];
+             while (in_array($parent_id, $category_ids))
+            {
+              $parent_id= $db_categories[$parent_id]['parent'];
+            }
+            if ($db_categories[$ids]['status']=='private' and !is_null($parent_id))
+            {
+              foreach ($granted_users[$parent_id] as $granted_user)
+              {
+                array_push(
+                  $insert_granted_users,
+                  array(
+                    'user_id' => $granted_user,
+                    'cat_id' => $ids
+                  )
+                );
+              }
+            }
+          }
+          mass_inserts(USER_ACCESS_TABLE, array('user_id','cat_id'), $insert_granted_users);
         }
-        mass_inserts(USER_ACCESS_TABLE, array('user_id','cat_id'), $insert_granted_users);
-
       }     
       else