evildmp / Arkestra

Arkestra extends Django CMS to provide an intelligent semantic web publishing system for organisations and institutions.
http://arkestra-project.org
BSD 2-Clause "Simplified" License
139 stars 27 forks source link

Person slugs include title PK, not text #95

Closed evildmp closed 11 years ago

evildmp commented 11 years ago

For example:

5-daniele-procida

instead of

mr-daniele-procida

Likely to have been introduced in #68 #70

darbula commented 11 years ago

This should be fixed on this line.

Documentation states _prepopulatedfields doesn’t accept DateTimeField, ForeignKey, nor ManyToManyField fields so consider removing title from the slug.

Removing title can be justified because slug is not changing and title could be changing these two could eventually become out of sync, oth name could be changed, also...