remove migration code

Author: gardenerik

bullet/bullet_admin/templates/bullet_admin/generic/list.html

@@ -21,30 +21,6 @@
         {% for link in list_links %}
           {% #abtn icon=link.icon label=link.label color=link.color url=link.url|add:back %}
         {% endfor %}
-
-        {# TODO: Remove #}
-        {% if help_url %}
-          {% #abtn icon="mdi:help" label="Help" url=help_url|add:back %}
-        {% endif %}
-        {% if new_folder_url %}
-          {% #abtn icon="mdi:folder-plus" label="New folder" url=new_folder_url|add:back %}
-        {% endif %}
-        {% if assign_numbers_url %}
-          {% #abtn icon="mdi:numeric" label="Assign numbers" url=assign_numbers_url|add:back %}
-        {% endif %}
-        {% if export_url %}
-          {% #abtn color="blue" icon="mdi:export" label="Export" url=export_url|add:back %}
-        {% endif %}
-        {% if create_url %}
-          {% if object_name %}
-            {% #abtn color="green" icon="mdi:plus" label="New "|add:object_name url=create_url|add:back %}
-          {% else %}
-            {% #abtn color="green" icon="mdi:plus" label="New" url=create_url|add:back %}
-          {% endif %}
-        {% endif %}
-        {% if upload_url %}
-          {% #abtn color="green" icon="mdi:upload" label="Upload File" url=upload_url|add:back %}
-        {% endif %}
       {% endslot %}
 
       {{ list_subtitle }}

bullet/bullet_admin/views/content.py

@@ -401,11 +401,11 @@ def get_language_content(self, object):
         lang = get_language_info(object.language)
         return lang["name"]
 
-    def get_edit_url(self, menu: Menu) -> str:
-        return reverse("badmin:menu_edit", args=[menu.pk])
-
-    def get_delete_url(self, menu: Menu) -> str:
-        return reverse("badmin:menu_delete", args=[menu.pk])
+    def get_row_links(self, object) -> list[Link]:
+        return [
+            EditIcon(reverse("badmin:menu_edit", args=[object.pk])),
+            DeleteIcon(reverse("badmin:menu_delete", args=[object.pk])),
+        ]
 
 
 class MenuItemEditView(TranslatorRequiredMixin, UpdateView):

bullet/bullet_admin/views/generic/list.py

@@ -1,4 +1,3 @@
-import warnings
 from functools import reduce
 from operator import attrgetter, or_
 from typing import Any, Callable
@@ -10,7 +9,7 @@
 
 from bullet_admin.mixins import MixinProtocol
 from bullet_admin.utils import get_allowed_countries
-from bullet_admin.views.generic.links import DeleteIcon, EditIcon, Link, ViewIcon
+from bullet_admin.views.generic.links import Link
 
 
 class ModelFiltering(MixinProtocol):
@@ -167,9 +166,6 @@ class GenericList(CountryNavigation, LanguageNavigation, OrderedSearch, MixinPro
     table_labels: dict[str, str] = {}
     table_field_templates: dict[str, str] = {}
 
-    # deprecated
-    object_name = None
-
     def get_context_data(self, *, object_list=None, **kwargs):
         ctx = {}
         qs = self.get_queryset()
@@ -184,23 +180,6 @@ def get_context_data(self, *, object_list=None, **kwargs):
 
         ctx.update(super().get_context_data(object_list=qs, **kwargs))
 
-        ctx["object_name"] = self.get_object_name()
-
-        OLD_NAMES = [
-            "help_url",
-            "create_url",
-            "upload_url",
-            "export_url",
-            "new_folder_url",
-            "assign_numbers_url",
-        ]
-        for name in OLD_NAMES:
-            val = getattr(self, name, None)
-            ctx[name] = val
-            if val:
-                warnings.warn(f"{name} is deprecated in {self.__class__.__name__}.")
-
-        # NEW
         ctx["list_title"] = self.get_list_title()
         ctx["list_subtitle"] = self.get_list_subtitle()
         ctx["list_links"] = self.get_list_links()
@@ -223,38 +202,12 @@ def get_list_links(self) -> list[Link]:
         return self.list_links
 
     def get_list_subtitle(self) -> str | None:
-        # TODO: Remove
-        if hasattr(self, "subtitle"):
-            warnings.warn(f"subtitle is deprecated in {self.__class__.__name__}")
-            return self.subtitle
         return self.list_subtitle
 
     def get_table_fields(self) -> list[str]:
-        # TODO: Remove
-        if hasattr(self, "get_fields"):
-            warnings.warn(f"get_fields is deprecated in {self.__class__.__name__}")
-            return self.get_fields()
-        if hasattr(self, "fields"):
-            warnings.warn(f"fields is deprecated in {self.__class__.__name__}")
-            return self.fields
         return self.table_fields
 
-    def get_object_name(self):
-        return (
-            self.object_name
-            if self.object_name
-            else self.get_list_title().split(" ")[-1].lower()[0:-1]
-        )
-
     def get_table_labels(self) -> list[tuple[str, str]]:
-        # TODO: Remove
-        if hasattr(self, "get_labels"):
-            warnings.warn(f"get_labels is deprecated in {self.__class__.__name__}")
-            return self.get_labels()
-        if hasattr(self, "labels"):
-            warnings.warn(f"labels is deprecated in {self.__class__.__name__}")
-            self.table_labels = self.labels
-
         labels = []
         for field in self.get_table_fields():
             label = field.replace("_", " ").capitalize()
@@ -265,35 +218,10 @@ def get_table_labels(self) -> list[tuple[str, str]]:
 
         return labels
 
-    def _migrate_old_links(self, object):
-        # TODO: Remove
-        links = []
-        MAPPING = {
-            "get_edit_url": EditIcon,
-            "get_view_url": ViewIcon,
-            "get_delete_url": DeleteIcon,
-            "get_download_url": None,
-            "get_generate": None,
-        }
-
-        for func, icon in MAPPING.items():
-            if hasattr(self, func):
-                warnings.warn(f"{func} is deprecated in {self.__class__.__name__}")
-                if not icon:
-                    continue
-                url = getattr(self, func)(object)
-                links.append(icon(url))
-
-        return links
-
     def get_row_links(self, object) -> list[Link]:
-        return self._migrate_old_links(object)
+        return []
 
     def get_row_fields(self, object) -> list[str]:
-        if hasattr(self, "field_templates"):
-            warnings.warn(f"field_templates is deprecated in {self.__class__.__name__}")
-            self.table_field_templates = self.field_templates
-
         fields = []
         for field in self.get_table_fields():
             data = getattr(object, field, None)
@@ -313,10 +241,6 @@ def get_row_fields(self, object) -> list[str]:
         return fields
 
     def get_row_context(self, object):
-        # TODO: Remove
-        if hasattr(self, "create_row"):
-            warnings.warn(f"create_row is deprecated in {self.__class__.__name__}")
-
         return {
             "fields": self.get_row_fields(object),
             "links": self.get_row_links(object),