Projet

Général

Profil

Bug #74239

Erreur JS sur l'édition d'un champ, car plus de roue crantée sur la condition avec disable-python-expressions = true

Ajouté par Sentry Io il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
06 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/105203/

AssertionError: 
(12 additional frame(s) were not displayed)
...
  File "wcs/fields.py", line 892, in add_to_form
    form.add(self.widget_class, 'f%s' % self.id, title=self.label, hint=hint, **kwargs)
  File "wcs/qommon/form.py", line 299, in add
    QuixoteForm.add(self, widget_class, name, *args, **kwargs)
  File "quixote/form/form.py", line 235, in add
    widget = widget_class(name, *args, **kwargs)
  File "wcs/qommon/form.py", line 3317, in __init__
    self.geojson_markers_url = data_source.get_geojson_url() if data_source else ''
  File "wcs/data_sources.py", line 862, in get_geojson_url
    assert self.type == 'geojson'

Fichiers


Demandes liées

Lié à w.c.s. - Development #73161: choix expression - retrait roue crantée // django <> python quand python est désactivéFermé09 janvier 2023

Actions

Révisions associées

Révision 7ca2f0ac (diff)
Ajouté par Lauréline Guérin il y a environ un an

misc: fix js for condition widget when python expressions are disabled (#74239)

Historique

#1

Mis à jour par Lauréline Guérin il y a environ un an

  • Projet changé de Suivi des traces à w.c.s.
#2

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a environ un an

C'est moi qui ai généré cette trace, en indiquant une URL vers un geojson dans le champ Donnée d'une liste affichée en carte
(au lieu de définir une source de donnée ayant cette URL).
On peut reproduire ici :
https://demarches-parsifal.test.entrouvert.org/backoffice/forms/198/fields/13/

#3

Mis à jour par Lauréline Guérin il y a environ un an

D'après le js posé dans 0002 de #47066, la liste des sources de données devrait être restreinte aux seules sources de données nommée geojson: il ne devrait pas être possible de poser une URL.
https://demarches-parsifal.test.entrouvert.org/backoffice/forms/198/fields/13/ il y a une erreur JS sur la page:

Uncaught TypeError: widget_name is undefined
    <anonymous> https://demarches-parsifal.test.entrouvert.org/static/js/qommon.admin.js?94a93e4e21e45c764a169be73dea22fc:118
    jQuery 2
    <anonymous> https://demarches-parsifal.test.entrouvert.org/static/js/qommon.admin.js?94a93e4e21e45c764a169be73dea22fc:115
    jQuery 2
qommon.admin.js:118:20

Le bout de JS qui restreint la liste des sources de données est après cette ligne.

#4

Mis à jour par Lauréline Guérin il y a environ un an

  • Sujet changé de Crash quand on met une url vers un geojson dans le champ donnée d'une liste affichée en carte à Erreur JS sur l'édition d'un champ, car plus de roue crantée sur la condition avec disable-python-expressions = true

Depuis #73161

#5

Mis à jour par Lauréline Guérin il y a environ un an

  • Lié à Development #73161: choix expression - retrait roue crantée // django <> python quand python est désactivé ajouté
#6

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Lauréline Guérin

Lauréline Guérin (lguerin) a ouvert une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a environ un an

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

Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Robot Gitea il y a environ un an

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

Lauréline Guérin (lguerin) a mergé une pull request sur Gitea concernant cette demande :

#9

Mis à jour par Transition automatique il y a environ un an

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

Mis à jour par Transition automatique il y a 12 mois

Automatic expiration

Formats disponibles : Atom PDF