Projet

Général

Profil

Bug #15944

apparition du lien fargo même sans présence de fargo

Ajouté par Thomas Noël il y a presque 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Immediat
Assigné à:
Version cible:
-
Début:
21 avril 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

cf #15922


Fichiers

Révisions associées

Révision f4a82557 (diff)
Ajouté par Thomas Noël il y a presque 7 ans

don't display portfolio pick link if it's not present (#15944)

Historique

#1

Mis à jour par Serghei Mihai il y a presque 7 ans

Constaté hier également sur la démo avec Brice lors de la présentation au SIGEC.

#2

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

C'est une regression suite à #15720 : avant, on n'envoyait pas le fargo.js quand le portfolio n'existait pas :

-                if (portfolio.has_portfolio()
-                        and get_request().user
-                        and not self.readonly):
+                if get_request().user:
                     get_response().add_javascript(['fargo.js'])
#3

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

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Thomas Noël
#5

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

  • Priorité changé de Normal à Immediat

C'est vraiment un sale bogue, si quelqu'un à un oeil à dispo pour relire, merci merci.

#6

Mis à jour par Jean-Baptiste Jaillet il y a presque 7 ans

ack

#7

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

Comme Jean-Baptiste m'a dit que c'était mieux avec des tests et qu'il a bien raison, voici.

#8

Mis à jour par Jean-Baptiste Jaillet il y a presque 7 ans

En effet ! Ack

#9

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

  • Statut changé de En cours à Résolu (à déployer)
commit f4a82557bad67b47ef27ffa5a903351078463513
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Apr 21 15:52:51 2017 +0200

    don't display portfolio pick link if it's not present (#15944)

#11

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