Project

General

Profile

Bug #673

des Textes paramétrables n'apparaissent pas dans l'interface

Added by Victor Claudet about 11 years ago. Updated over 8 years ago.

Status:
Fermé
Priority:
Haut
Assignee:
-
Category:
-
Target version:
Start date:
13 September 2011
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Planning:

Description

Les deux textes suivants :
Texte de bienvenue pour les utilisateurs identifiés
Texte de bienvenue pour les utilisateurs non-identifiés

ils n'apparaissent pas sur la page d'accueil du site. il faudrait pouvoir les afficher je vais en avoir besoin.

History

#1

Updated by Thomas Noël about 11 years ago

sur http://dev.entrouvert.org/projects/au-quotidien/repository/entry/extra/modules/root.ptl#L914 (ligne 914) il y a un «if get_response().iframe_mode:» qui me semble superfétatoire, mais je ne sais pas l'historique de ce if...

Si personne ne sait plus, on peut le retirer àmha.

#2

Updated by Frédéric Péters (de retour le 10/10) about 11 years ago

Le commentaire de commit est "In iframe mode also display welcome text on top of services page", la condition existe pour afficher ces textes de bienvenue sur la page listant les services, car c'est elle qui fait office de page d'accueil en mode iframe (et est assez proche de la page d'accueil d'un w.c.s. normal, d'où la réutilisation de ces textes). Le comportement "non-iframe" propose dans au quotidien une toute autre page d'accueil, dans laquelle il n'y a pas de place prévue pour un petit texte de bienvenue. S'agit-il bien de cela ici ?

Quel serait le texte à mettre, et où ?

#3

Updated by Thomas Noël about 11 years ago

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

Le commentaire de commit est "In iframe mode also display welcome text on top of services page", la condition existe pour afficher ces textes de bienvenue sur la page listant les services, car c'est elle qui fait office de page d'accueil en mode iframe (et est assez proche de la page d'accueil d'un w.c.s. normal, d'où la réutilisation de ces textes). Le comportement "non-iframe" propose dans au quotidien une toute autre page d'accueil, dans laquelle il n'y a pas de place prévue pour un petit texte de bienvenue. S'agit-il bien de cela ici ?

Yep.

Quel serait le texte à mettre, et où ?

welcome-logged (ou welcome-unlogged). En fait, je pense qu'il suffirait de retirer le "if iframe" de la ligne 914...

#4

Updated by Frédéric Péters (de retour le 10/10) about 11 years ago

Non, d'une part parce que ça l'ajouterait sur la page /services, qui n'est pas la page d'accueil, d'autre part parce que ça l'ajouterait deux fois sur la page d'accueil, si jamais des formulaires étaient positionnés dans la seconde colonne.

#5

Updated by Victor Claudet about 11 years ago

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

Non, d'une part parce que ça l'ajouterait sur la page /services, qui n'est pas la page d'accueil, d'autre part parce que ça l'ajouterait deux fois sur la page d'accueil, si jamais des formulaires étaient positionnés dans la seconde colonne.

Si "services" n'est pas la page d'accueil, elle est où la page d'accueil. Pour moi la page "services" est la page d'accueil à partir du moment où il s'agit de la page affichée par défaut.

Sinon est ce qu'on peut récupérer le contenu de ces textes par l'intermédiaire d'une variable de substitution ?

#6

Updated by Frédéric Péters (de retour le 10/10) about 11 years ago

Pour parler de la même chose :

- La page d'accueil : http://demo.au-quotidien.com/
- La page "services" : http://demo.au-quotidien.com/services

#7

Updated by Thomas Noël about 11 years ago

En relisant le code dans auquo/root.ptl, je ne vois qu'un seul moment ou les message welcome-logged/welcome-unlogged sont affichés, c'est dans le bloc "service" (qui est appelé pour dessiner le cadre du milieu de la page d'accueil OU isolé sur sa page tout seul), et cet affichage est bien conditionné par un "if iframe".

Donc à mon avis, en mode normal (non iframe), ces messages ne sont jamais affichés sur un auquo.

Ou bien y'a un appel que j'ai pas vu (j'ai juste fait un grep, j'avoue).

#8

Updated by Frédéric Péters (de retour le 10/10) about 11 years ago

On va se perdre, mais j'écrivais : « Le comportement "non-iframe" propose dans au quotidien une toute autre page d'accueil, dans laquelle il n'y a pas de place prévue pour un petit texte de bienvenue. » confirmant qu'en mode non-iframe il n'y a pas d'endroit où ces textes sont affichés.

Pour éviter un nouveau tour de manège, j'anticipe déjà et reprécise que non, même pour la page d'accueil, même en mode non iframe, ce n'est pas seulement appelé pour dessiner le cadre du milieu de la page, mais également pour un éventuel cadre dans la colonne droite, qui existe pour contenir les formulaires rangés dans cette colonne. (code: self.box_services(position='2nd'))

Mais mais mais, que lis-je, que vois-je, (et pourquoi juste maintenant, et pourquoi cela n'utilise-t-il pas le même texte), dans le code de la page d'accueil,

        t = TextsDirectory.get_html_text('aq-home-page')
        if t:
            '<div id="home-page-intro">'
            t
            '</div>'

La correction pourrait donc être, si aq-home-page n'est pas défini, d'aller voir dans welcome-(un)logged pour trouver un texte à afficher.

Sauf si la réponse à la seconde partie de « Quel serait le texte à mettre, et où ? » est "ah non pas à cet endroit-là"…

#9

Updated by Frédéric Péters (de retour le 10/10) about 11 years ago

Et une tentative de réponse au pourquoi aq-home-page, c'est qu'il est défini avec wysiwyg=True, qui me rappelle un souvenir de "on doit pouvoir afficher sur l'accueil le mot du maire avec sa photo et du texte tricolore, si si, vive la France !".

#10

Updated by Thomas Noël about 11 years ago

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

La correction pourrait donc être, si aq-home-page n'est pas défini, d'aller voir dans welcome-(un)logged pour trouver un texte à afficher.

Moi je serais d'avis de même pas tester si aq-home-page est défini ou pas, et d'afficher le welcome-*logged dans un id=home-page-welcome...?

Le soucis, si je comprends bien, c'est l'existant, n'est-ce pas ?

#11

Updated by Frédéric Péters (de retour le 10/10) about 11 years ago

Ouaip, dans l'existant ça peut apparaître bizarre si welcome-(un)logged vient s'ajouter sous le texte déjà défini dans aq-home-page; d'où ma suggestion de ne l'afficher que si ce n'est aq-home-page n'est pas défini.

#12

Updated by Victor Claudet about 10 years ago

  • Project changed from Noyelles-Godault to Au quotidien
  • Target version set to Au-quotidien 2014.5
#13

Updated by Frédéric Péters (de retour le 10/10) over 8 years ago

  • Status changed from Nouveau to Résolu (à déployer)

Parce que c'est trivial d'implémenter ma suggestion (commentaire 8), que vu l'âge du bug on est arrivé à s'en passer, et qu'à l'avenir la page d'accueil sera une des premières pouvant être overridée par un template django, j'ai poussé :

commit 23f4b82771e73fa63dc67a9e25fe099a08d6c471
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Jan 3 14:20:01 2014 +0100

    display welcome-(un)logged on homepage if there's no aq-home-page (#673)
#14

Updated by Thomas Noël over 8 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#15

Updated by Thomas Noël over 8 years ago

  • Status changed from Solution déployée to Fermé

Also available in: Atom PDF