Produits Entr'ouvert » w.c.s. » Au quotidien

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.

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ù ?

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...

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.

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 ?

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

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).

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à"…

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 !".

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 ?

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.