ci: auto fixes from pre-commit hooks

for more information, see https://pre-commit.ci

.tx/config

@@ -7,4 +7,3 @@ source_file   = djangocms_alias/locale/en/LC_MESSAGES/django.po
 type          = PO
 minimum_perc  = 0
 resource_name = django.po
-

LICENSE.txt

@@ -21,4 +21,4 @@ DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
 LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
 ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
-SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

djangocms_alias/__init__.py

@@ -1 +1 @@
-__version__ = '2.0.0'
+__version__ = "2.0.0"

djangocms_alias/admin.py

@@ -34,13 +34,13 @@
 
 
 __all__ = [
-    'AliasAdmin',
-    'CategoryAdmin',
-    'AliasContentAdmin',
+    "AliasAdmin",
+    "CategoryAdmin",
+    "AliasContentAdmin",
 ]
 
 alias_admin_classes = [GrouperModelAdmin]
-alias_admin_list_display = ['content__name', 'category', 'admin_list_actions']
+alias_admin_list_display = ["content__name", "category", "admin_list_actions"]
 djangocms_versioning_enabled = AliasCMSConfig.djangocms_versioning_enabled
 
 if djangocms_versioning_enabled:
@@ -59,7 +59,7 @@
 
 @admin.register(Category)
 class CategoryAdmin(TranslatableAdmin):
-    list_display = ['name']
+    list_display = ["name"]
 
     def save_model(self, request, obj, form, change):
         change = not obj._state.adding
@@ -77,9 +77,12 @@ def save_model(self, request, obj, form, change):
 class AliasAdmin(*alias_admin_classes):
     list_display = alias_admin_list_display
     list_display_links = None
-    list_filter = (SiteFilter, CategoryFilter,)
-    fields = ('content__name', 'category', 'site', 'content__language')
-    readonly_fields = ('static_code', )
+    list_filter = (
+        SiteFilter,
+        CategoryFilter,
+    )
+    fields = ("content__name", "category", "site", "content__language")
+    readonly_fields = ("static_code",)
     form = AliasGrouperAdminForm
     extra_grouping_fields = ("language",)
     EMPTY_CONTENT_VALUE = mark_safe(_("<i>Missing language</i>"))
@@ -91,7 +94,9 @@ def get_actions_list(self) -> list:
         """Add alias usage list actions"""
         return super().get_actions_list() + [self._get_alias_usage_link]
 
-    def can_change_content(self, request: HttpRequest, content_obj: AliasContent) -> bool:
+    def can_change_content(
+        self, request: HttpRequest, content_obj: AliasContent
+    ) -> bool:
         """Returns True if user can change content_obj"""
         if content_obj and is_versioning_enabled():
             version = Version.objects.get_for_content(content_obj)
@@ -104,12 +109,14 @@ def has_delete_permission(self, request: HttpRequest, obj: Alias = None) -> bool
         if obj:
             if not obj.is_in_use:
                 return request.user.has_perm(
-                    get_model_permission_codename(self.model, 'add'),
+                    get_model_permission_codename(self.model, "add"),
                 )
             return request.user.is_superuser
         return False
 
-    def save_model(self, request: HttpRequest, obj: Alias, form: forms.Form, change: bool) -> None:
+    def save_model(
+        self, request: HttpRequest, obj: Alias, form: forms.Form, change: bool
+    ) -> None:
         super().save_model(request, obj, form, change)
 
         # Only emit content changes if Versioning is not installed because
@@ -121,10 +128,15 @@ def save_model(self, request: HttpRequest, obj: Alias, form: forms.Form, change:
             )
 
     def get_deleted_objects(self, objs, request: HttpRequest) -> tuple:
-        deleted_objects, model_count, perms_needed, protected = super().get_deleted_objects(objs, request)
+        (
+            deleted_objects,
+            model_count,
+            perms_needed,
+            protected,
+        ) = super().get_deleted_objects(objs, request)
         # This is bad and I should feel bad.
-        if 'placeholder' in perms_needed:
-            perms_needed.remove('placeholder')
+        if "placeholder" in perms_needed:
+            perms_needed.remove("placeholder")
         return deleted_objects, model_count, perms_needed, protected
 
     def delete_model(self, request: HttpRequest, obj: Alias):
@@ -138,14 +150,20 @@ def delete_model(self, request: HttpRequest, obj: Alias):
                 sender=self.model,
             )
 
-    def _get_alias_usage_link(self, obj: Alias, request: HttpRequest, disabled: bool = False) -> str:
+    def _get_alias_usage_link(
+        self, obj: Alias, request: HttpRequest, disabled: bool = False
+    ) -> str:
         url = admin_reverse(USAGE_ALIAS_URL_NAME, args=[obj.pk])
         return self.admin_action_button(url, "info", _("View usage"), disabled=disabled)
 
     def _get_alias_delete_link(self, obj: Alias, request: HttpRequest) -> str:
         url = admin_reverse(DELETE_ALIAS_URL_NAME, args=[obj.pk])
