Changeset 29244

Timestamp:

Aug 22, 2014, 10:56:51 AM (10 years ago)

Author:

plg

Message:

feature 2810: for duplicates filter, add "date & time" (checked by default) and "width & height" as options.

Change algorithm for a single SQL query with GROUP_CONCAT instead of 2 queries.

Location:

trunk

Files:

• admin/batch_manager.php (modified) (2 diffs)
• admin/themes/default/template/batch_manager_global.tpl (modified) (2 diffs)
• admin/themes/default/theme.css (modified) (1 diff)
• language/en_UK/admin.lang.php (modified) (1 diff)
• language/fr_FR/admin.lang.php (modified) (1 diff)

trunk/admin/batch_manager.php

@@ -80 +80 @@
   {
     $_SESSION['bulk_manager_filter']['prefilter'] = $_POST['filter_prefilter'];
+
+    if ('duplicates' == $_POST['filter_prefilter'])
+    {
+      if (isset($_POST['filter_duplicates_date']))
+      {
+        $_SESSION['bulk_manager_filter']['duplicates_date'] = true;
+      }
+      
+      if (isset($_POST['filter_duplicates_dimensions']))
+      {
+        $_SESSION['bulk_manager_filter']['duplicates_dimensions'] = true;
+      }
+    }
   }
 
@@ -301 +314 @@
 
   case 'duplicates':
-    // we could use the group_concat MySQL function to retrieve the list of
-    // image_ids but it would not be compatible with PostgreSQL, so let's
-    // perform 2 queries instead. We hope there are not too many duplicates.
-    $query = '
-SELECT file
-  FROM '.IMAGES_TABLE.'
-  GROUP BY file, date_creation
+    $duplicates_on_fields = array('file');
+
+    if (isset($_SESSION['bulk_manager_filter']['duplicates_date']))
+    {
+      $duplicates_on_fields[] = 'date_creation';
+    }
+    
+    if (isset($_SESSION['bulk_manager_filter']['duplicates_dimensions']))
+    {
+      $duplicates_on_fields[] = 'width';
+      $duplicates_on_fields[] = 'height';
+    }
+    
+    $query = '
+SELECT
+    GROUP_CONCAT(id) AS ids
+  FROM '.IMAGES_TABLE.'
+  GROUP BY '.implode(',', $duplicates_on_fields).'
   HAVING COUNT(*) > 1
 ;';
-    $duplicate_files = query2array($query, null, 'file');
-
-    $query = '
-SELECT id
-  FROM '.IMAGES_TABLE.'
-  WHERE file IN (\''.implode("','", array_map('pwg_db_real_escape_string', $duplicate_files)).'\')
-;';
-    $filter_sets[] = query2array($query, null, 'id');
+    $array_of_ids_string = query2array($query, null, 'ids');
+
+    $ids = array();
+    
+    foreach ($array_of_ids_string as $ids_string)
+    {
+      $ids = array_merge($ids, explode(',', $ids_string));
+    }
+    
+    $filter_sets[] = $ids;
 
     break;

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

@@ -393 +393 @@
   jQuery("select[name=filter_prefilter]").change(function() {
     jQuery("#empty_caddie").toggle(jQuery(this).val() == "caddie");
+    jQuery("#duplicates_options").toggle(jQuery(this).val() == "duplicates");
   });
 });
@@ -419 +420 @@
         </select>
         <a id="empty_caddie" href="admin.php?page=batch_manager&amp;action=empty_caddie" style="{if !isset($filter.prefilter) or $filter.prefilter ne 'caddie'}display:none{/if}">{'Empty caddie'|translate}</a>
+
+        <span id="duplicates_options" style="{if !isset($filter.prefilter) or $filter.prefilter ne 'duplicates'}display:none{/if}">
+          {'based on'|translate}
+          <input type="checkbox" checked="checked" disabled="disabled"> {'file name'|translate}
+          <label><input type="checkbox" name="filter_duplicates_date" {if isset($filter.duplicates_date) or (isset($filter.prefilter) and $filter.prefilter ne 'duplicates')}checked="checked"{/if}> {'date & time'|translate}</label>
+          <label><input type="checkbox" name="filter_duplicates_dimensions" {if isset($filter.duplicates_dimensions)}checked="checked"{/if}> {'width & height'|translate}</label>
+        </span>
       </li>
 

trunk/admin/themes/default/theme.css

@@ -982 +982 @@
 #batchManagerGlobal .ui-slider-horizontal {width:650px;margin:5px 0 10px 0;}
 
+#batchManagerGlobal #duplicates_options label {margin-left:10px;}
+
 #order_filters a.addFilter {font-weight:normal;margin-left:20px;}
 #order_filters a.removeFilter {font-weight:normal;}

trunk/language/en_UK/admin.lang.php

@@ -977 +977 @@
 $lang['Empty caddie'] = 'Empty caddie';
 $lang['between %s and %s MB'] = 'between %s and %s MB';
+$lang['based on'] = 'based on';
+$lang['file name'] = 'file name';
+$lang['date & time'] = 'date & time';
+$lang['width & height'] = 'width & height';
 ?>

trunk/language/fr_FR/admin.lang.php

@@ -979 +979 @@
 $lang['The settings for the guest are from the %s user'] = 'Les préféreces des invités sont celles de l\'utilisateur %s';
 $lang['between %s and %s MB'] = 'entre %s et %s Mo';
+$lang['based on'] = 'basé sur';
+$lang['file name'] = 'nom de fichier';
+$lang['date & time'] = 'date et heure';
+$lang['width & height'] = 'largeur et hauteur';