test/cached_slug_test.rb test/models/city.rb @@ -161,7 +161,7 @@ generating URLs. For that, you need a column on your model: Then, you can enable cached slugs: class User < ActiveRecord::Base - has_friendly_id :name, :use_slug => true, :cached_column => 'cached_slug' + has_friendly_id :name, :use_slug => true, :cache_column => 'cached_slug' end Important: if you activate this option on a table with already existing slugs, README.rdoc @@ -55,7 +55,7 @@ module FriendlyId::SluggableInstanceMethods # id. def slug(reload = false) @most_recent_slug = nil if reload - @most_recent_slug ||= slugs.first + @most_recent_slug ||= slugs.first(:order => "id DESC") end # Returns the friendly id, or if none is available, the numeric id. @@ -120,7 +120,8 @@ private slugs.find(:all, :conditions => {:name => slug_text, :scope => scope}).each { |s| s.destroy } slug = slugs.build slug_attributes if cache = friendly_id_options[:cache_column] - update_attribute(cache, slug.to_friendly_id) unless send(cache) == slug.to_friendly_id + new_slug = slug.to_friendly_id + send("#{cache}=", new_slug) unless send(cache) == new_slug end slug end lib/friendly_id/sluggable_instance_methods.rb @@ -40,6 +40,11 @@ ActiveRecord::Schema.define(:version => 1) do t.column "event_date", "datetime" end + create_table "cities", :force => true do |t| + t.column "name", "string" + t.column "cached_slug", "string" + end + create_table "slugs", :force => true do |t| t.column "name", "string" t.column "sluggable_id", "integer" test/schema.rb @@ -23,8 +23,9 @@ require 'models/book' require 'models/novel' require 'models/thing' require 'models/event' +require 'models/city' ActiveRecord::Base.establish_connection :adapter => "sqlite3", :database => ":memory:" silence_stream(STDOUT) do load(File.dirname(__FILE__) + "/schema.rb") -end \ No newline at end of file +end test/test_helper.rb 8ef86f69e47d6ff14d8ce2e34e17e0958098c09b Bruno Michel bmichel@menfin.info http://github.com/kakra/friendly_id/commit/68719022fc0a979710ef569d84f8a7e5d62203b3 68719022fc0a979710ef569d84f8a7e5d62203b3 2009-09-22T16:09:19-07:00 2009-09-22T16:09:19-07:00 Added some unit tests d42fcd8f436726dfddc5beb74c1093cee6d6f511 Bruno Michel bmichel@menfin.info