0
@@ -1,5 +1,7 @@
0
 * SVN *
0
 
0
+* Fix duplicate tag bug.
0
+
0
 * add Mephisto.root javascript variable in case a mephisto site sets the relative_url_root
0
 
0
 * Allow Modules to use #include_into to dynamically mixin to loaded classes (plugin stuff)

0
@@ -1,4 +1,5 @@
0
 class Tag < ActiveRecord::Base
0
+  @@tag_parse_regex = /((?: |)['"]{0,1})['"]?\s*(.*?)\s*(?:[,'"]|$)(?:\1(?: |$))/
0
   has_many :taggings
0
 
0
   class << self
0
@@ -18,8 +19,8 @@ class Tag < ActiveRecord::Base
0
     #   # => ['a', 'b', 'c']
0
     def parse(list)
0
       return list if list.is_a?(Array)
0
-      returning list.scan(/((?: |)['"]{0,1})['"]?(.*?)(?:[,'"]|$)(?:\1(?: |$))/) do |tags|
0
-        tags.collect! &:last
0
+      returning list.scan(@@tag_parse_regex) do |tags|
0
+        tags.collect! { |t| t.last.strip!; t.last }
0
         tags.uniq!
0
         tags.delete_if &:blank?
0
       end

0
@@ -27,6 +27,12 @@ class TagTest < Test::Unit::TestCase
0
   def test_should_return_tag_array
0
     assert_equal %w(a b c), Tag.parse(%w(a b c))
0
   end
0
+  
0
+  def test_should_return_unique_tags
0
+    ["a, b, b", %('a' 'b' 'b'), %("a" "b" " b ")].each do |input|
0
+      assert_equal %w(a b), Tag.parse(input), "Failed for: #{input.inspect}"
0
+    end
0
+  end
0
 
0
   def test_should_find_or_create_tags
0
     assert_difference Tag, :count do