From: Michael J. Rubinsky <mrubinsk@horde.org>
Date: Sat, 31 Jan 2009 14:52:56 +0000 (-0500)
Subject: More work on tagging.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=0b82d4468406803d048d1aa7d2082055c2d88c91;p=horde.git

More work on tagging.

Better dom ids for the tagging nodes, pass the resource id to the
autocompleter so we can (eventually) filter out existing tags on
the item when presenting autocompletion choices.
---

diff --git a/kronolith/js/tagactions.js b/kronolith/js/tagactions.js
index 6c5cb4c90..84c85bccc 100755
--- a/kronolith/js/tagactions.js
+++ b/kronolith/js/tagactions.js
@@ -20,7 +20,7 @@ function removeTag(resource, type, tagid, endpoint)
 {
     var params = new Object();
     params.imple = "TagActions/action=remove/resource=" + resource + "/type=" + type + "/tags=" + tagid;
-    new Ajax.Updater({success:'tags-' + resource},
+    new Ajax.Updater({success:'tags_' + resource},
                      endpoint,
                      {
                          method: 'post',
@@ -30,9 +30,9 @@ function removeTag(resource, type, tagid, endpoint)
     return true;
 }
 
-function toggleNewTags(domid)
+function toggleTags(domid)
 {
-	$('tag-show-' + domid).toggle();
-	$('tag-hide-' + domid).toggle();
-	$('newtags-' + domid).toggle();
+	$('tag-show_' + domid).toggle();
+	$('tag-hide_' + domid).toggle();
+	$('tagnode_' + domid).toggle();
 }
\ No newline at end of file
diff --git a/kronolith/lib/Imple/TagActions.php b/kronolith/lib/Imple/TagActions.php
index ea58180d0..619a49a20 100644
--- a/kronolith/lib/Imple/TagActions.php
+++ b/kronolith/lib/Imple/TagActions.php
@@ -104,7 +104,7 @@ class Kronolith_Imple_TagActions extends Kronolith_Imple
         }
 
         foreach ($tags as $tag_id => $tag) {
-            $html .= '<li>' .  $tag . ($hasEdit ? '<a href="#" onclick="removeTag(\'' . $id . '\', \'' . $type . '\',' . $tag_id . ', \'' . Horde::url('imple.php', true) . '\'); Event.stop(event)" id="remove' . md5($id . $tag_id) . '">' . Horde::img('delete-small.png', _("Remove Tag"), '', $GLOBALS['registry']->getImageDir('horde')) . '</a>' : '') . '</li>';
+            $html .= '<li class="panel-tags">' .  $tag . ($hasEdit ? '<a href="#" onclick="removeTag(\'' . $id . '\', \'' . $type . '\',' . $tag_id . ', \'' . Horde::url('imple.php', true) . '\'); Event.stop(event)" id="remove' . md5($id . $tag_id) . '">' . Horde::img('delete-small.png', _("Remove Tag"), '', $GLOBALS['registry']->getImageDir('horde')) . '</a>' : '') . '</li>';
         }
 
         return $html;
diff --git a/kronolith/lib/Imple/TagAutoCompleter.php b/kronolith/lib/Imple/TagAutoCompleter.php
index 24fa07d0d..09e572ce7 100644
--- a/kronolith/lib/Imple/TagAutoCompleter.php
+++ b/kronolith/lib/Imple/TagAutoCompleter.php
@@ -21,9 +21,11 @@ class Kronolith_Imple_TagAutoCompleter extends Kronolith_Imple
      */
     public function __construct($params)
     {
-        if (empty($params['triggerId'])) {
-            $params['triggerId'] = $this->_randomid();
+
+        if (!count($params)) {
+            return;
         }
+        $params['triggerId'] = $params['triggerId'] . '_' . $params['id'];
         if (empty($params['resultsId'])) {
             $params['resultsId'] = $params['triggerId'] . '_results';
         }
@@ -84,6 +86,9 @@ class Kronolith_Imple_TagAutoCompleter extends Kronolith_Imple
     {
         $tagger = new Kronolith_Tagger();
         $tags = $tagger->listTags($search);
+
+
+
         return array_values($tags);
     }
 
diff --git a/kronolith/templates/panel.inc b/kronolith/templates/panel.inc
index f5ff4c579..c410548e7 100644
--- a/kronolith/templates/panel.inc
+++ b/kronolith/templates/panel.inc
@@ -46,10 +46,10 @@ $tagger = new Kronolith_Tagger();
   <li><label><input type="checkbox" class="checkbox" name="display_cal[]" value="<?php echo htmlspecialchars($id) ?>"<?php echo (in_array($id, $display_calendars) ? ' checked="checked"' : '') . ' /> ' . htmlspecialchars($cal->get('name')) ?></label> <?php echo $info_img ?>
    <div class="panel-tags">
    <?php $tags = $tagger->getTags($id, 'calendar'); ?>
-    <span style="cursor:pointer;" id="tag-show-<?php echo $id?>" onclick="toggleNewTags('<?php echo $id?>');" title="<?php echo _("Add new tags")?>">+</span><span id="tag-hide-<?php echo $id?>" style="display:none;cursor:pointer;" onclick="toggleNewTags('<?php echo $id?>');">-</span>
-   <h5 style="display: inline;"><?php echo _("Tags")?></h5>
-   <div id="newtags-<?php echo $id?>" style="display:none;">
-        <ul id="tags-<?php echo $id?>">
+    <span style="cursor:pointer;" id="tag-show_<?php echo $id?>" onclick="toggleTags('<?php echo $id?>');" title="<?php echo _("Add new tags")?>">+</span><span id="tag-hide_<?php echo $id?>" style="display:none;cursor:pointer;" onclick="toggleTags('<?php echo $id?>');">-</span>
+    <h5 style="display: inline;"><?php echo _("Tags")?></h5>
+    <div id="tagnode_<?php echo $id?>" style="display:none;">
+      <ul id="tags_<?php echo $id?>">
         <?php foreach ($tags as $tag_id => $tag): ?>
          <li class="panel-tags"><?php echo $tag . ($cal->hasPermission(Auth::getAuth(), PERMS_EDIT) ? '<a href="#" id="remove' . md5($id . $tag_id) . '">' . Horde::img('delete-small.png', _("Remove Tag"), '', $registry->getImageDir('horde')) . '</a>' : '') . '</li>'?>
         <?php
@@ -62,11 +62,11 @@ $tagger = new Kronolith_Tagger();
        <?php endforeach;?>
       </ul>
       <div class="clear">&nbsp;</div>
-      <input type="text" name="newtags-input-<?php echo $id?>" id="newtags-input-<?php echo $id?>" size="10" /><input type="button" class="button" value="add" id="test" />
-      <div id="newtags-input-<?php echo $id?>_results" class="autocomplete"></div>
-      <span id="newtags-input-<?php echo $id?>_loading_img" style="display:none;"><?php echo Horde::img('loading.gif', _("Loading...")) ?></span>
+      <input type="text" name="newtags-input_<?php echo $id?>" id="newtags-input_<?php echo $id?>" size="10" /><input type="button" class="button" value="add" id="test" />
+      <div id="newtags-input_<?php echo $id?>_results" class="autocomplete"></div>
+      <span id="newtags-input_<?php echo $id?>_loading_img" style="display:none;"><?php echo Horde::img('loading.gif', _("Loading...")) ?></span>
      <?php
-      Kronolith_Imple::factory('TagAutoCompleter', array('triggerId' => 'newtags-input-' . $id));
+      Kronolith_Imple::factory('TagAutoCompleter', array('triggerId' => 'newtags-input', 'id' => $id));
       ?>
   </div>
   </div>