Как сделать ЧПУ на django?

Question

[MarsFM]

Можете подсказать почему не работает, плз.
Первый метод нужен для того, чтобы если заголовки одинаковые, то добавляется id.
Второй переводит на английский title.

Если делать всё по отдельности то заработает.

def create_slug(instance, new_slug=None):
    slug = instance.title
    if new_slug is not None:
        slug = new_slug
    qs = Post.objects.filter(slug=slug).order_by("-id")
    exists = qs.exists()
    if exists:
        new_slug = "%s-%s" %(slug, qs.first().id)
        return create_slug(instance, new_slug=new_slug)
    return slug


def set_post_slug(sender, instance, *args, **kwargs):
    from pytils import translit
    instance.slug = translit.slugify(create_slug(instance))

pre_save.connect(set_post_slug, sender=Post)

Answer 1

[Абдулла Мурсалов]

def handle_post_slug(sender, instance, *args, **kwargs):
	from pyutils import translit
	slug = translit(instance.title)
	if Post.objects.filter(slug=slug).exclude(id=instance.id).exists():
		slug = "-".join([slug, str(instance.id)])
	instance.slug = slug

pre_save.connect(handle_post_slug, sender=Post)

Навскидку. Расскажите подробнее, что "не работает"?

Comments

[MarsFM]

Если сделать instance.slug = translit.slugify(instanse.title.strip()), то всё переведётся, если как в моём примере оставить то выдаст ошибку UNIQUE constraint failed: posts_post.slug.

У меня получается либо я добавляю id, либо перевод а вместе не получается совместить

[MarsFM]

А в вашем примере ошибка в коде pyutils исправить на pytils
Но всё равно выдаёт после исправления другую ошибку 'module' object is not callable
'module' object is not callable
Ругается на эту строку translit(instance.title), видимо нельзя translit это объект

[Абдулла Мурсалов]

кстати вместо translit нужно использовать https://docs.djangoproject.com/ja/1.9/ref/utils/#d...

[MarsFM]

Не совсем понял, если просто использовать slugify, то нужно где транслет сделать.
Плюс почему то ругается на exists()
QuerySet' object has no attribute 'extists'

def handle_post_slug(sender, instance, *args, **kwargs):
    from pytils import translit
    slug = translit.slugify(instance.title)
    if Post.objects.filter(slug=slug).exclude(id=instance.id).extists():
        slug = "-".join([slug, str(instance.id)])
    instance.slug = slug

pre_save.connect(handle_post_slug, sender=Post)

[Абдулла Мурсалов]

MarsFM: опечатка, exists() вместо extists()

[MarsFM]

Спасибо)

такой код сработал

def handle_post_slug(sender, instance, *args, **kwargs):
    from pytils import translit
    slug = translit.slugify(instance.title)
    if Post.objects.filter(slug=slug).exclude(id=instance.id):
        slug = "-".join([slug, str(instance.id)])
    instance.slug = slug

pre_save.connect(handle_post_slug, sender=Post)

[MarsFM]

exists() тоже добавил, спасибо, теперь работает)

[Абдулла Мурсалов]

MarsFM: вообще то он не должен работать как ожидается, потому что перепутал кое-что. slugify просто заменяет пробелы, и удаляет все что не цифра, дефис и англ буква

[Абдулла Мурсалов]

MarsFM:

from django.utils.text import slugify
>>> str = 'Здравствуй мир!'
>>> slugify(str)
''

Вот такое случится