Как генерировать уникальный slug для ManyToMany отношений?

Question

[ogreshrek]

Добрый день! Делаю учебный проект про музыкантов и их релизы.

Мой вопрос в следующем:
Я имею модель музыканта и релиза. Релиз может иметь несколько музыкантов.
Я хочу, чтобы у каждого релиза был уникальный slug, состоящий из {автор}-{название релиза}, если автор один и {автор}-{автор}-...-{название релиза}, если авторов несколько.

Вот мои модели:

class Musician(models.Model):
    name = models.CharField(verbose_name='Название', 
        blank=False, unique=True
        )

    slug_name = models.SlugField(verbose_name='Слаг', 
        blank=True, unique=True)

    def slugify_name(self):
        """
        Обновляет слаг при сохранении.
        """
        self.slug_name = slugify(self.name)

    def save(self, *args, **kwargs):
        self.slugify_name()
        super().save(*args, **kwargs)

class Release(models.Model):
    author = models.ManyToManyField(
        Musician, 
        verbose_name='Автор', related_name='releases')

    name = models.CharField(
        verbose_name='Название', blank=False,
        )

    slug_name = models.SlugField(verbose_name='Слаг', 
        blank=True, unique=True)

    def get_author_slug(self):
        """
        Возвращает всех авторов релиза в формате '{автор}-{автор}-...'
        """
        if len(self.author.all()) == 1:
            return slugify(''.join([str(a) for a in self.author.all()]))
        else:
            return slugify('-'.join([str(a) for a in self.author.all()]))

    def get_slug(self):
        """
        Генерирует слаг в формате {автор}-{автор}-...-{название релиза}
        """
        author = self.get_author_slug()
        name = self.get_name_slug()
        self.slug_name = slugify(author + '-' + name)

    def slugify_name(self):
        """
        Генерирует слаг в формате {название релиза}-{id}
        """
        self.slug_name = slugify('{}-{}'.format(self.name, self.id))

def create_slug_name(sender, instance, created, **kwargs):
    if created:
        instance.slugify_name()
        instance.save()

post_save.connect(create_slug_name, sender=Release)

На данный момент, что я имею.
У меня есть методы в модели релиза:
get_author_slug(), get_slug(), а также сигнал create_slug_name

Метод get_slug() создает весь уникальный slug, как я и хочу. Проблема в том, что этот метод можно применить только в shell, после создания релиза с уникальным слагом.
Сигнал генерирует слаг в формате {название релиза}-{id}. Уникально - да, но не то, что нужно.

Что можно придумать, чтобы слаг создавался каждый раз при создании/изменении имени релиза.

Comments

[szelga]

поменять create_slug_name?

[ogreshrek]

szelga, каким образом?
При вызове get_slug() я получу в формате {название релиза}, почему то instance.author.all() пустой <QuerySet: []> мне дает в обработчике сигнала.

[ogreshrek]

Забыл добавить, что метод self.get_slug_name() в get_slug() следующий:

def get_slug_name(self):
    return slugify(self.name)

[szelga]

попробуйте вместо self.author.all()Author.objects.filter(releases_id=instance.id) или что-то в этом роде.

[ogreshrek]

szelga, сделал, как вы сказали, изменил методы следующим образом:

class Release(models.Model):
    def get_author_slug(self, instance):
        author = Musician.objects.filter(releases=instance.id)
        if len(author) == 1:
            return slugify(''.join([str(a) for a in author]))
        else:
            return slugify('-'.join([str(a) for a in author]))

    def get_slug(self, instance):
        author = self.get_author_slug(instance)
        name = self.get_name_slug()
        self.slug_name = slugify(author + '-' + name)

Сигнал изменил следующим образом:

def create_slug_name(sender, instance, created, **kwargs):
    if created:
        instance.get_slug(instance)
        instance.save()

post_save.connect(create_slug_name, sender=Release)

Все равно генерируется слаг, как {название релиза}. Не знаю почему, но вот почему-то в instance нету музыкантов, только название релиза.

[szelga]

у меня рабочего окружения сейчас под рукой нет, так что конкретно подсказать не могу.
могу посоветовать только:
1. подправить немного код, чтобы он стал яснее и явнее. например, в .join([str(a) for a in author]) неясно, что будет выдавать str(a) в каждом случае, лучше заменить на a.name. (может даже, дело именно в этом, но я сейчас не могу сказать.) также, instance в аргументах методов модели Release лишнее и только затрудняет понимание.
2. тупо отлаживать print'ами. может, для продакшена в кровавом энтерпрайзе это и плохо, но для учебного проекта вполне себе решение. после каждой строчки, в которой сомневаетесь, ставьте вывод результата её выполнения в консоль. и для тех, в которых не сомневаетесь, тоже.
извините, что больше ничем не могу помочь.

[ogreshrek]

szelga, спасибо, с помощью print() я и понял, что в instance не содержится авторов

Answer 1

[Yura Khlyan]

Сигнал нужно вызывать на m2m-changed

Comments

[ogreshrek]

Спасибо, я пробовал использовать этот сигнал, но у меня не получалось. Перечитал доки, которые вы скинули и сработало, вот код:

def create_slug_name(sender, instance, action, **kwargs):
    if action == 'post_add':
        instance.get_slug()
        instance.save()

m2m_changed.connect(create_slug_name, sender=Release.author.through)