Before, if a user edited a layer's name, they would be redirected to a url utilizing the old name and then receive a 404 Page not found error. Now, the url utilizes the new name.
Signed-off-by: Amanda Brindle <[email protected]> --- layerindex/views.py | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/layerindex/views.py b/layerindex/views.py index eaeb5c3..2a25455 100644 --- a/layerindex/views.py +++ b/layerindex/views.py @@ -105,11 +105,6 @@ def edit_layer_view(request, template_name, branch='master', slug=None): layerbranch = get_object_or_404(LayerBranch, layer=layeritem, branch=branchobj) deplistlayers = LayerItem.objects.exclude(id=layeritem.id).order_by('name') returnto = request.GET.get('returnto', 'layer_item') - if returnto: - if returnto == 'layer_review': - return_url = reverse_lazy(returnto, args=(layeritem.name,)) - else: - return_url = reverse_lazy(returnto, args=(branch, layeritem.name)) else: # Submit mode layeritem = LayerItem() @@ -185,6 +180,15 @@ def edit_layer_view(request, template_name, branch='master', slug=None): msg.send() return HttpResponseRedirect(reverse('submit_layer_thanks')) messages.success(request, 'Layer %s saved successfully.' % layeritem.name) + if slug: + returnto = request.GET.get('returnto', 'layer_item') + if returnto: + if returnto == 'layer_review': + return_url = reverse_lazy(returnto, args=(layeritem.name,)) + else: + return_url = reverse_lazy(returnto, args=(branch, layeritem.name)) + + if return_url: return HttpResponseRedirect(return_url) else: -- 2.7.4 -- _______________________________________________ yocto mailing list [email protected] https://lists.yoctoproject.org/listinfo/yocto
