Slugged not playing nice with CanCan/Devise

[sutto]

Slugged not playing nice with CanCan/Devise

[rklubenspies]

This issue has been fixed by doing the following:

# in controller
load_and_authorize_resource :find_by => :slug

# in model
def self.find_by_slug!(slug)
  find_using_slug(slug)
end

All credit goes to ryanb and the entire thread can be seen at https://github.com/ryanb/cancan/issues#issue/256

[hkockerbeck]

Because I'm not sure where it fits better, I post my suggestion from the CanCan issue linked above here as well:

There's a better solution in my opinion. Rails generates dynamic finders for a models attributes. But the name of the attribute that Slugged adds isn't slug, but cached_slug. So with

load_and_authorize_resource :find_by => :cached_slug

we not only save the trouble of adding a custom find_by_slug! method to each of possible many models. Additionally, if we try to find a slug that isn't there, find_by_cached_slug! will raise ActiveRecord::RecordNotFound as expected. The custom find_by_slug! posted above will just deliver nil, on which the views will probably throw up later.

[Sutto]

@hkockerbeck hence it should use find_using_slug!; The advantage of the latter is it can use the history feature; If you're not using that at all, using find_by_cached_slug is probably a better idea.