fix: Undo thread critical opt.

fsbraun · fsbraun · commit e2e78c1539f2 · 2025-09-12T11:59:20.000+02:00
diff --git a/cms/models/pluginmodel.py b/cms/models/pluginmodel.py
@@ -449,31 +449,24 @@ def get_action_urls(self, js_compat=True):
         This method replaces the set of legacy methods `get_add_url`, ``get_edit_url`, `get_move_url`,
         `get_delete_url`, `get_copy_url`.
         """
-        if not hasattr(CMSPlugin, '_edit_url'):
-            CMSPlugin._edit_url = admin_reverse('cms_placeholder_edit_plugin', args=(0,))
-            CMSPlugin._add_url = admin_reverse('cms_placeholder_add_plugin')
-            CMSPlugin._delete_url = admin_reverse('cms_placeholder_delete_plugin', args=(0,))
-            CMSPlugin._move_url = admin_reverse('cms_placeholder_move_plugin')
-            CMSPlugin._copy_url = admin_reverse('cms_placeholder_copy_plugins')
-
         if js_compat:
             # TODO: Remove this condition
             # once the javascript files have been refactored
             # to use the new naming schema (ending in _url).
             return {
-                'edit_plugin': re.sub(r"/0/", f"/{self.pk}/", CMSPlugin._edit_url),
-                'add_plugin': CMSPlugin._add_url,
-                'delete_plugin': re.sub(r"/0/", f"/{self.pk}/", CMSPlugin._delete_url),
-                'move_plugin': CMSPlugin._move_url,
-                'copy_plugin': CMSPlugin._copy_url,
+                'edit_plugin': admin_reverse('cms_placeholder_edit_plugin', args=(self.pk,)),
+                'add_plugin': admin_reverse('cms_placeholder_add_plugin'),
+                'delete_plugin': admin_reverse('cms_placeholder_delete_plugin', args=(self.pk,)),
+                'move_plugin': admin_reverse('cms_placeholder_move_plugin'),
+                'copy_plugin':admin_reverse('cms_placeholder_copy_plugins'),
             }
         else:
             return {
-                'edit_url': re.sub(r"/0/", f"/{self.pk}/", CMSPlugin._edit_url),
-                'add_url': CMSPlugin._add_url,
-                'delete_url': re.sub(r"/0/", f"/{self.pk}/", CMSPlugin._delete_url),
-                'move_url': CMSPlugin._move_url,
-                'copy_url': CMSPlugin._copy_url,
+                'edit_url': admin_reverse('cms_placeholder_edit_plugin', args=(self.pk,)),
+                'add_url': admin_reverse('cms_placeholder_add_plugin'),
+                'delete_url': admin_reverse('cms_placeholder_delete_plugin', args=(self.pk,)),
+                'move_url': admin_reverse('cms_placeholder_move_plugin'),
+                'copy_url': admin_reverse('cms_placeholder_copy_plugins'),
             }
 
 
