Use reverse_lazy instead of reverse

This commit is contained in:
Brian Tiemann 2025-07-14 09:57:34 -04:00
parent 0c95ac6b1a
commit 2905c124c3

View File

@ -1,4 +1,4 @@
from django.urls import reverse from django.urls import reverse_lazy
from django.utils.text import slugify from django.utils.text import slugify
from django.utils.translation import gettext as _ from django.utils.translation import gettext as _
@ -49,7 +49,7 @@ class PluginMenuItem:
self.auth_required = auth_required self.auth_required = auth_required
self.staff_only = staff_only self.staff_only = staff_only
if link: if link:
self._url = reverse(link) self._url = reverse_lazy(link)
if permissions is not None: if permissions is not None:
if type(permissions) not in (list, tuple): if type(permissions) not in (list, tuple):
raise TypeError(_("Permissions must be passed as a tuple or list.")) raise TypeError(_("Permissions must be passed as a tuple or list."))
@ -82,7 +82,7 @@ class PluginMenuButton:
self.title = title self.title = title
self.icon_class = icon_class self.icon_class = icon_class
if link: if link:
self._url = reverse(link) self._url = reverse_lazy(link)
if permissions is not None: if permissions is not None:
if type(permissions) not in (list, tuple): if type(permissions) not in (list, tuple):
raise TypeError(_("Permissions must be passed as a tuple or list.")) raise TypeError(_("Permissions must be passed as a tuple or list."))