-        return self.admin_action_button(url, "bin", _("Delete Alias"),
-                                        disabled=not self.has_delete_permission(request, obj))
+        return self.admin_action_button(
+            url,
+            "bin",
+            _("Delete Alias"),
+            disabled=not self.has_delete_permission(request, obj),
+        )
 
 
 @admin.register(AliasContent)
@@ -154,20 +172,31 @@ class AliasContentAdmin(admin.ModelAdmin):
     actions = None
     change_form_template = "admin/djangocms_alias/aliascontent/change_form.html"
 
-    def changelist_view(self, request: HttpRequest, extra_context: dict = None) -> HttpResponse:
+    def changelist_view(
+        self, request: HttpRequest, extra_context: dict = None
+    ) -> HttpResponse:
         """Needed for the Alias Content Admin breadcrumbs"""
-        return HttpResponseRedirect(admin_reverse(
-            LIST_ALIAS_URL_NAME,
-        ))
-
-    def change_view(self, request: HttpRequest, object_id: int, form_url: str = '', extra_context: dict = None) -> HttpResponse:
+        return HttpResponseRedirect(
+            admin_reverse(
+                LIST_ALIAS_URL_NAME,
+            )
+        )
+
+    def change_view(
+        self,
+        request: HttpRequest,
+        object_id: int,
+        form_url: str = "",
+        extra_context: dict = None,
+    ) -> HttpResponse:
         """Needed for the Alias Content Admin breadcrumbs"""
         obj = self.model.admin_manager.filter(pk=object_id).first()
         if not obj:
             raise Http404()
-        return HttpResponseRedirect(admin_reverse(
-            CHANGE_ALIAS_URL_NAME, args=(obj.alias_id,)
-        ) + f"?language={obj.language}")
+        return HttpResponseRedirect(
+            admin_reverse(CHANGE_ALIAS_URL_NAME, args=(obj.alias_id,))
+            + f"?language={obj.language}"
+        )
 
     def has_module_permission(self, request: HttpRequest) -> bool:
         """Hides admin class in admin site overview"""

djangocms_alias/apps.py

@@ -3,5 +3,5 @@
 
 
 class AliasConfig(AppConfig):
-    name = 'djangocms_alias'
-    verbose_name = _('django CMS Alias')
+    name = "djangocms_alias"
+    verbose_name = _("django CMS Alias")

djangocms_alias/cms_config.py

@@ -8,7 +8,7 @@
 
 
 try:
-    apps.get_app_config('djangocms_internalsearch')
+    apps.get_app_config("djangocms_internalsearch")
     from .internal_search import AliasContentConfig
 except (ImportError, LookupError):
     AliasContentConfig = None
@@ -22,20 +22,22 @@ class AliasCMSConfig(CMSAppConfig):
     moderated_models = [AliasContent]
 
     djangocms_moderation_enabled = getattr(
-        settings, 'MODERATING_ALIAS_MODELS_ENABLED', True)
-    djangocms_versioning_enabled = getattr(
-        settings, 'VERSIONING_ALIAS_MODELS_ENABLED', True) and djangocms_versioning_installed
+        settings, "MODERATING_ALIAS_MODELS_ENABLED", True
+    )
+    djangocms_versioning_enabled = (
+        getattr(settings, "VERSIONING_ALIAS_MODELS_ENABLED", True)
+        and djangocms_versioning_installed
+    )
 
     if djangocms_versioning_enabled:
-
         from cms.utils.i18n import get_language_tuple
 
         from djangocms_versioning.datastructures import VersionableItem
 
         versioning = [
             VersionableItem(
                 content_model=AliasContent,
-                grouper_field_name='alias',
+                grouper_field_name="alias",
                 extra_grouping_fields=["language"],
                 version_list_filter_lookups={"language": get_language_tuple},
                 copy_function=copy_alias_content,
@@ -44,9 +46,10 @@ class AliasCMSConfig(CMSAppConfig):
         ]
 
     djangocms_references_enabled = getattr(
-        settings, 'REFERENCES_ALIAS_MODELS_ENABLED', True)
+        settings, "REFERENCES_ALIAS_MODELS_ENABLED", True
+    )
     reference_fields = [
-        (AliasPlugin, 'alias'),
+        (AliasPlugin, "alias"),
     ]
 
     # Internalsearch configuration

djangocms_alias/cms_menus.py

@@ -9,9 +9,8 @@ class AliasDisableMenu(Modifier):
     """Disable menu rendering on alias pages"""
 
     def modify(self, request, nodes, namespace, root_id, post_cut, breadcrumb):
