Bug #15110
trace sur la recette lors d'une edition d'une annonce
Début:
23 février 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Internal Server Error: /manage/announce/edit/8
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/usr/lib/python2.7/dist-packages/django/contrib/auth/decorators.py", line 22, in _wrapped_view
return view_func(request, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/django/views/generic/base.py", line 71, in view
return self.dispatch(request, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/django/views/generic/base.py", line 89, in dispatch
return handler(request, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/django/views/generic/edit.py", line 272, in post
return super(BaseUpdateView, self).post(request, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/django/views/generic/edit.py", line 215, in post
return self.form_valid(form)
File "/usr/lib/python2.7/dist-packages/django/views/generic/edit.py", line 194, in form_valid
return super(ModelFormMixin, self).form_valid(form)
File "/usr/lib/python2.7/dist-packages/django/views/generic/edit.py", line 108, in form_valid
return HttpResponseRedirect(self.get_success_url())
File "/usr/lib/python2.7/dist-packages/corbo/views.py", line 90, in get_success_url
return reverse('view_category', kwargs={'pk': self.object.category.pk})
File "/usr/lib/python2.7/dist-packages/django/core/urlresolvers.py", line 578, in reverse
return force_text(iri_to_uri(resolver._reverse_with_prefix(view, prefix, *args, **kwargs)))
File "/usr/lib/python2.7/dist-packages/django/core/urlresolvers.py", line 495, in _reverse_with_prefix
(lookup_view_s, args, kwargs, len(patterns), patterns))
NoReverseMatch: Reverse for 'view_category' with arguments '()' and keyword arguments '{'pk': 5}' not found. 1 pattern(s) tried: ['manage/category/(?P<slug>[\\w-]+)/$']
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a environ 7 ans
- Fichier 0001-manager-fix-redirect-url-after-announce-edit-15110.patch 0001-manager-fix-redirect-url-after-announce-edit-15110.patch ajouté
- Statut changé de Nouveau à En cours
- Assigné à mis à Serghei Mihai
- Priorité changé de Normal à Haut
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 7 ans
Dans les tests, je trouve plutôt pas mal important de faire du début à la fin le déroulé de l'usager, en suivant des liens plutôt qu'à coup de reverse().
Par exemple ici, entre ces deux lignes :
assert resp.location == 'http://localhost:80/manage/'
announce_page = app.get(reverse('add_announce', kwargs={'slug': 'alerts'}))
on perd l'opportunité de passer sur la page /manage/ avec une catégorie, d'entrer dans celle-ci et donc de tester la vue de celle-ci, puis la présence d'un lien d'ajout qui fonctionne, etc.
Mis à jour par Serghei Mihai il y a environ 7 ans
Ok. Je refais le patch. Et celui pour les tests du backoffice du coup.
Mis à jour par Serghei Mihai il y a environ 7 ans
Mis à jour par Frédéric Péters il y a environ 7 ans
resp = app.get(reverse('view_category', kwargs={'slug': 'alerts'}))
et
announce_edit_page = app.get(reverse('edit_announce', kwargs={'pk': '1'}))
Même note que dans #13343 : pas moyen de faire un resp.click(...) de la page où on est juste avant ?
Mis à jour par Serghei Mihai il y a environ 7 ans
Mis à jour par Serghei Mihai il y a environ 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 7d8a683eb3f53ae995b86103f95a78204953f81a
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Thu Feb 23 11:09:45 2017 +0100
manager: fix redirect url after announce edit (#15110)
manager: fix redirect url after announce edit (#15110)