Changeset 12629 for branches/2.3/admin

Timestamp:

Nov 15, 2011, 12:48:37 PM (12 years ago)

Author:

plg

Message:

bug 2506 fixed: Batch Manager does not take permissions into account when
filtering on tags

feature 2507 added: Batch Manager can filter on "all tags" or "any tag"

Location:

branches/2.3/admin

Files:

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

branches/2.3/admin/batch_manager.php

@@ -72 +72 @@
   {
     $_SESSION['bulk_manager_filter']['tags'] = get_tag_ids($_POST['filter_tags'], false);
+
+    if (isset($_POST['tag_mode']) and in_array($_POST['tag_mode'], array('AND', 'OR')))
+    {
+      $_SESSION['bulk_manager_filter']['tag_mode'] = $_POST['tag_mode'];
+    }
   }
 
@@ -298 +303 @@
 if (!empty($_SESSION['bulk_manager_filter']['tags']))
 {
-        $query = '
-SELECT image_id
-        FROM '.IMAGE_TAG_TABLE.
-        'WHERE tag_id IN('.implode(',',$_SESSION['bulk_manager_filter']['tags']).')
-        GROUP BY image_id
-        HAVING COUNT(tag_id)='.count($_SESSION['bulk_manager_filter']['tags']);
-        array_push(
+  array_push(
     $filter_sets,
-                get_image_ids_for_tags($_SESSION['bulk_manager_filter']['tags'])
-                );
+    get_image_ids_for_tags(
+      $_SESSION['bulk_manager_filter']['tags'],
+      $_SESSION['bulk_manager_filter']['tag_mode'],
+      null,
+      null,
+      false // we don't apply permissions in administration screens
+      )
+    );
 }
 

branches/2.3/admin/themes/default/template/batch_manager_global.tpl

@@ -481 +481 @@
         <label><input type="checkbox" name="filter_category_recursive" {if isset($filter.category_recursive)}checked="checked"{/if}> {'include child albums'|@translate}</label>
       </li>
-                        <li id="filter_tags" {if !isset($filter.tags)}style="display:none"{/if}>
-                                <a href="#" class="removeFilter" title="remove this filter"><span>[x]</span></a>
-                                <input type="checkbox" name="filter_tags_use" class="useFilterCheckbox" {if isset($filter.tags)}checked="checked"{/if}>
-                                {'Tags'|@translate}
-                                <select id="tagsFilter" name="filter_tags">
-                                        {foreach from=$filter_tags item=tag}
-                                        <option value="{$tag.id}">{$tag.name}</option>
-                                        {/foreach}
-                                </select>
-                        </li>
+      <li id="filter_tags" {if !isset($filter.tags)}style="display:none"{/if}>
+        <a href="#" class="removeFilter" title="remove this filter"><span>[x]</span></a>
+        <input type="checkbox" name="filter_tags_use" class="useFilterCheckbox" {if isset($filter.tags)}checked="checked"{/if}>
+        {'Tags'|@translate}
+        <select id="tagsFilter" name="filter_tags">
+          {foreach from=$filter_tags item=tag}
+          <option value="{$tag.id}">{$tag.name}</option>
+          {/foreach}
+        </select>
+        <label><span><input type="radio" name="tag_mode" value="AND" {if !isset($filter.tag_mode) or $filter.tag_mode eq 'AND'}checked="checked"{/if}> {'All tags'|@translate}</span></label>
+        <label><span><input type="radio" name="tag_mode" value="OR" {if isset($filter.tag_mode) and $filter.tag_mode eq 'OR'}checked="checked"{/if}> {'Any tag'|@translate}</span></label>
+      </li>
       <li id="filter_level" {if !isset($filter.level)}style="display:none"{/if}>
         <a href="#" class="removeFilter" title="remove this filter"><span>[x]</span></a>