Fix: Cut children from inactive nodes when level is less or equal to 0 (#8322) (#8324)

fsbraun · stefanw · web-flow · commit bc2cf65a0c91 · 2025-09-01T21:05:02.000+02:00
It can be less than zero when the from_level is greater than zero.

Co-authored-by: Stefan Wehrmeyer &lt;stefanw@users.noreply.github.com&gt;
diff --git a/cms/tests/test_menu.py b/cms/tests/test_menu.py
@@ -227,6 +227,16 @@ def test_menu_failfast_on_invalid_usage(self):
             tpl = Template("{% load menu_tags %}{% show_menu 0 0 0 0 'menu/menu.html' child %}")
             self.assertRaises(TemplateSyntaxError, tpl.render, context)
 
+    def test_show_menu_cut_inactive(self):
+        root = self.get_page(2)
+        context = self.get_context(page=root)
+        tpl = Template("{% load menu_tags %}{% show_menu 1 100 0 1 %}")
+        tpl.render(context)
+        nodes = context["children"]
+        self.assertEqual(len(nodes), 2)
+        self.assertEqual(len(nodes[0].children), 1)
+        self.assertEqual(len(nodes[1].children), 0)
+
     def test_show_submenu_nephews(self):
         page_2 = self.get_page(2)
         context = self.get_context(path=page_2.get_absolute_url(), page=page_2)
diff --git a/menus/templatetags/menu_tags.py b/menus/templatetags/menu_tags.py
@@ -32,7 +32,7 @@ def cut_after(node, levels, removed=None):
     if removed is not None:
         raise TypeError("menus.template_tags.cut_after() does not accept 'removed' list argument")
 
-    if levels == 0:
+    if levels <= 0:
         node.children = []
     else:
         for child in node.children:
