From 27e7c08b184c1271425e909f268bb14599062060 Mon Sep 17 00:00:00 2001 From: AlternC VM Date: Mon, 30 Jun 2014 17:20:10 +0200 Subject: [PATCH] refactoring with regard to old API ;) --- bureau/class/m_action.php | 37 ++++++++++++------------------------- 1 file changed, 12 insertions(+), 25 deletions(-) diff --git a/bureau/class/m_action.php b/bureau/class/m_action.php index 844516ed..5270dfbb 100644 --- a/bureau/class/m_action.php +++ b/bureau/class/m_action.php @@ -185,31 +185,18 @@ class m_action { $err->log("action", "set", $type); $serialized = serialize($parameters); - switch ($type) { - case 'create_file': - $query = "insert into actions values ('','CREATE_FILE','$serialized',now(),'','','$user','');"; - break; - case 'create_dir': - $query = "insert into actions values ('','CREATE_DIR','$serialized',now(),'','','$user','');"; - break; - case 'move': - $query = "insert into actions values ('','MOVE','$serialized',now(),'','','$user','');"; - break; - case 'fix_user': - $query = "insert into actions values ('','FIX_USER','$serialized',now(),'','','$user','');"; - break; - case 'fix_file': - $query = "insert into actions values ('','FIX_FILE','$serialized',now(),'','','$user','');"; - break; - case 'fix_dir': - $query = "insert into actions values ('','FIX_DIR','$serialized',now(),'','','$user','');"; - break; - case 'delete': - $query = "insert into actions values ('','DELETE','$serialized',now(),'','','$user','');"; - break; - default: - return false; - } + $type = strtoupper($type); + if (in_array($type, array('CREATE_FILE', + 'CREATE_DIR', + 'MOVE', + 'FIX_USER', + 'FIX_FILE', + 'FIX_DIR', + 'DELETE'))) { + $query = "INSERT INTO `actions` (type, parameters, creation, user) VALUES('$type', '$serialized', now(), '$user');"; + } else { + return False; + } if (!$db->query($query)) { $err->raise("action", _("Error setting actions")); return false;