fix: Modify language menu for pages only if it is present (#179)

* fix: toolbar fails if language menu is not present * update test environment * Add test * Update compile.py * Update tests/test_toolbar.py Co-authored-by: Mark Walker <[email protected]> --------- Co-authored-by: Mark Walker <[email protected]>
django-cms · May 26, 2023 · 4060eb9 · 4060eb9
1 parent e7f8256
commit 4060eb9
Show file tree

Hide file tree

Showing 31 changed files with 682 additions and 3,728 deletions.
diff --git a/djangocms_alias/cms_toolbars.py b/djangocms_alias/cms_toolbars.py
@@ -178,6 +178,8 @@ def enable_create_wizard_button(self):
 
     def override_language_switcher(self):
         language_menu = self.toolbar.get_menu(LANGUAGE_MENU_IDENTIFIER, _('Language'))
+        if not language_menu:
+            return
         # Remove all existing language links
         # remove_item uses `items` attribute so we have to copy object
         for _item in copy(language_menu.items):

diff --git a/tests/requirements/compile.py b/tests/requirements/compile.py
@@ -69,7 +69,7 @@ def run(*args, **kwargs):
         "-m",
         "piptools",
         "compile",
-        "--generate-hashes",
+        # temporarily remove "--generate-hashes", until all dependencies are actual releases
         "--allow-unsafe",
     ] + sys.argv[1:]
 

diff --git a/tests/requirements/djmain-py310-default.txt b/tests/requirements/djmain-py310-default.txt