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 /lib/api.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 'lib/api.php')
| -rw-r--r-- | lib/api.php | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/api.php b/lib/api.php index cb67e0c2a89..c464cd04913 100644 --- a/lib/api.php +++ b/lib/api.php @@ -127,8 +127,7 @@ class OC_API { return false; } else { $subAdmin = OC_SubAdmin::isSubAdmin($user); - $admin = OC_Group::inGroup($user, 'admin'); - if($subAdmin || $admin) { + if($subAdmin) { return true; } else { return false; @@ -141,7 +140,7 @@ class OC_API { if(!$user) { return false; } else { - return OC_Group::inGroup($user, 'admin'); + return OC_User::isAdminUser($user); } break; default: |