diff options
author | Aleksander Machniak <alec@alec.pl> | 2014-07-16 15:12:53 +0200 |
---|---|---|
committer | Aleksander Machniak <alec@alec.pl> | 2014-07-16 15:12:53 +0200 |
commit | 9d2550502a68ea3cc495e287ebff04e142a59e2e (patch) | |
tree | 71e1bc9ae1c310b95e5c0d417a3127eb40ca8051 /plugins/password | |
parent | b65823eb2ad04dfd8977717f668a5fbc6fa3c303 (diff) |
CS fixes after PR merge
Diffstat (limited to 'plugins/password')
-rw-r--r-- | plugins/password/drivers/gearman.php | 71 |
1 files changed, 38 insertions, 33 deletions
diff --git a/plugins/password/drivers/gearman.php b/plugins/password/drivers/gearman.php index 6c1ad3cf0..36571a98f 100644 --- a/plugins/password/drivers/gearman.php +++ b/plugins/password/drivers/gearman.php @@ -1,4 +1,5 @@ <?php + /** * Gearman Password Driver * @@ -11,40 +12,44 @@ class rcube_gearman_password { - function save($currpass, $newpass) - { - $user = $_SESSION['username']; - $rcmail = rcmail::get_instance(); + function save($currpass, $newpass) + { + if (extension_loaded('gearman')) { + $rcmail = rcmail::get_instance(); + $user = $_SESSION['username']; + $payload = array( + 'username' => $user, + 'oldPassword' => $currpass, + 'newPassword' => $newpass, + ); - if (extension_loaded('gearman')) { - $success = false; - $gmc= new GearmanClient(); + $gmc = new GearmanClient(); + $gmc->addServer($rcmail->config->get('password_gearman_host')); - $gmc->addServer($rcmail->config->get('password_gearman_host')); - $payload = array("username" => $user, "oldPassword" => $currpass, "newPassword" => $newpass); - $result = $gmc->doNormal("setPassword", json_encode($payload)); - $success = json_decode($result); - if ($success->result == 1) { - return PASSWORD_SUCCESS; - } else { - rcube::raise_error(array( - 'code' => 600, - 'type' => 'php', - 'file' => __FILE__, 'line' => __LINE__, - 'message' => "Password plugin: Gearman authentication failed for user $user: $error" - ), true, false); - } - } - else { - rcube::raise_error(array( - 'code' => 600, - 'type' => 'php', - 'file' => __FILE__, 'line' => __LINE__, - 'message' => "Password plugin: PECL Gearman module not loaded" - ), true, false); - } + $result = $gmc->doNormal('setPassword', json_encode($payload)); + $success = json_decode($result); - return PASSWORD_ERROR; - } + if ($success && $success->result == 1) { + return PASSWORD_SUCCESS; + } + else { + rcube::raise_error(array( + 'code' => 600, + 'type' => 'php', + 'file' => __FILE__, 'line' => __LINE__, + 'message' => "Password plugin: Gearman authentication failed for user $user: $error" + ), true, false); + } + } + else { + rcube::raise_error(array( + 'code' => 600, + 'type' => 'php', + 'file' => __FILE__, 'line' => __LINE__, + 'message' => "Password plugin: PECL Gearman module not loaded" + ), true, false); + } + + return PASSWORD_ERROR; + } } -?> |