Changeset 498

Timestamp:

Aug 29, 2004, 1:21:21 PM (20 years ago)

Author:

z0rglub

Message:

simplier display of update result :

• number of new categories
• number of new elements
• number of deleted categories
• number of deleted elements
• only errors are fully listed

Location:

trunk

Files:

• admin/include/functions.php (modified) (4 diffs)
• admin/update.php (modified) (10 diffs)
• language/en_UK.iso-8859-1/admin.lang.php (modified) (1 diff)
• template/default/admin/update.tpl (modified) (1 diff)
• template/default/default.css (modified) (1 diff)

trunk/admin/include/functions.php

@@ -193 +193 @@
 function delete_categories($ids)
 {
+  global $counts;
+  
   // destruction of all the related elements
   $query = '
@@ -252 +254 @@
 ;';
   mysql_query($query);
-}
-        
+
+  if (isset($counts['del_categories']))
+  {
+    $counts['del_categories']+= count($ids);
+  }
+}
 
 // The function delete_elements deletes the elements identified by the
@@ -262 +268 @@
 function delete_elements($ids)
 {
-  global $count_deleted;
+  global $counts;
   
   // destruction of the comments on the image
@@ -296 +302 @@
   mysql_query($query);
 
-  $count_deleted+= count($ids);
+  if (isset($counts['del_elements']))
+  {
+    $counts['del_elements']+= count($ids);
+  }
 }
 

trunk/admin/update.php

@@ -69 +69 @@
 function insert_local_category($id_uppercat)
 {
-  global $conf, $page, $user, $lang;
+  global $conf, $page, $user, $lang, $counts;
  
   $uppercats = '';
@@ -189 +189 @@
       else
       {
-        $output.= '<span style="color:red;">"'.$fs_subdir.'" : ';
+        $output.= '<span class="update_category_error">"'.$fs_subdir.'" : ';
         $output.= $lang['update_wrong_dirname'].'</span><br />';
       }
@@ -206 +206 @@
 ;';
     mysql_query($query);
+
+    $counts['new_categories']+= count($inserts);
     // updating uppercats field
     $query = '
@@ -266 +268 @@
 function insert_local_element($dir, $category_id)
 {
-  global $lang,$conf,$count_new;
+  global $lang,$conf,$counts;
 
   $output = '';
@@ -399 +401 @@
           $insert['tn_ext'] = "'".$tn_ext."'";
 
-          $count_new++;
-          $output.= $unregistered_element;
-          $output.= ' <span style="font-weight:bold;">';
-          $output.= $lang['update_research_added'].'</span>';
-          $output.= ' ('.$lang['update_research_tn_ext'].' '.$tn_ext.')';
-          $output.= '<br />';
+          $counts['new_elements']++;
           array_push($inserts, $insert);
         }
         else
         {
-          $output.= '<span style="color:orange;">';
+          $output.= '<span class="update_error_element">';
           $output.= $lang['update_missing_tn'].' : '.$unregistered_element;
           $output.= ' (<span style="font-weight:bold;">';
@@ -450 +447 @@
         }
 
-        $count_new++;
-        $output.= $unregistered_element;
-        $output.= ' <span style="font-weight:bold;">';
-        $output.= $lang['update_research_added'].'</span>';
-        $output.= '<br />';
+        $counts['new_elements']++;
         array_push($inserts, $insert);
       }
@@ -460 +453 @@
     else
     {
-      $output.= '<span style="color:red;">"'.$unregistered_element.'" : ';
+      $output.= '<span class="update_error_element">"';
+      $output.= $unregistered_element.'" : ';
       $output.= $lang['update_wrong_dirname'].'</span><br />';
     }
@@ -553 +547 @@
   'L_ALL_UPDATE'=>$lang['update_all'],
   'L_RESULT_UPDATE'=>$lang['update_part_research'],
