[checkmk-commits] Check_MK Git: check_mk: WATO: fix permissions when moving folders

Module: check_mk Branch: master Commit: f0050f1f0de49923188f3dccca155aaba39e2606 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=f0050f1f0de499… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Feb 21 16:30:41 2012 +0100 WATO: fix permissions when moving folders --- .bugs/621 | 8 ++++++-- ChangeLog | 1 + web/htdocs/wato.py | 1 + 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/.bugs/621 b/.bugs/621 index 6ebd595..d3215bc 100644 --- a/.bugs/621 +++ b/.bugs/621 @@ -1,8 +1,12 @@ Title: Bulk moveto berücksichtigt die Nutzer Rechte nicht Component: wato -State: open +Class: bug +State: done Date: 2012-01-18 10:30:57 Targetversion: 1.2.0 -Class: bug Ich kann Hosts aus anderen Foldern in meine Folder verschieben + +2012-02-21 16:30:09: changed state open -> done +Ist jetzt repariert. War auch beim normalen verschieben. Jetzt +braucht man Schreibrechte auf Quelle und Ziel. diff --git a/ChangeLog b/ChangeLog index d9dddd8..72dd885 100644 --- a/ChangeLog +++ b/ChangeLog @@ -25,6 +25,7 @@ * FIX: honor site disabling in replication module * FIX: honor write permissions on folder in "bulk delete" * FIX: honor permissions for "bulk cleanup" and "bulk edit" + * FIX: honor write permissions and source folder when moving hosts Checks & Agents: * hpux_if: fix missing default parameter errors diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py index cb30ba3..951ccbb 100644 --- a/web/htdocs/wato.py +++ b/web/htdocs/wato.py @@ -1343,6 +1343,7 @@ def move_hosts_to(hostnames, path): return target_folder = g_folders[path] + check_folder_permissions(g_folder, "write") check_folder_permissions(target_folder, "write") if target_folder == g_folder: