Changeset 2491 for trunk/admin/site_update.php

Timestamp:

Aug 29, 2008, 2:35:16 PM (16 years ago)

Author:

rvelices

Message:

synchro improvements:

• able to sync metadata at the same time as the files/dirs
• by default empty metadata does not overwrite database infos (checkbox can switch to previous behaviour) (bug 132)
• the form is shown again even after a successfull non simulated run

File:

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

trunk/admin/site_update.php

@@ -43 +43 @@
 SELECT galleries_url
   FROM '.SITES_TABLE.'
-  WHERE id = '.$site_id.'
-;';
+  WHERE id = '.$site_id;
 list($site_url) = mysql_fetch_row(pwg_query($query));
 if (!isset($site_url))
@@ -140 +139 @@
   $start = get_moment();
   // which categories to update ?
-  $cat_ids = array();
-
   $query = '
 SELECT id, uppercats, global_rank, status, visible
@@ -162 +159 @@
     }
   }
-  $query.= '
-;';
-  $result = pwg_query($query);
-
-  $db_categories = array();
-  while ($row = mysql_fetch_array($result))
-  {
-    $db_categories[$row['id']] = $row;
-  }
+  $db_categories = hash_from_query($query, 'id');
 
   // get categort full directories in an array for comparison with file
@@ -195 +184 @@
   $query = '
 SELECT id
-  FROM '.CATEGORIES_TABLE.'
-;';
+  FROM '.CATEGORIES_TABLE;
   $result = pwg_query($query);
   while ($row = mysql_fetch_array($result))
@@ -207 +195 @@
 SELECT id_uppercat, MAX(rank)+1 AS next_rank
   FROM '.CATEGORIES_TABLE.'
-  GROUP BY id_uppercat
-;';
+  GROUP BY id_uppercat';
   $result = pwg_query($query);
   while ($row = mysql_fetch_array($result))
@@ -223 +210 @@
   $query = '
 SELECT IF(MAX(id)+1 IS NULL, 1, MAX(id)+1) AS next_id
-  FROM '.CATEGORIES_TABLE.'
-;';
+  FROM '.CATEGORIES_TABLE;
   list($next_id) = mysql_fetch_array(pwg_query($query));
 
@@ -389 +375 @@
         80,
         "\n"
-        ).')
-;';
-    $result = pwg_query($query);
-    while ($row = mysql_fetch_array($result))
-    {
-      $db_elements[$row['id']] = $row['path'];
-    }
+        ).')';
+    $db_elements = simple_hash_from_query($query, 'id', 'path');
 
     // searching the unvalidated waiting elements (they must not be taken into
@@ -404 +385 @@
   WHERE storage_category_id IN (
 '.wordwrap(implode(', ', $cat_ids), 80, "\n").')
-    AND validated = \'false\'
-;';
+    AND validated = \'false\'';
     $result = pwg_query($query);
     while ($row = mysql_fetch_array($result))
@@ -422 +402 @@
   $query = '
 SELECT IF(MAX(id)+1 IS NULL, 1, MAX(id)+1) AS next_element_id
-  FROM '.IMAGES_TABLE.'
-;';
+  FROM '.IMAGES_TABLE;
   list($next_element_id) = mysql_fetch_array(pwg_query($query));
 
@@ -565 +544 @@
   WHERE storage_category_id IN (
 '.wordwrap(implode(', ', $cat_ids), 80, "\n").')
-    AND validated = \'true\'
-;';
+    AND validated = \'true\'';
     $result = pwg_query($query);
 
@@ -586 +564 @@
   FROM '.IMAGES_TABLE.'
   WHERE storage_category_id = '.$row['storage_category_id'].'
-    AND file = \''.$row['file'].'\'
-;';
+    AND file = \''.$row['file'].'\'';
       list($data['id']) = mysql_fetch_array(pwg_query($query));
 
@@ -724 +701 @@
 // |                          synchronize metadata                         |
 // +-----------------------------------------------------------------------+