-        if (
-            request.toolbar.app_name == PLUGIN_URL_NAME_PREFIX
-            or isinstance(request.toolbar.obj, AliasContent)
+        if request.toolbar.app_name == PLUGIN_URL_NAME_PREFIX or isinstance(
+            request.toolbar.obj, AliasContent
         ):
             return []
         return nodes

djangocms_alias/cms_plugins.py

@@ -20,13 +20,13 @@
 
 
 __all__ = [
-    'Alias',
+    "Alias",
 ]
 
 
 @plugin_pool.register_plugin
 class Alias(CMSPluginBase):
-    name = _('Alias')
+    name = _("Alias")
     model = AliasPlugin
     form = AliasPluginForm
 
@@ -35,8 +35,8 @@ def get_render_template(self, context, instance, placeholder):
             isinstance(instance.placeholder.source, AliasContent)
             and instance.is_recursive()
         ):
-            return 'djangocms_alias/alias_recursive.html'
-        return f'djangocms_alias/{instance.template}/alias.html'
+            return "djangocms_alias/alias_recursive.html"
+        return f"djangocms_alias/{instance.template}/alias.html"
 
     @classmethod
     def get_extra_plugin_menu_items(cls, request, plugin):
@@ -49,10 +49,10 @@ def get_extra_plugin_menu_items(cls, request, plugin):
 
             plugin_menu_items = [
                 PluginMenuItem(
-                    _('Edit Alias'),
+                    _("Edit Alias"),
                     edit_endpoint,
-                    action='sideframe',
-                    attributes={'cms-icon': 'alias'},
+                    action="sideframe",
+                    attributes={"cms-icon": "alias"},
                 ),
             ]
 
@@ -63,69 +63,69 @@ def get_extra_plugin_menu_items(cls, request, plugin):
             ):
                 plugin_menu_items.append(
                     PluginMenuItem(
-                        _('Detach Alias'),
+                        _("Detach Alias"),
                         detach_endpoint,
-                        action='modal',
-                        attributes={'cms-icon': 'alias'},
+                        action="modal",
+                        attributes={"cms-icon": "alias"},
                     )
                 )
             return plugin_menu_items
 
         data = {
-            'plugin': plugin.pk,
-            'language': get_language_from_request(request, check_path=True),
+            "plugin": plugin.pk,
+            "language": get_language_from_request(request, check_path=True),
         }
         endpoint = add_url_parameters(admin_reverse(CREATE_ALIAS_URL_NAME), **data)
         return [
             PluginMenuItem(
-                _('Create Alias'),
+                _("Create Alias"),
                 endpoint,
-                action='modal',
-                attributes={'cms-icon': 'alias'},
+                action="modal",
+                attributes={"cms-icon": "alias"},
             ),
         ]
 
     @classmethod
     def get_extra_placeholder_menu_items(cls, request, placeholder):
         data = {
-            'placeholder': placeholder.pk,
-            'language': get_language_from_request(request, check_path=True),
+            "placeholder": placeholder.pk,
+            "language": get_language_from_request(request, check_path=True),
         }
         endpoint = add_url_parameters(admin_reverse(CREATE_ALIAS_URL_NAME), **data)
 
         menu_items = [
             PluginMenuItem(
-                _('Create Alias'),
+                _("Create Alias"),
                 endpoint,
-                action='modal',
-                attributes={'cms-icon': 'alias'},
+                action="modal",
+                attributes={"cms-icon": "alias"},
             ),
         ]
         return menu_items
 
     @classmethod
     def can_create_alias(cls, user, plugins=None, replace=False):
         if not user.has_perm(
-            get_model_permission_codename(AliasModel, 'add'),
+            get_model_permission_codename(AliasModel, "add"),
         ):
             return False
 
         if not plugins:
             return True
         elif replace:
             target_placeholder = plugins[0].placeholder
-            if (
-                not target_placeholder.check_source(user)
-                or not has_plugin_permission(user, Alias.__name__, 'add')
+            if not target_placeholder.check_source(user) or not has_plugin_permission(
+                user, Alias.__name__, "add"
             ):
                 return False
 
         return all(
             has_plugin_permission(
                 user,
                 plugin.plugin_type,
-                'add',
-            ) for plugin in plugins
+                "add",
+            )
+            for plugin in plugins
         )
 
     @classmethod
@@ -134,13 +134,16 @@ def can_detach(cls, user, target_placeholder, plugins):
             has_plugin_permission(
                 user,
                 plugin.plugin_type,
-                'add',
-            ) for plugin in plugins
+                "add",
+            )
+            for plugin in plugins
         ) and target_placeholder.check_source(user)
 
     @classmethod
     def detach_alias_plugin(cls, plugin, language):
-        source_placeholder = plugin.alias.get_placeholder(language, show_draft_content=True)  # We're in edit mode
+        source_placeholder = plugin.alias.get_placeholder(
+            language, show_draft_content=True
+        )  # We're in edit mode
         target_placeholder = plugin.placeholder
 
         # Deleting uses a copy of a plugin to preserve pk on existing