diff options
| author | Lukas Reschke <lukas@statuscode.ch> | 2013-01-14 22:45:17 +0400 |
|---|---|---|
| committer | Lukas Reschke <lukas@statuscode.ch> | 2013-01-14 22:45:17 +0400 |
| commit | 31b1a73e1f508a33843835b632ee05c7bfc2aaaf (patch) | |
| tree | e590662315799744155151b2c6c9148d2e9e860e /lib/migrate.php | |
| parent | fa78fbe0c3e281c91468bee92c871be7b1e48c79 (diff) | |
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/migrate.php')
| -rw-r--r-- | lib/migrate.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/migrate.php b/lib/migrate.php index 5ff8e338a44..62ab00bc332 100644 --- a/lib/migrate.php +++ b/lib/migrate.php @@ -219,7 +219,7 @@ class OC_Migrate{ // We need to be an admin if we are not importing our own data if(($type == 'user' && self::$uid != $currentuser) || $type != 'user' ) { - if( !OC_Group::inGroup( OC_User::getUser(), 'admin' )) { + if( !OC_User::isAdminUser($currentuser)) { // Naughty. OC_Log::write( 'migration', 'Import not permitted.', OC_Log::ERROR ); return json_encode( array( 'success' => false ) ); |