-if (isset($_POST['submit']) and preg_match('/^metadata/', $_POST['sync'])
+if (isset($_POST['submit']) and isset($_POST['sync_meta'])
          and !$general_failure)
 {
   // sync only never synchronized files ?
-  if ($_POST['sync'] == 'metadata_new')
-  {
-    $opts['only_new'] = true;
-  }
-  else
-  {
-    $opts['only_new'] = false;
-  }
+  $opts['only_new'] = isset($_POST['meta_all']) ? false : true;
   $opts['category_id'] = '';
   $opts['recursive'] = true;
@@ -777 +747 @@
     AND id IN (
 '.wordwrap(implode(', ', $image_ids), 80, "\n").'
-)
-;';
-
-    $result = pwg_query($query);
-    while ($row = mysql_fetch_array($result))
-    {
-      array_push($has_high_images, $row['id']);
-    }
+)';
+    $has_high_images = array_from_query($query, 'id' );
   }
 
@@ -844 +808 @@
             )
           ),
-        $datas
+        $datas,
+        isset($_POST['meta_empty_overrides']) ? 0 : MASS_UPDATES_SKIP_EMPTY
         );
     }
@@ -894 +859 @@
 // |                        introduction : choices                         |
 // +-----------------------------------------------------------------------+
-if (!isset($_POST['submit']) or (isset($simulate) and $simulate))
-{
-  if (isset($simulate) and $simulate)
-  {
-    $tpl_introduction = array(
-        'sync'  => $_POST['sync'],
-        'display_info' => isset($_POST['display_info']) and $_POST['display_info']==1,
-        'add_to_caddie' => isset($_POST['add_to_caddie']) and $_POST['add_to_caddie']==1,
-        'subcats_included' => isset($_POST['subcats-included']) and $_POST['subcats-included']==1,
-        'privacy_level_selected' => (int)@$_POST['privacy_level'],
-      );
-
-    if (isset($_POST['cat']) and is_numeric($_POST['cat']))
-    {
-      $cat_selected = array($_POST['cat']);
-    }
-    else
-    {
-      $cat_selected = array();
-    }
+if (isset($_POST['submit']))
+{
+  $tpl_introduction = array(
+      'sync'  => $_POST['sync'],
+      'sync_meta'  => isset($_POST['sync_meta']) ? true : false,
+      'display_info' => isset($_POST['display_info']) and $_POST['display_info']==1,
+      'add_to_caddie' => isset($_POST['add_to_caddie']) and $_POST['add_to_caddie']==1,
+      'subcats_included' => isset($_POST['subcats-included']) and $_POST['subcats-included']==1,
+      'privacy_level_selected' => (int)@$_POST['privacy_level'],
+      'meta_all'  => isset($_POST['meta_all']) ? true : false,
+      'meta_empty_overrides'  => isset($_POST['meta_empty_overrides']) ? true : false,
+    );
+
+  if (isset($_POST['cat']) and is_numeric($_POST['cat']))
+  {
+    $cat_selected = array($_POST['cat']);
   }
   else
   {
-    $tpl_introduction = array(
-        'sync'  => 'dirs',
-        'display_info' => false,
-        'add_to_caddie' => false,
-        'subcats_included' => true,
-        'privacy_level_selected' => 0,
-      );
-
     $cat_selected = array();
   }
-
-  $tpl_introduction['privacy_level_options']=array();
-  foreach ($conf['available_permission_levels'] as $level)
-  {
-    $tpl_introduction['privacy_level_options'][$level] = l10n( sprintf('Level %d', $level) );
-  }
-
-  $template->assign('introduction', $tpl_introduction);
-
-  $query = '
+}
+else
+{
+  $tpl_introduction = array(
+      'sync'  => 'dirs',
+      'sync_meta'  => true,
+      'display_info' => false,
+      'add_to_caddie' => false,
+      'subcats_included' => true,
+      'privacy_level_selected' => 0,
+      'meta_all'  => false,
+      'meta_empty_overrides'  => false,
+    );
+
+  $cat_selected = array();
+}
+
+$tpl_introduction['privacy_level_options']=array();
+foreach ($conf['available_permission_levels'] as $level)
+{
+  $tpl_introduction['privacy_level_options'][$level] = l10n( sprintf('Level %d', $level) );
+}
+
+$template->assign('introduction', $tpl_introduction);
+
+$query = '
 SELECT id,name,uppercats,global_rank
   FROM '.CATEGORIES_TABLE.'
-  WHERE site_id = '.$site_id.'
-;';
-  display_select_cat_wrapper($query,
-                             $cat_selected,
-                             'category_options',
-                             false);
-}
+  WHERE site_id = '.$site_id;
+display_select_cat_wrapper($query,
+                           $cat_selected,
+                           'category_options',
+                           false);
+
 
 if (count($errors) > 0)