Projet

Général

Profil

Development #24770

Ressources pour les cellules w.c.s.

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
25 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Depuis des années on voudrait pouvoir paramétrer pour une catégorie une photographie, pour un autre site ce sera un logo, et un deuxième pour le survol, pour un autre ce sera aux formulaires qu'on voudra attacher une photo ou deux ou trois ou un logo etc. Bref le besoin est simplement exprimé mais il n'y avait pas de solution.

Maintenant que le portail sert la navigation, qu'on y a une gestion des ressources utiles pour un thème, on pourrait également y avoir une gestion des ressources pour une catégorie ou un formulaire.

L'idée est ici de permettre aux cellules de déclarer les ressources dont elles peuvent avoir besoin et de permettre aux cellules catégories/formulaires de générer cette info sur base d'un modèle.


Fichiers

Révisions associées

Révision 70c5da0d (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

assets: add dynamic asset slots to category and form cells (#24770)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

Patch tout simple, ça se paramètre genre :

WCS_CATEGORY_ASSET_SLOTS = {'logo': {'prefix': 'Logo'}

Et voilà, il y aura un slot pour accueillir un fichier par catégorie, nommé "Logo — nom de la catégorie".

#2

Mis à jour par Thomas Noël il y a presque 6 ans

Pour clarifier

for cell in CellBase.get_cells(lambda x: bool(x.get_asset_slots)):

on pourrait écrire :
for cell in CellBase.get_cells(cell_filter=lambda x: bool(x.get_asset_slots)):

Sur le label %(prefix)s — %(label)s je me dis que ça serait peut-être plus "lisible" de faire un %(label)s — %(suffix)s dans l'idée que tous les assets sur le label seront ainsi listés ensemble. Mais en fait j'en sais trop rien, j'ai pas de cas d'usage clair de cette feature.

slot_template_data['prefix'] ça va crasher si y'a pas de préfixe, peut-être un fallback sur slot_template_key ? Et en fait non oublions, on impose déjà label pour les SLOTS de base.

Bref, ack avec au moins l'ajout de cell_filter=

#3

Mis à jour par Frédéric Péters il y a presque 6 ans

Sur le label %(prefix)s — %(label)s je me dis que ça serait peut-être plus "lisible" de faire un %(label)s — %(suffix)s dans l'idée que tous les assets sur le label seront ainsi listés ensemble. Mais en fait j'en sais trop rien, j'ai pas de cas d'usage clair de cette feature.

J'ai vraiment oscillé entre les deux positions, sans trop savoir au final. (et au début j'avais mis ça sous forme de template pour ne pas me poser la question mais j'ai trouvé ça nul).

#4

Mis à jour par Frédéric Péters il y a presque 6 ans

Voilà le patch avec le passage de paramètre nommé.

#5

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 70c5da0d239560b4e3c8db5a4caddb14cf9330ca
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Jun 25 20:23:59 2018 +0200

    assets: add dynamic asset slots to category and form cells (#24770)
#6

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF