[checkmk-commits] 4077 FIX Fixed "Request URI too long" when cloning role after editing another one

Module: check_mk Branch: master Commit: f74d454f34d899f87a404b1f6bd0e01455648641 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=f74d454f34d899… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Mon Nov 28 14:38:16 2016 +0100 4077 FIX Fixed "Request URI too long" when cloning role after editing another one --- .werks/4077 | 9 +++++++++ ChangeLog | 1 + web/htdocs/wato.py | 4 ++-- 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/.werks/4077 b/.werks/4077 new file mode 100644 index 0000000..02abfb6 --- /dev/null +++ b/.werks/4077 @@ -0,0 +1,9 @@ +Title: Fixed "Request URI too long" when cloning role after editing another one +Level: 1 +Component: wato +Compatible: compat +Version: 1.4.0i3 +Date: 1480340280 +Class: fix + + diff --git a/ChangeLog b/ChangeLog index c82c455..e6878e9 100644 --- a/ChangeLog +++ b/ChangeLog @@ -60,6 +60,7 @@ * 4068 FIX: Fixed possible exception in case of wrong automation secrets between multiple sites... * 3992 FIX: netapp_api_connection: fixed GUI display bug when editing rules * 4076 FIX: Git integration: Using mail address of user in author info + * 4077 FIX: Fixed "Request URI too long" when cloning role after editing another one Notifications: * 4042 FIX: Fix missing From address in synchronous SMTP notifications... diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py index 1164e61..88b2695 100644 --- a/web/htdocs/wato.py +++ b/web/htdocs/wato.py @@ -10751,8 +10751,8 @@ def mode_roles(phase): # Actions table.cell(_("Actions"), css="buttons") edit_url = folder_preserving_link([("mode", "edit_role"), ("edit", id)]) - clone_url = html.makeactionuri([("_clone", id)]) - delete_url = html.makeactionuri([("_delete", id)]) + clone_url = make_action_link([("mode", "roles"), ("_clone", id)]) + delete_url = make_action_link([("mode", "roles"), ("_delete", id)]) html.icon_button(edit_url, _("Properties"), "edit") html.icon_button(clone_url, _("Clone"), "clone") if not role.get("builtin"):