Changeset 642 for trunk/admin/update.php

Timestamp:

Dec 12, 2004, 10:06:39 PM (19 years ago)

Author:

plg

Message:

• in admin menu, status option for categories is not "permissions" but "private or public" choice = different language item

• get_cat_display_name changed : use $conflevel_separator to unify presentation

• default values for category properties commentable, uploadable, status and visible (set in include/config.inc.php) used for category creation (admin/update, admin/remote_site, admin/cat_list)

• use mass_inserts in admin/update for inserting new categories

• only one query for counting the number of sub categories in admin/cat_list

File:

• trunk/admin/update.php (modified) (7 diffs)

trunk/admin/update.php

@@ -48 +48 @@
   if (is_numeric($id_uppercat))
   {
-    $query = 'SELECT name,uppercats,dir FROM '.CATEGORIES_TABLE;
-    $query.= ' WHERE id = '.$id_uppercat;
-    $query.= ';';
-    $row = mysql_fetch_array( pwg_query( $query));
-    $uppercats = $row['uppercats'];
-    $name      = $row['name'];
-    $dir       = $row['dir'];
-
-    $upper_array = explode( ',', $uppercats);
+    $query = '
+SELECT id,name,uppercats,dir,visible,status
+  FROM '.CATEGORIES_TABLE.'
+  WHERE id = '.$id_uppercat.'
+;';
+    $row = mysql_fetch_array(pwg_query($query));
+    $parent = array('id' => $row['id'],
+                    'name' => $row['name'],
+                    'dir' => $row['dir'],
+                    'uppercats' => $row['uppercats'],
+                    'visible' => $row['visible'],
+                    'status' => $row['status']);
+
+    $upper_array = explode( ',', $parent['uppercats']);
 
     $local_dir = '';
@@ -62 +67 @@
     $database_dirs = array();
     $query = '
-SELECT id,dir FROM '.CATEGORIES_TABLE.'
-  WHERE id IN ('.$uppercats.')
-;';
-    $result = pwg_query( $query);
+SELECT id,dir
+  FROM '.CATEGORIES_TABLE.'
+  WHERE id IN ('.$parent['uppercats'].')
+;';
+    $result = pwg_query($query);
     while ($row = mysql_fetch_array($result))
     {
@@ -79 +85 @@
     // 1. display the category name to update
     $output = '<ul class="menu">';
-    $output.= '<li><strong>'.$name.'</strong>';
-    $output.= ' [ '.$dir.' ]';
+    $output.= '<li><strong>'.$parent['name'].'</strong>';
+    $output.= ' [ '.$parent['dir'].' ]';
     $output.= '</li>';
 
@@ -95 +101 @@
   $sub_category_dirs = array();
   $query = '
-SELECT id,dir FROM '.CATEGORIES_TABLE.'
+SELECT id,dir
+  FROM '.CATEGORIES_TABLE.'
   WHERE site_id = 1
 ';
@@ -132 +139 @@
   // array of new categories to insert
   $inserts = array();
+
+  // calculate default value at category creation
+  $create_values = array();
+  if (isset($parent))
+  {
+    // at creation, must a category be visible or not ? Warning : if
+    // the parent category is invisible, the category is automatically
+    // create invisible. (invisible = locked)
+    if ('false' == $parent['visible'])
+    {
+      $create_values{'visible'} = 'false';
+    }
+    else
+    {
+      $create_values{'visible'} = $conf['newcat_default_visible'];
+    }
+    // at creation, must a category be public or private ? Warning :
+    // if the parent category is private, the category is
+    // automatically create private.
+    if ('private' == $parent['status'])
+    {
+      $create_values{'status'} = 'private';
+    }
+    else
+    {
+      $create_values{'status'} = $conf['newcat_default_status'];
+    }
+  }
+  else
+  {
+    $create_values{'visible'} = $conf['newcat_default_visible'];
+    $create_values{'status'} = $conf['newcat_default_status'];
+  }
   
   foreach ($fs_subdirs as $fs_subdir)
@@ -140 +180 @@
     if (!is_numeric($category_id))
     {
+      $insert = array();
+      
       if (preg_match('/^[a-zA-Z0-9-_.]+$/', $fs_subdir))
       {
         $name = str_replace('_', ' ', $fs_subdir);
 
-        $value = "('".$fs_subdir."','".$name."',1";
-        if (!is_numeric($id_uppercat))
-        {
-          $value.= ',NULL';
-        }
-        else
-        {
-          $value.= ','.$id_uppercat;
-        }
-        $value.= ",'undef'";
-        $value.= ')';
-        array_push($inserts, $value);
+        $insert{'dir'} = $fs_subdir;
+        $insert{'name'} = $name;
+        $insert{'site_id'} = 1;
+        $insert{'uppercats'} = 'undef';
+        $insert{'commentable'} = $conf['newcat_default_commentable'];
+        $insert{'uploadable'} = $conf['newcat_default_uploadable'];
+        $insert{'status'} = $create_values{'status'};
+        $insert{'visible'} = $create_values{'visible'};
+
+        if (isset($parent))
+        {
+          $insert{'id_uppercat'} = $parent['id'];
+        }
+        
+        array_push($inserts, $insert);
       }
       else
@@ -168 +213 @@
   if (count($inserts) > 0)
   {
-    $query = '
-INSERT INTO '.CATEGORIES_TABLE.'
-  (dir,name,site_id,id_uppercat,uppercats) VALUES
-';
-    $query.= implode(',', $inserts);
-    $query.= '
-;';
-    pwg_query($query);
-
+    $dbfields = array(
+      'dir','name','site_id','id_uppercat','uppercats','commentable',
+      'uploadable','visible','status'
+      );
+    mass_inserts(CATEGORIES_TABLE, $dbfields, $inserts);
+    
     $counts['new_categories']+= count($inserts);
     // updating uppercats field
     $query = '
-UPDATE '.CATEGORIES_TABLE.'
-  SET uppercats = ';
-    if ($uppercats != '')
-    {
-      $query.= "CONCAT('".$uppercats."',',',id)";
+UPDATE '.CATEGORIES_TABLE;
+    if (isset($parent))
+    {
+      $query.= "
+  SET uppercats = CONCAT('".$parent['uppercats']."',',',id)
+  WHERE id_uppercat = ".$parent['id'];
     }
     else
     {
-      $query.= 'id';
-    }
-    $query.= '
-  WHERE id_uppercat ';
-    if (!is_numeric($id_uppercat))
-    {
-      $query.= 'IS NULL';
-    }
-    else
-    {
-      $query.= '= '.$id_uppercat;
+      $query.= '
+  SET uppercats = id
+  WHERE id_uppercat IS NULL';
     }
     $query.= '