Projet

Général

Profil

Bug #43604

faux positif sur invalidité d'une cellule json avec une URL parametrée

Ajouté par Thomas Noël il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Version cible:
-
Début:
04 juin 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Soit une cellule avec une URL {{ whatever }}/foo/bar/{{ user_nameid }} qui renvoie un 404 quand le user_nameid lui est inconnu.

Si au premier render de la cellule, on va avoir un 404, elle est déclarée globalement invalide.

Ce qui est faux : ça dépend de la variable.

C'est la suite de #41139.


Fichiers


Demandes liées

Lié à Combo - Development #41139: validation d'url contenant des variablesFermé29 mars 2020

Actions

Révisions associées

Révision 3b4563b8 (diff)
Ajouté par Lauréline Guérin il y a presque 4 ans

cells: don't check validity for json cells with multiple variables in url (#43604)

Historique

#2

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

#3

Mis à jour par Lauréline Guérin il y a presque 4 ans

  • Assigné à mis à Lauréline Guérin
#4

Mis à jour par Lauréline Guérin il y a presque 4 ans

#5

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

Je pense qu'on peut garder self.url.count('{{')

L'usage des [xxx] est obsolète, et ça laisse un code relativement plus lisible.

#7

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

  • Statut changé de Solution proposée à Solution validée

Thank you very much !

#8

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 3b4563b81adc50b140c97dc084230749768b24a9
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Jun 4 14:55:20 2020 +0200

    cells: don't check validity for json cells with multiple variables in url (#43604)

#9

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF