feature 1078 added: ability to merge tags

git-svn-id: http://piwigo.org/svn/trunk@12032 68402e56-0260-453c-a942-63ccdbb3a9ee
Piwigo · Sep 2, 2011 · bd7f8c0 · bd7f8c0
1 parent 88951a7
commit bd7f8c0
Show file tree

Hide file tree

Showing 6 changed files with 197 additions and 16 deletions.
diff --git a/admin/include/functions.php b/admin/include/functions.php
@@ -1394,6 +1394,36 @@ function add_tags($tags, $images)
     );
 }
 
+/**
+ *
+ */
+function delete_tags($tag_ids)
+{
+  if (is_numeric($tag_ids))
+  {
+    $tag_ids = array($tag_ids);
+  }
+
+  if (!is_array($tag_ids))
+  {
+    return false;
+  }
+
+  $query = '
+DELETE
+  FROM '.IMAGE_TAG_TABLE.'
+  WHERE tag_id IN ('.implode(',', $tag_ids).')
+;';
+  pwg_query($query);
+
+  $query = '
+DELETE
+  FROM '.TAGS_TABLE.'
+  WHERE id IN ('.implode(',', $tag_ids).')
+;';
+  pwg_query($query);
+}
+
 function tag_id_from_tag_name($tag_name)
 {
   global $page;

diff --git a/admin/tags.php b/admin/tags.php
@@ -101,31 +101,121 @@
 }
 
 // +-----------------------------------------------------------------------+
-// |                               delete tags                             |
+// |                               merge tags                              |
 // +-----------------------------------------------------------------------+
 
-if (isset($_POST['delete']) and isset($_POST['tags']))
+if (isset($_POST['confirm_merge']))
 {
-  $query = '
-SELECT name
+  if (!isset($_POST['destination_tag']))
+  {
+    array_push(
+      $page['errors'],
+      l10n('No destination tag selected')
+      );
+  }
+  else
+  {
+    $destination_tag_id = $_POST['destination_tag'];
+    $tag_ids = explode(',', $_POST['merge_list']);
+
+    if (is_array($tag_ids) and count($tag_ids) > 1)
+    {
+      $name_of_tag = array();
+      $query = '
+SELECT
+    id,
+    name
   FROM '.TAGS_TABLE.'
-  WHERE id IN ('.implode(',', $_POST['tags']).')
+  WHERE id IN ('.implode(',', $tag_ids).')
 ;';
-  $tag_names = array_from_query($query, 'name');
-
-  $query = '
-DELETE
+      $result = pwg_query($query);
+      while ($row = pwg_db_fetch_assoc($result))
+      {
+        $name_of_tag[ $row['id'] ] = trigger_event('render_tag_name', $row['name']);
+      }
+
+      $tag_ids_to_delete = array_diff(
+        $tag_ids,
+        array($destination_tag_id)
+        );
+
+      $query = '
+SELECT
+    DISTINCT(image_id)
   FROM '.IMAGE_TAG_TABLE.'
-  WHERE tag_id IN ('.implode(',', $_POST['tags']).')
+  WHERE tag_id IN ('.implode(',', $tag_ids_to_delete).')
 ;';
-  pwg_query($query);
-
+      $image_ids = array_from_query($query, 'image_id');
+
+      delete_tags($tag_ids_to_delete);
+
+      $query = '
+SELECT
+    image_id
+  FROM '.IMAGE_TAG_TABLE.'
+  WHERE tag_id = '.$destination_tag_id.'
+;';
+      $destination_tag_image_ids = array_from_query($query, 'image_id');
+
+      $image_ids_to_link = array_diff(
+        $image_ids,
+        $destination_tag_image_ids
+        );
+
+      $inserts = array();
+      foreach ($image_ids_to_link as $image_id)
+      {
+        array_push(
+          $inserts,
+          array(
+            'tag_id' => $destination_tag_id,
+            'image_id' => $image_id
+            )
+          );
+      }
+
+      if (count($inserts) > 0)
+      {
+        mass_inserts(
+          IMAGE_TAG_TABLE,
+          array_keys($inserts[0]),
+          $inserts
+          );
+      }
+
+      $tags_deleted = array();
+      foreach ($tag_ids_to_delete as $tag_id)
+      {
+        $tags_deleted[] = $name_of_tag[$tag_id];
+      }
+
+      array_push(
+        $page['infos'],
+        sprintf(
+          l10n('Tags <em>%s</em> merged into tag <em>%s</em>'),
+          implode(', ', $tags_deleted),
+          $name_of_tag[$destination_tag_id]
+          )
+        );
+    }
+  }
+}
+
+
+// +-----------------------------------------------------------------------+
+// |                               delete tags                             |
+// +-----------------------------------------------------------------------+
+
+if (isset($_POST['delete']) and isset($_POST['tags']))
+{
   $query = '
-DELETE
+SELECT name
   FROM '.TAGS_TABLE.'
   WHERE id IN ('.implode(',', $_POST['tags']).')
 ;';
-  pwg_query($query);
+  $tag_names = array_from_query($query, 'name');
+
+  delete_tags($_POST['tags']);
 
   array_push(
     $page['infos'],
@@ -250,11 +340,17 @@
     )
   );
 
