slugable 0.0.3 → 0.0.4

data/changelog.md

@@ -1,3 +1,7 @@
+# 0.0.4 (May 30, 2013)
+## fixed
+* `to_slug` method not skip nil in path
+
 # 0.0.3 (March 04, 2013)
 ## added
 * config for allowing or disabling caching

data/lib/slugable/has_slug.rb

@@ -101,9 +101,9 @@ module Slugable
       # def to_slug
       #  if respond_to?(:path_ids)
       #    slugs = if true
-      #      path_ids.map{|id| self.class.cached_slug(id)}.select{|i| i.size > 0 }
+      #      path_ids.map{|id| self.class.cached_slug(id)}.compact.select{|i| i.size > 0 }
       #    else
-      #      path.map{|record| record.send(:"slug")}.select{|i| i.size > 0 }
+      #      path.map{|record| record.send(:"slug")}.compact.select{|i| i.size > 0 }
       #    end
       #    slugs.empty? ? "" : slugs
       #  else
@@ -114,9 +114,9 @@ module Slugable
         def to_#{to}
           if respond_to?(:path_ids)
             slugs = if #{cache_tree}
-              path_ids.map{|id| self.class.cached_#{to}(id)}.select{|i| i.size > 0 }
+              path_ids.map{|id| self.class.cached_#{to}(id)}.compact.select{|i| i.size > 0 }
             else
-              path.map{|record| record.send(:"#{to}")}.select{|i| i.size > 0 }
+              path.map{|record| record.send(:"#{to}")}.compact.select{|i| i.size > 0 }
             end
             slugs.empty? ? "" : slugs
           else

data/lib/slugable/version.rb

@@ -1,3 +1,3 @@
 module Slugable
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/spec/slugable/has_slug_spec.rb

@@ -144,6 +144,18 @@ describe Slugable::HasSlug do
 
         child.to_slug.should eq ["root", "child"]
       end
+
+      it "should skip nil values from slug path" do
+        root = Category.create!(:name => "root", :slug => "root")
+        child = Category.new(:name => "child", :slug => "child")
+        child.parent = root
+        child.save!
+
+        Category.update_all({:slug => nil}, {:id => root.id})
+        Category.clear_cached_slugs
+
+        child.to_slug.should eq ["child"]
+      end
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: slugable
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-04 00:00:00.000000000 Z
+date: 2013-05-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -159,7 +159,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2198894308197098802
+      hash: -494734928332233680
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -168,7 +168,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2198894308197098802
+      hash: -494734928332233680
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24