Projet

Général

Profil

Development #13902

dans la vue d'une source de donnée, appliquer la substitution de données sur l'URL

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 novembre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Sur la page d'une source de données sa configuration est reprise, et notamment l'URL, qui pourrait être [passerelle_url]csvdatasource/type_voies/data, et c'est un lien cliquable. Il faudrait appliquer la substitution de variables à l'URL dans le href.


Fichiers

Révisions associées

Révision a56ae27b (diff)
Ajouté par Thomas Noël il y a environ 4 ans

backoffice: compute data_source URL if it's a template (#13902)

Historique

#1

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

  • Assigné à mis à Thomas Noël
#2

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

#3

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

Je préférerais un test avec une vraie situation, histoire de ne pas devoir repasser dessus quand on ne mettra pas de href pour ce qui n'est pas une URL, par exemple.

#4

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

Facilement,

--- a/tests/test_admin_pages.py
+++ b/tests/test_admin_pages.py
@@ -4995,11 +4995,11 @@ def test_data_sources_view(pub):
     assert 'foo' in resp.text

     # variadic url
-    data_source.data_source = {'type': 'json', 'value': 'Foo{{ 2|add:2 }}bar'}
+    data_source.data_source = {'type': 'json', 'value': '{{ site_url }}/foo/bar'}
     data_source.store()
     with HttpRequestsMocking() as http_requests:
         resp = app.get('/backoffice/settings/data-sources/%s/' % data_source.id)
-    assert 'Foo4bar' in resp.text
+    assert '<a href="http://example.net/foo/bar"' in resp.text

#5

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit a56ae27bcb65aa9c625dafb203b9f4111e187ec2
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Mon Mar 9 16:34:12 2020 +0100

    backoffice: compute data_source URL if it's a template (#13902)

(je me suis permis de pousser sans attendre le cycle suivant, considérant que c'était une sorte de bogue)

#7

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

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

Formats disponibles : Atom PDF