-if (isset($_POST['edit']) and isset($_POST['tags']))
+if ((isset($_POST['edit']) or isset($_POST['merge'])) and isset($_POST['tags']))
 {
+  $list_name = 'EDIT_TAGS_LIST';
+  if (isset($_POST['merge']))
+  {
+    $list_name = 'MERGE_TAGS_LIST';
+  }
+
   $template->assign(
     array(
-      'EDIT_TAGS_LIST' => implode(',', $_POST['tags']),
+      $list_name => implode(',', $_POST['tags']),
       )
     );
 

diff --git a/admin/themes/default/template/tags.tpl b/admin/themes/default/template/tags.tpl
@@ -1,5 +1,28 @@
 {include file='include/tag_selection.inc.tpl'}
 
+{footer_script}{literal}
+jQuery(document).ready(function(){
+  function displayDeletionWarnings() {
+    jQuery(".warningDeletion").show();
+    jQuery("input[name=destination_tag]:checked").parent("label").children(".warningDeletion").hide();
+  }
+
+  displayDeletionWarnings();
+
+  jQuery("#mergeTags label").click(function() {
+    displayDeletionWarnings();
+  });
+
+  jQuery("input[name=merge]").click(function() {
+    if (jQuery("ul.tagSelection input[type=checkbox]:checked").length < 2) {
+      alert("{/literal}{'Select at least two tags for merging'|@translate}{literal}");
+      return false;
+    }
+  });
+});
+{/literal}{/footer_script}
+
+
 <div class="titrePage">
   <h2>{'Manage tags'|@translate}</h2>
 </div>
@@ -30,6 +53,19 @@
   </fieldset>
   {/if}
 
+  {if isset($MERGE_TAGS_LIST)}
+  <input type="hidden" name="merge_list" value="{$MERGE_TAGS_LIST}">
+
+  <fieldset id="mergeTags">
+    <legend>{'Merge tags'|@translate}</legend>
+    {'Select the destination tag'|@translate}<br><br>
+    {foreach from=$tags item=tag name=tagloop}
+    <label><input type="radio" name="destination_tag" value="{$tag.ID}"{if $smarty.foreach.tagloop.index == 0} checked="checked"{/if}> {$tag.NAME}<span class="warningDeletion"> {'(this tag will be deleted)'|@translate}</span></label><br>
+    {/foreach}
+    <br><input type="submit" name="confirm_merge" value="{'Confirm merge'|@translate}">
+  </fieldset>
+  {/if}
+
   <fieldset>
     <legend>{'Add a tag'|@translate}</legend>
 
@@ -49,6 +85,7 @@
     <p>
       <input type="hidden" name="pwg_token" value="{$PWG_TOKEN}">
       <input class="submit" type="submit" name="edit" value="{'Edit selected tags'|@translate}">
+      <input class="submit" type="submit" name="merge" value="{'Merge selected tags'|@translate}">
       <input class="submit" type="submit" name="delete" value="{'Delete selected tags'|@translate}" onclick="return confirm('{'Are you sure?'|@translate}');">
     </p>
   </fieldset>

diff --git a/admin/themes/default/theme.css b/admin/themes/default/theme.css
@@ -1078,3 +1078,5 @@ p#uploadModeInfos {text-align:left;margin-top:1em;font-size:90%;color:#999;}
 #progressbar {border:1px solid #ccc; background-color:#eee;}
 .ui-progressbar-value { background-image: url(images/pbar-ani.gif); height:10px;margin:-1px;border:1px solid #E78F08;}
 
+/* Tag Manager */
+.warningDeletion {display:none;font-style:italic;}
diff --git a/language/en_UK/admin.lang.php b/language/en_UK/admin.lang.php
@@ -849,4 +849,12 @@
 $lang['Photo %s of %s'] = 'Photo %s of %s';
 $lang['show details'] = 'show details';
 $lang['hide details'] = 'hide details';
+$lang['Merge tags'] = 'Merge tags';
+$lang['Select the destination tag'] = 'Select the destination tag';
+$lang['(this tag will be deleted)'] = '(this tag will be deleted)';
+$lang['Confirm merge'] = 'Confirm merge';
+$lang['Merge selected tags'] = 'Merge selected tags';
+$lang['No destination tag selected'] = 'No destination tag selected';
+$lang['Tags <em>%s</em> merged into tag <em>%s</em>'] = 'Tags <em>%s</em> merged into tag <em>%s</em>';
+$lang['Select at least two tags for merging'] = 'Select at least two tags for merging';
 ?>
diff --git a/language/fr_FR/admin.lang.php b/language/fr_FR/admin.lang.php
@@ -858,4 +858,12 @@
 $lang['Photo %s of %s'] = 'Photo %s sur %s';
 $lang['show details'] = 'montrer les détails';
 $lang['hide details'] = 'cacher les détails';
+$lang['Merge tags'] = 'Fusionner les tags';
+$lang['Select the destination tag'] = 'Sélectionnez le tag de destination';
+$lang['(this tag will be deleted)'] = '(ce tag sera supprimé)';
+$lang['Confirm merge'] = 'Confirmez la fusion';
+$lang['Merge selected tags'] = 'Fusionner les tags sélectionnés';
+$lang['No destination tag selected'] = 'Vous n\'avez pas sélectionné de tag de destination';
+$lang['Tags <em>%s</em> merged into tag <em>%s</em>'] = 'Les tags <em>%s</em> ont été fusionnés dans le tag <em>%s</em>';
+$lang['Select at least two tags for merging'] = 'Sélectionnez au moins deux tags pour la fusion';
 ?>