Bug #5

[XPATCH] Store permalinks in the articles and categories tables

Added by scoop - 1227 days ago. Updated 605 days ago.

Status :Closed Start :
Priority :High Due date :
Assigned to :scoop - % Done :

0%

Category :spam
Target version :release2
Resolution :

fixed


Description

[XPATCH] Store permalinks in the articles and categories tables

Associated revisions

Revision b38aaa77fba726196be81d350f2566e585682001
Added by Frédéric de Villamil 270 days ago

fixes 5.0.3 change

git-svn-id: http://svn.typosphere.org/typo/trunk@1660 820eb932-12ee-0310-9ca8-eeb645f39767

History

07/23/2005 10:23 PM - scoop -

  • Status changed from New to Assigned

Needs a re-diff against current trunk. Tobi also requested the complete URL to be stored in the permalinks column.

07/23/2005 11:07 PM - scoop -

  • Status changed from Assigned to New

New patch attached (against r378).

Actually I think going the full-URL-in-permalink-column is a bad idea. For one, it clobbers the model with controller code (trying to generate a proper URL) and also fails in cases like this:

GET /articles/2005/1/1/my-article-title

when the permalink column is:

/articles/2005/01/01/my-article-title

Tobias, any change you can be convinced that the dashed-URL is perfectly sufficient? :)

07/24/2005 01:28 AM - tobi -

You are right. Lets just go with the dashed title then.

07/24/2005 07:48 AM - scoop -

  • Status changed from New to Assigned

07/24/2005 08:07 AM - scoop -

  • Status changed from Assigned to Closed
  • Resolution set to fixed

(In r394) Merged permalink patch which no longer breaks links when the blog author changes an article title. It also allows for custom permalinks by editing it via the content admin. Storing permalinks in the database also means we're no longer doing nasty string comparisons for permalink-accessed articles since we're able to do a direct SQL query on that column. (closes #147)

Also available in: Atom PDF