search

refinery / refinerycms-news

News Plugin for Refinery CMS
http://www.refinerycms.com
MIT License
121 stars 120 forks source link

Slug doesn't update upon saving with changed title #146

Closed amdest closed 10 years ago

amdest commented 10 years ago

Rails 4.0.2. RefineryCMS — with "edge" template. RefineryCMS-News — master. All installations are fresh (couple of hours ago).

ugisozols commented 10 years ago

This is due to change in friendly_id 5.0.x.

When https://github.com/refinery/refinerycms/pull/2538 gets merged I'll push a similar fix for this repo.

amdest commented 10 years ago

To be honest — I don't get it why this behavior was changed in friendly_id? This feature is essential... OK, thanks!

There is one more issue related, I think: odd behavior with non-ASCII titles, depending on default_locale setting and the order titles are assigned to translations.

Right now i get, at least, three different results... What do You think — should I wait for the https://github.com/refinery/refinerycms/pull/2538 gets merged, or I'll just post another issue here without waiting?

ugisozols commented 10 years ago

Just open a new issue.