Skip to content
Snippets Groups Projects
Commit 249022a3 authored by Jonathan Weth's avatar Jonathan Weth :keyboard:
Browse files

Merge branch 'bugfix/system-check-mistakes' into 'master'

Fix some mistakes in app config system check

See merge request AlekSIS!164
parents f824a314 4ece0c14
No related branches found
No related tags found
1 merge request!164Fix some mistakes in app config system check
Pipeline #902 failed
......@@ -10,7 +10,7 @@ from .util.apps import AppConfig
@register(Tags.compatibility)
def check_app_configs_base_class(
app_configs: Optional[django.apps.registry.Apps] = None, **kwargs
) -> None:
) -> list:
""" Checks whether all apps derive from AlekSIS's base app config """
results = []
......@@ -18,11 +18,11 @@ def check_app_configs_base_class(
if app_configs is None:
app_configs = django.apps.apps.get_app_configs()
for app_config in app_configs:
if app_config.name.startswith("aleksis.apps.") and not isinstance(app_config, AppConfig):
for app_config in filter(lambda c: c.name.startswith("aleksis."), app_configs):
if not isinstance(app_config, AppConfig):
results.append(
Warning(
"App config %s does not derive from aleksis.core.util.apps.AppConfig.",
"App config %s does not derive from aleksis.core.util.apps.AppConfig." % app_config.name,
hint="Ensure the app uses the correct base class for all registry functionality to work.",
obj=app_config,
id="aleksis.core.W001",
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment