Slugifying news and events urls

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Slugifying news and events urls

Magnus Hagander-2
PFA two patches that do this.

AFAIK that gives for better SEO as well, but in general it just makes things look nicer :)

We already had support for the <id>-<slug>/ format URLs, but we never generated them. But for things to look nicer, this patch instead uses <slug>-<id>/.

Old URLs still work, and if the title (and thus slug) is changed it will redirect to the new location, to maximize future caching for it.

For news, it adds the slugified url to the admin list, primarily to make it easier for the person who's doing a "web release" to be able to copy/paste the actual complete URL instead of having to rely on trying to figure out what slugify does.

Thoughts? And for those who actually know Web Stuff (TM), is it correct that this should also help with SEO stuff?

--

0002-Slugify-event-URLs.patch (6K) Download Attachment
0001-Slugify-news-URLs.patch (13K) Download Attachment