Development #39380
TypeError authentic2.manager.widgets in filter_queryset (remonté par sentry)
Début:
29 janvier 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/410/?query=is%3Aunresolved
TypeError: 'str' object is not callable
File "django/core/handlers/exception.py", line 41, in inner
response = get_response(request)
File "django/core/handlers/base.py", line 249, in _legacy_get_response
response = self._get_response(request)
File "django/core/handlers/base.py", line 187, in _get_response
response = self.process_exception_by_middleware(e, request)
File "django/core/handlers/base.py", line 185, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "django/views/generic/base.py", line 68, in view
return self.dispatch(request, *args, **kwargs)
File "django/views/generic/base.py", line 88, in dispatch
return handler(request, *args, **kwargs)
File "django_select2/views.py", line 40, in get
self.object_list = self.get_queryset()
File "django_select2/views.py", line 60, in get_queryset
return self.widget.filter_queryset(self.term, self.queryset, **kwargs)
File "authentic2/manager/widgets.py", line 69, in filter_queryset
qs = super(SecurityCheckMixin, self).filter_queryset(term, queryset=queryset)
File "authentic2/manager/widgets.py", line 45, in filter_queryset
qs = six.moves.reduce(self.split_term_operator, queries)
Fichiers
Révisions associées
manager: remove unused widgets (#39380)
Historique
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Assigné à mis à Valentin Deniaud
Reproduit en cherchant 'a a' dans le widget d'ajout d'un rôle parent (en gros il suffit d'un espace).
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0001-manager-replace-invalid-widget-split-term-operator-3.patch 0001-manager-replace-invalid-widget-split-term-operator-3.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Un test de non régression et c'est bon.
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0001-manager-replace-invalid-widget-split-term-operator-3.patch 0001-manager-replace-invalid-widget-split-term-operator-3.patch ajouté
- Statut changé de Solution validée à Solution proposée
Avec un test pour RolesForChangeWidget. L'autre widget concerné n'a l'air utilisé nulle part, on pourrait peut-être le virer.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Valentin Deniaud a écrit :
Avec un test pour RolesForChangeWidget. L'autre widget concerné n'a l'air utilisé nulle part, on pourrait peut-être le virer.
Je suis toujours content quand on vire du code, gogogo (ici ou dans un autre ticket comme tu veux).
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0001-manager-replace-invalid-widget-split-term-operator-3.patch 0001-manager-replace-invalid-widget-split-term-operator-3.patch ajouté
- Fichier 0002-manager-remove-unused-widgets-39380.patch 0002-manager-remove-unused-widgets-39380.patch ajouté
Fait dans le patch 2.
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a3306ed576af620c6f0147aa6ba6d0f449c91a53
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Jan 30 17:51:13 2020 +0100
manager: remove unused widgets (#39380)
commit f50d9cdbaa494eff9ac42b5b7c299259802bf1ac
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jan 29 15:31:22 2020 +0100
manager: replace invalid widget split term operator (#39380)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
manager: replace invalid widget split term operator (#39380)