stringex 1.0.1 → 1.0.2

data/lib/lucky_sneaks/acts_as_url.rb

@@ -73,15 +73,13 @@ module LuckySneaks
         conditions << send(self.class.scope_for_url)
       end
       url_owners = self.class.find(:all, :conditions => conditions)
-      if url_owners.size > 0
-        return unless url_owners.map { |o| o.send(url_attribute) }.include?(base_url)
+      write_attribute url_attribute, base_url
+      if url_owners.any?{|owner| owner.send(url_attribute) == base_url}
         n = 1
-        while url_owners.detect{|u| u.send(url_attribute) == "#{base_url}-#{n}"}
+        while url_owners.any?{|owner| owner.send(url_attribute) == "#{base_url}-#{n}"}
           n = n.succ
         end
         write_attribute url_attribute, "#{base_url}-#{n}"
-      else
-        write_attribute url_attribute, base_url
       end
     end
   end

data/stringex.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{stringex}
-  s.version = "1.0.1"
+  s.version = "1.0.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Russell Norris"]

data/test/acts_as_url_test.rb

@@ -109,6 +109,14 @@ class ActsAsUrlTest < Test::Unit::TestCase
     assert_equal "twonique-1", @other_doc2.url
   end
   
+  def test_should_create_unique_url_when_partial_url_already_exists
+    @doc = Document.create!(:title => "House Farms")
+    @other_doc = Document.create!(:title => "House Farm")
+
+    assert_equal "house-farms", @doc.url
+    assert_equal "house-farm", @other_doc.url
+  end
+  
   def test_should_scope_uniqueness
     @moc = Mocument.create!(:title => "Mocumentary", :other => "I dunno why but I don't care if I'm unique")
     @other_moc = Mocument.create!(:title => "Mocumentary")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: stringex
 version: !ruby/object:Gem::Version 
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors: 
 - Russell Norris