Summary | Inconsistent handling of commas in tags |
Queue | Horde Base |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | mrubinsk (at) horde (dot) org |
Requester | jan (at) horde (dot) org |
Created | 10/08/2012 (4712 days ago) |
Due | |
Updated | 10/15/2012 (4705 days ago) |
Assigned | |
Resolved | 10/12/2012 (4708 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
commit 5ce8c40f1e1634fa4807659c7191ab55798b86c3
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Fri Oct 12 09:17:14 2012 -0400
Use a simple split on commas for tags.
Bug: 11470framework/Core/lib/Horde/Core/Tagger.php | 9 ++++++---
1 files changed, 6 insertions(+), 3 deletions(-)
http://git.horde.org/horde-git/-/commit/5ce8c40f1e1634fa4807659c7191ab55798b86c3
commit 5ce8c40f1e1634fa4807659c7191ab55798b86c3
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Fri Oct 12 09:17:14 2012 -0400
Use a simple split on commas for tags.
Bug: 11470framework/Core/lib/Horde/Core/Tagger.php | 9 ++++++---
1 files changed, 6 insertions(+), 3 deletions(-)
http://git.horde.org/horde-git/-/commit/5ce8c40f1e1634fa4807659c7191ab55798b86c3
too. Chuck original included support for this in the original content
tagger code (in Content_Tagger::split()). Not sure I want to remove
that code there, but rather override that method in Horde_Core_Tagger,
so other users of the code can make the decision for themselves...
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Assigned to Michael Rubinsky
Queue ⇒ Horde Base
Summary ⇒ Inconsistent handling of commas in tags
Type ⇒ Bug
State ⇒ Assigned
applications too:
In traditional view, I can create tags including commas by surrounding
the tags with double quotes. When displaying or editing the tags, the
quotes are missing though.
On the other hand, the tags are displayed correctly in dynamic view
(without quotes, but complete tag including commas inside the tag
label), but they can't be added, because a new tag is created as soon
as you enter comma, even if you have an open double quote.
Not sure if we want to allow commas inside tags at all, but we should
consistently allow (and support), or deny them.