summaryrefslogtreecommitdiff
path: root/plugins/password
diff options
context:
space:
mode:
authorsimonp <simon.plasger@web.de>2014-06-10 13:06:10 +0200
committersimonp <simon.plasger@web.de>2014-06-10 13:06:10 +0200
commit12514266b94269eebf399cb6052687762f370e9d (patch)
treeeffafd3fc18b94c762e57c3c1a89e2b63b0814a2 /plugins/password
parent6f7042e5820997464fa2198487d37e90cbf054f9 (diff)
Move login/hosts to seperate function
Diffstat (limited to 'plugins/password')
-rw-r--r--plugins/password/password.php64
1 files changed, 32 insertions, 32 deletions
diff --git a/plugins/password/password.php b/plugins/password/password.php
index 74c53c9d9..7b6b80dc7 100644
--- a/plugins/password/password.php
+++ b/plugins/password/password.php
@@ -44,8 +44,6 @@ class password extends rcube_plugin
public $noframe = true;
public $noajax = true;
private $newuser = false;
- private $login_exceptions;
- private $allowed_hosts;
function init()
{
@@ -53,27 +51,10 @@ class password extends rcube_plugin
$this->load_config();
- // Host exceptions
- $hosts = $rcmail->config->get('password_hosts');
- $this->allowed_hosts = $hosts;
- if (!empty($hosts) && !in_array($_SESSION['storage_host'], $hosts)) {
+ if($rcmail->task == 'settings' && !$this->check_host_login_exceptions()) {
return;
}
-
- // Login exceptions
- if ($exceptions = $rcmail->config->get('password_login_exceptions')) {
- $exceptions = array_map('trim', (array) $exceptions);
- $exceptions = array_filter($exceptions);
- $this->login_exceptions = $exceptions;
- $username = $_SESSION['username'];
-
- foreach ($exceptions as $ec) {
- if ($username === $ec) {
- return;
- }
- }
- }
-
+
$this->add_hook('settings_actions', array($this, 'settings_actions'));
if($rcmail->config->get('password_force_new_user')) {
$this->add_hook('user_create', array($this, 'user_create'));
@@ -322,17 +303,8 @@ class password extends rcube_plugin
function login_after($args)
{
- $rcmail = rcmail::get_instance();
- $userstruct = $rcmail->user;
- $username = $userstruct->get_username();
- foreach ($this->login_exceptions as $ec) {
- if ($username === $ec) {
- return $args;
- }
- }
- $domain = $userstruct->get_username('domain');
- if (!empty($this->allowed_hosts) && !in_array($domain, $this->allowed_hosts)) {
- return;
+ if(!$this->check_host_login_exceptions()) {
+ return $args;
}
if($this->newuser)
{
@@ -342,4 +314,32 @@ class password extends rcube_plugin
}
return $args;
}
+
+ // Check if host and login is allowed to change the password, false = not allowed, true = not allowed
+ private function check_host_login_exceptions()
+ {
+ $rcmail = rcmail::get_instance();
+ // Host exceptions
+ $hosts = $rcmail->config->get('password_hosts');
+ $this->allowed_hosts = $hosts;
+ if (!empty($hosts) && !in_array($_SESSION['storage_host'], $hosts)) {
+ return false;
+ }
+
+
+ // Login exceptions
+ if ($exceptions = $rcmail->config->get('password_login_exceptions')) {
+ $exceptions = array_map('trim', (array) $exceptions);
+ $exceptions = array_filter($exceptions);
+ $this->login_exceptions = $exceptions;
+ $username = $_SESSION['username'];
+
+ foreach ($exceptions as $ec) {
+ if ($username === $ec) {
+ return false;
+ }
+ }
+ }
+ return true;
+ }
}