-  'L_NEW_CATEGORY'=>$lang['update_research_conclusion'],
-  'L_DEL_CATEGORY'=>$lang['update_deletion_conclusion'],
+  'L_NB_NEW_ELEMENTS'=>$lang['update_nb_new_elements'],
+  'L_NB_NEW_CATEGORIES'=>$lang['update_nb_new_categories'],
+  'L_NB_DEL_ELEMENTS'=>$lang['update_nb_del_elements'],
+  'L_NB_DEL_CATEGORIES'=>$lang['update_nb_del_categories'],
   'L_UPDATE_SYNC_METADATA_QUESTION'=>$lang['update_sync_metadata_question'],
   
@@ -571 +567 @@
   check_cat_id($_GET['update']);
   $start = get_moment();
-  $count_new = 0;
-  $count_deleted = 0;
+  $counts = array(
+    'new_elements' => 0,
+    'new_categories' => 0,
+    'del_elements' => 0,
+    'del_categories' => 0
+    );
   
   if (isset($page['cat']))
@@ -583 +583 @@
   }
   echo get_elapsed_time($start,get_moment()).' for scanning directories<br />';
-  $template->assign_block_vars('update',array(
-    'CATEGORIES'=>$categories,
-        'NEW_CAT'=>$count_new,
-        'DEL_CAT'=>$count_deleted
-   ));
-  if ($count_new > 0)
+  $template->assign_block_vars(
+    'update',
+    array(
+      'CATEGORIES'=>$categories,
+      'NB_NEW_CATEGORIES'=>$counts['new_categories'],
+      'NB_DEL_CATEGORIES'=>$counts['del_categories'],
+      'NB_NEW_ELEMENTS'=>$counts['new_elements'],
+      'NB_DEL_ELEMENTS'=>$counts['del_elements']
+      ));
+  if ($counts['new_elements'] > 0)
   {
     $url = PHPWG_ROOT_PATH.'admin.php?page=update&amp;metadata=1';

trunk/language/en_UK.iso-8859-1/admin.lang.php

@@ -262 +262 @@
 $lang['update_disappeared'] = 'doesn\'t exist';
 $lang['update_part_deletion'] = 'Deletion of images that have no thumbnail or that doesn\'t exist';
-$lang['update_deletion_conclusion'] = 'pictures removed from database';
 $lang['update_part_research'] = 'Search for new images in the directories';
 $lang['update_research_added'] = 'added';
 $lang['update_research_tn_ext'] = 'thumbnail in';
-$lang['update_research_conclusion'] = 'pictures added to the database';
+$lang['update_nb_new_elements'] = 'elements added in the database';
+$lang['update_nb_del_elements'] = 'elements deleted in the database';
+$lang['update_nb_new_categories'] = 'categories added in the database';
+$lang['update_nb_del_categories'] = 'categories deleted in the database';
 $lang['update_default_title'] = 'Choose an option';
 $lang['update_only_cat'] = 'update categories, not pictures';

trunk/template/default/admin/update.tpl

@@ -8 +8 @@
 <!-- BEGIN update -->
 <div class="admin">{L_RESULT_UPDATE}</div>
-<br /><span style="color:#9cb4ce;">{update.NEW_CAT} {L_NEW_CATEGORY}</span>
-<br /><span style="color:#ffe1e1;">{update.DEL_CAT} {L_DEL_CATEGORY}</span>
+<ul style="text-align:left;">
+  <li class="update_summary_new">{update.NB_NEW_CATEGORIES} {L_NB_NEW_CATEGORIES}</li>
+  <li class="update_summary_new">{update.NB_NEW_ELEMENTS} {L_NB_NEW_ELEMENTS}</li>
+  <li class="update_summary_del">{update.NB_DEL_CATEGORIES} {L_NB_DEL_CATEGORIES}</li>
+  <li class="update_summary_del">{update.NB_DEL_ELEMENTS} {L_NB_DEL_ELEMENTS}</li>
+</ul>
 <!-- BEGIN sync_metadata -->
 <br />[ <a href="{update.sync_metadata.U_URL}">{L_UPDATE_SYNC_METADATA_QUESTION}</a> ]

trunk/template/default/default.css

@@ -328 +328 @@
   font-weight:normal;
 }
+
+span.update_error_element, span.update_category_error {
+  color:orange;
+}
+
+.update_summary_new {
+  color:#9cb4ce;
+}
+
+.update_summary_del {
+  color:#ffe1e1;
+}