Projet

Général

Profil

Development #28099

donner tout le contexte à l'évaluation de l'url d'une cellule de type lien

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

L'URL est déterminée dans get_cell_extra_context() avec uniquement un contexte minimal; pour permettre des URL davantage calculées (genre en fonction de la page en cours, de l'utilisateur sélectionnée, etc.) il faudrait fournir tout le contexte.


Fichiers


Demandes liées

Lié à Publik - Development #19756: Personnalisation accrue du portail agent pour en faire aussi la page d'entrée des agents d'accueilFermé29 octobre 201730 juin 2018

Actions

Révisions associées

Révision 79978044 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

misc: give linkcell full context to create URL (#28099)

Historique

#1

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

Le patch est un peu verbeux parce que je renomme le contexte additionnel qu'on crée en extra_context; à part ça, la seule modification c'est qu'on passe le contexte d'origine à get_template_url() :

-            context['url'] = utils.get_templated_url(self.url)
+            extra_context['url'] = utils.get_templated_url(self.url, context=context)
#2

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

  • Lié à Development #19756: Personnalisation accrue du portail agent pour en faire aussi la page d'entrée des agents d'accueil ajouté
#3

Mis à jour par Thomas Noël il y a plus de 5 ans

Je ne sais pas si c'est important, mais on n'aura pas le context qui provient du super (en l'occurrence celui de CellBase qui contiendrait juste "cell").

#4

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

Ouaip, mais faisons sans (je pense qu'il y a des choses à mettre carré autour des context / context.update / extra_context et ça dépasse le besoin immédiat de ce ticket).

#5

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

Frédéric Péters a écrit :

(je pense qu'il y a des choses à mettre carré autour des context / context.update / extra_context et ça dépasse le besoin immédiat de ce ticket).

D'accord avec ça.

#6

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 799780447e307f80a74e4cbc2623a78a5f62f449
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Nov 17 11:29:58 2018 +0100

    misc: give linkcell full context to create URL (#28099)
#7

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