Projet

Général

Profil

Development #61210

exposer settings.TEMPLATE_VARS

Ajouté par Benjamin Dauvergne il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
28 janvier 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Pour #61192 j'accède aux variables de template via settings.TEMPLATE_VARS, ce qui marche partout sauf dans w.c.s., ce serait pratique d'avoir ça pour pouvoir utiliser le context_processor directement.

Je remarque que pour manager_homepage_* le comportement de hobo.context_processors.portal_agent_url est reproduit dans wcs.qommon.publisher.Publisher.get_substitution_variables() (d'ailleurs ça n'est pas identique, ici il y a un _title en plus) :

        d['manager_homepage_url'] = d.get('portal_agent_url')
        d['manager_homepage_title'] = d.get('portal_agent_title')

C'est ça ou je devrais mettre un if 'wcs' in settings.INSTALLED_APPS dans mon code hobo pour aller chercher les données via get_publisher().get_substitution_variables() :/


Demandes liées

Lié à Hobo - Development #61192: Introduire des variables pour URLs usuelles login_url, logout_url et registration_urlFermé27 janvier 2022

Actions

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Lié à Development #61192: Introduire des variables pour URLs usuelles login_url, logout_url et registration_url ajouté
#2

Mis à jour par Frédéric Péters il y a environ 2 ans

settings.TEMPLATE_VARS et cie de hobo c'est totalement lié au multitenant qui n'est pas utilisé dans wcs; mais à lire l'autre ticket et les patchs etc. la demande c'est "un settings.TEMPLATE_VARS suffisamment proche".

Le truc trivial à imaginer serait :

+        site_options_vars['manager_homepage_url'] = d.get('portal_agent_url')
+        site_options_vars['manager_homepage_title'] = d.get('portal_agent_title')
+        settings.TEMPLATE_VARS = site_options_vars

et ça marcherait sans doute dans les tests mais ça foirerait dès des threads, ça demanderait alors de taper un équivalent TenantSettingsWrapper, etc.

Je préfère la condition moche posée dans le context_processors de hobo.

Sur l'autre bout,

d'ailleurs ça n'est pas identique, ici il y a un _title en plus) :

Dans hobo on a ça et pour moi c'est identique, je capte pas le commentaire :

            variables['manager_homepage_title'] = variables.get(settings.HOBO_MANAGER_HOMEPAGE_TITLE_VAR)
#3

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Statut changé de Nouveau à Rejeté

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

Je préfère la condition moche posée dans le context_processors de hobo.

Ok.

d'ailleurs ça n'est pas identique, ici il y a un _title en plus) :

Dans hobo on a ça et pour moi c'est identique, je capte pas le commentaire :

[...]

Les deux ne sont pas produits au même, j'en ai déduit qu'il en manquait forcément un. Oublie.

Formats disponibles : Atom PDF