Projet

Général

Profil

Bug #12887

s'assurer que c'est vraiment bien une page de gadjo avant d'y faire des modifications

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
20 août 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Il y a des pages publiques qui chargent le gadjo.js pour avoir la gestion des popups mais bardaf, se retrouvent maintenant à exécuter le code de mise à jour de l'entête.


Fichiers

Révisions associées

Révision 11e4f358 (diff)
Ajouté par Frédéric Péters il y a plus de 7 ans

make sure javascript code is only executed on real gadjo pages (#12887)

Historique

#1

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

#3

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

Yep ; ou alors on considère que gadjo est légitime en utilisation frontoffice, et on fait un « data-gadjo-backoffice="true" » (plutôt que data-gadjo="true" qui a un peu moins de sens ?)

#4

Mis à jour par Benjamin Dauvergne il y a plus de 7 ans

Il n'y a qu'authentic qui utilise gadjo en front, je ne sais pas s'il y a déjà un de contournement pour éviter d'y voir le menu ou c'est simplement que dans la plupart des cas il n'y a pas de variable SERVICES visible dans ce cas.

#5

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

Yep ; ou alors on considère que gadjo est légitime en utilisation frontoffice, et on fait un « data-gadjo-backoffice="true" » (plutôt que data-gadjo="true" qui a un peu moins de sens ?)

Perso je n'ai pas d'ambition pour gadjo autre que les backoffices.

Il n'y a qu'authentic qui utilise gadjo en front, je ne sais pas s'il y a déjà un de contournement pour éviter d'y voir le menu ou c'est simplement que dans la plupart des cas il n'y a pas de variable SERVICES visible dans ce cas.

Dans Authentic, pour ne pas avoir le menu, j'avais préparé le terrain avec #12755 et #12656.

#6

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

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

Yep ; ou alors on considère que gadjo est légitime en utilisation frontoffice, et on fait un « data-gadjo-backoffice="true" » (plutôt que data-gadjo="true" qui a un peu moins de sens ?)

Perso je n'ai pas d'ambition pour gadjo autre que les backoffices.

Ca roule, ack, donc

#7

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

  • Statut changé de En cours à Résolu (à déployer)
commit 11e4f3588e4c97ed045028640ef7d170d06e0304
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Aug 20 11:33:32 2016 +0200

    make sure javascript code is only executed on real gadjo pages (#12887)

#8

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF