diff options
author | Lukas Reschke <lukas@statuscode.ch> | 2013-01-14 19:45:17 +0100 |
---|---|---|
committer | Lukas Reschke <lukas@statuscode.ch> | 2013-01-14 19:45:17 +0100 |
commit | 31b1a73e1f508a33843835b632ee05c7bfc2aaaf (patch) | |
tree | e590662315799744155151b2c6c9148d2e9e860e /settings/ajax/removeuser.php | |
parent | fa78fbe0c3e281c91468bee92c871be7b1e48c79 (diff) | |
download | nextcloud-server-31b1a73e1f508a33843835b632ee05c7bfc2aaaf.tar.gz nextcloud-server-31b1a73e1f508a33843835b632ee05c7bfc2aaaf.zip |
Check if user is admin - bool
There was no "isAdminUser()" function which returned bool. This is
irritiating as there were a loooooooot of places in the code which
checked this itself with `OC_Group::inGroup($uid, 'admin)` - why not
use a function for this?
(Especially if you consider that we might change the group name in the
future, which would lead to problems then)
Additionally, @Raydiation needed such a method for his AppFramework :)
Diffstat (limited to 'settings/ajax/removeuser.php')
-rw-r--r-- | settings/ajax/removeuser.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/settings/ajax/removeuser.php b/settings/ajax/removeuser.php index 9ffb32a0b23..bf3a34f1472 100644 --- a/settings/ajax/removeuser.php +++ b/settings/ajax/removeuser.php @@ -10,7 +10,7 @@ if(OC_User::getUser() === $username) { exit; } -if(!OC_Group::inGroup(OC_User::getUser(), 'admin') && !OC_SubAdmin::isUserAccessible(OC_User::getUser(), $username)) { +if(!OC_User::isAdminUser(OC_User::getUser()) && !OC_SubAdmin::isUserAccessible(OC_User::getUser(), $username)) { $l = OC_L10N::get('core'); OC_JSON::error(array( 'data' => array( 'message' => $l->t('Authentication error') ))); exit(); |