Changeset 8398

Timestamp:

Dec 30, 2010, 4:54:43 PM (14 years ago)

Author:

plg

Message:

feature 2089: report feature:1845 in the new Batch Manager.

+ once the photos are deleted, redirect the page to reset the current photo set
and avoid being on an empty 2nd page.

Location:

trunk/admin

Files:

• batch_manager_global.php (modified) (2 diffs)
• themes/default/template/batch_manager_global.tpl (modified) (1 diff)

trunk/admin/batch_manager_global.php

@@ -152 +152 @@
       // we remove the dissociated images if we are currently displaying the
       // category to dissociate from.
+      //
+      // TODO we can display the photo of a given album without the $_GET['cat']
       if (is_numeric($_GET['cat']) and $_POST['dissociate'] == $_GET['cat'])
       {
@@ -270 +272 @@
     if (isset($_POST['confirm_deletion']) and 1 == $_POST['confirm_deletion'])
     {
-      // filter selection on photos that have no storage_category_id (ie
-      // that were added via pLoader)
-      $query = '
-SELECT id
-  FROM '.IMAGES_TABLE.'
-  WHERE id IN ('.implode(',', $collection).')
-    AND storage_category_id IS NULL
-;';
-      $deletables = array_from_query($query, 'id');
-
-      if (count($deletables) > 0)
+      $deleted_count = delete_elements($collection, true);
+      if ($deleted_count > 0)
       {
-        $physical_deletion = true;
-        delete_elements($deletables, $physical_deletion);
-
-        array_push(
-          $page['infos'],
+        $_SESSION['page_infos'] = array(
           sprintf(
             l10n_dec(
               '%d photo was deleted',
               '%d photos were deleted',
-              count($deletables)
+              $deleted_count
               ),
-            count($deletables)
+            $deleted_count
             )
           );
 
-        // we have to remove the deleted photos from the current set
-        $page['cat_elements_id'] = array_diff($page['cat_elements_id'], $deletables);
+        $redirect_url = get_root_url().'admin.php?page='.$_GET['page'];
+        redirect($redirect_url);
       }
       else

trunk/admin/themes/default/template/batch_manager_global.tpl

@@ -496 +496 @@
     <!-- delete -->
     <div id="action_delete" class="bulkAction">
-{if $ENABLE_SYNCHRONIZATION}
-    <p style="font-style:italic;width:500px;">{'Note: photo deletion does not apply to photos added by synchronization. For photos added by synchronization, remove them from the filesystem and then perform another synchronization.'|@translate}</p>
-{/if}
     <p><label><input type="checkbox" name="confirm_deletion" value="1"> {'Are you sure?'|@translate}</label></p>
     </div>