Re: [Django] #12651: AutoSlugField, that can recreate unique slugs during saving.

Django

unread,

Mar 23, 2013, 5:32:39 PM3/23/13

to django-...@googlegroups.com

#12651: AutoSlugField, that can recreate unique slugs during saving.
-------------------------------------+-------------------------------------
Reporter: Ciantic | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version:
(models, ORM) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: | Needs documentation: 1
Has patch: 1 | Patch needs improvement: 1
Needs tests: 1 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by aaugustin):

* stage: Design decision needed => Accepted

Comment:

The concept described in this ticket was accepted, the discussion is now
about the implementation.

--
Ticket URL: <https://code.djangoproject.com/ticket/12651#comment:13>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,

Jun 22, 2016, 6:13:17 AM6/22/16

to django-...@googlegroups.com

#12651: AutoSlugField, that can recreate unique slugs during saving.
-------------------------------------+-------------------------------------
Reporter: Ciantic | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version:
(models, ORM) |

Comment (by fmalina):

Just to revive this ticket. Here's an example implementation I find quite
acceptable in user code:

{{{#!python
class Article(models.Model):
title = models.CharField(max_length=128)
slug = models.SlugField(max_length=130)
# ...

def save(self, *args, **kwargs):
if not len(self.slug.strip()):
self.slug = slugify(self.title)

_slug = self.slug
_count = 1

while True:
try:
Article.objects.all().exclude(pk=self.pk).get(slug=_slug)
except MultipleObjectsReturned:
pass
except ObjectDoesNotExist:
break
_slug = "%s-%s" % (self.slug, _count)
_count += 1

self.slug = _slug

super(Article, self).save(*args, **kwargs)
}}}

I've also used a regex based alternative implementation:

{{{#!python
import re

class Article(models.Model):
...
def get_slug(self):
return '%s-%s' % (slugify(self.this), slugify(self.that))

def check_slug(self, slug):
# ensure uniqueness
while(Article.objects.filter(slug__iexact=slug).count()): # if
not unique
suff = re.search("\d+$", slug) # get the current number
suffix if present
suff = suff and suff.group() or 0
next = str(int(suff) +1) # increment it & turn to string for
re.sub
slug = re.sub("(\d+)?$", next, slug) # replace with higher
suffix, retry
return slug

def save(self, *args, **kwargs):
#...
slug = self.get_slug()
if not self.pk or (self.pk and not slug in self.slug):
self.slug = self.check_slug(slug)
super( ...
}}}

Could this be implemented by adding a pre_save method to the SlugField?
https://github.com/django/django/blob/master/django/db/models/fields/__init__.py#L2073

Maybe call it add_auto_increment=True.

Any ideas on how this could actually look in the core, as opposed to just
user code?

--
Ticket URL: <https://code.djangoproject.com/ticket/12651#comment:14>

Django

unread,

Feb 11, 2024, 11:27:57 AM2/11/24

to django-...@googlegroups.com

#12651: AutoSlugField, that can recreate unique slugs during saving.
-------------------------------------+-------------------------------------

Reporter: Jari Pennanen | Owner: nobody