Made carousel tags thread-safe

[Plasclem]

Refs m2bpo/espadon#49

[Plasclem]

ça ne marche pas, les logos ne défilent pas sur la frontpage 😞

[bmispelon]

En local (et sur testing) c'est normal: on a les a viré de la base de données.

Tu peux essayer en en rajoutant depuis le backoffice django.

[bmispelon]

On peut renommer la fonction en carousel simplement (et les suivantes aussi) pour éviter de devoir spécifier un nom?

[bmispelon]

Je centraliserais la logique de max_items dans une seule fonction commune.

D'ailleurs a y réfléchir, les fonctions do_carousel_with_* devraient pouvoir simplement récupérer l'objet puis appeler do_carousel avec.

[Plasclem]

je sais pas trop comment ça marche l'héritage de tags, c'est quel format qu'il faut pour les logos?
j'ai laissé do_carousel, je change le nom en carousel?

[bmispelon]

Il n'y a pas d'héritage a proprement parler: ce sont juste des fonctions qu'on utilise:

@register.inclusion_tag(...)
def carousel(carousel, max_items=None):
    ...

@register.inclusion_tag(...)
def carousel_with_name(name, max_items=None):
    obj = Carousel.objects.get(name=name)
    return carousel(obj, max_items)

Pour les logos, n'importe quelle image devrait suffire.

[Plasclem]

🆙

[bmispelon]

too much indentation (should be 4 spaces)