governor_tags 0.1.0 → 0.1.1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ./
   specs:
-    governor_tags (0.1.0)
+    governor_tags (0.1.1)
       governor (>= 0.5.2)
       rails (~> 3.0.5)
 

data/VERSION

@@ -1 +1 @@
-0.1.0
+0.1.1

data/app/views/governor/articles/_tags.html.erb

@@ -1,6 +1,8 @@
-<div class="tags">
-Tags:
-<% article.tags.each do |tag| %>
-  <span class="tag"><%= link_to tag.name, tag %></span>
-<% end %>
-</div>
+<% if article.tags.present? %>
+  <div class="tags">
+  Tags:
+  <% article.tags.each do |tag| %>
+    <span class="tag"><%= link_to tag.name, tag %></span>
+  <% end %>
+  </div>
+<% end %>

data/governor_tags.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{governor_tags}
-  s.version = "0.1.0"
+  s.version = "0.1.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Liam Morley"]

data/lib/governor_tags/instance_methods.rb

@@ -4,7 +4,7 @@ module GovernorTags
       self.tags.map{|t| t.name }.join(GovernorTags.delimiter)
     end
     def tag_list=(tag_names)
-      self.tags = tag_names.split(GovernorTags.delimiter).map{|t| Tag.find_or_create_by_name(t.strip) }
+      self.tags = tag_names.split(GovernorTags.delimiter).reject{|t| t.blank? }.map{|t| Tag.find_or_create_by_name(t.strip) }
     end
   end
 end

data/spec/models/article_spec.rb

@@ -10,6 +10,14 @@ describe Article do
     article.tags.map(&:name).should == ['i fell in love again', 'all things go']
   end
   
+  it "trims blank tags" do
+    expect {
+      article.tag_list = 'drove to chicago, , all things know, all things know, '
+      article.save
+    }.to change{Tag.count}.from(0).to(2)
+    article.tags.map(&:name).should == ['drove to chicago', 'all things know']
+  end
+  
   it "displays the list of tags" do
     article.tags = [Tag.create(:name => 'kicking ass'), Tag.create(:name => 'chewing bubblegum')]
     article.save

data/spec/rails_app/Gemfile.lock

@@ -8,7 +8,7 @@ GIT
 PATH
   remote: ../..
   specs:
-    governor_tags (0.1.0)
+    governor_tags (0.1.1)
       governor (>= 0.5.2)
       rails (~> 3.0.5)
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: governor_tags
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: false
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Liam Morley