Fix double quoting of primary key in InspectView.get_edit_url and get_delete_url

self.pk is already the quoted value, so we should either use self.pk directly or quote(self.object.pk).

Use the latter for consistency with other views.
pull/11024/head
Sage Abdullah 2023-10-05 14:48:00 +01:00 zatwierdzone przez Matt Westcott
rodzic aee4f42a7d
commit 60e2a6dfe4
1 zmienionych plików z 2 dodań i 2 usunięć

Wyświetl plik

@ -1078,7 +1078,7 @@ class InspectView(PermissionCheckedMixin, WagtailAdminTemplateMixin, TemplateVie
)
):
return None
return reverse(self.edit_url_name, args=(quote(self.pk),))
return reverse(self.edit_url_name, args=(quote(self.object.pk),))
def get_delete_url(self):
if not self.delete_url_name or (
@ -1088,7 +1088,7 @@ class InspectView(PermissionCheckedMixin, WagtailAdminTemplateMixin, TemplateVie
)
):
return None
return reverse(self.delete_url_name, args=(quote(self.pk),))
return reverse(self.delete_url_name, args=(quote(self.object.pk),))
def get_context_data(self, **kwargs):
context = super().get_context_data(**kwargs)