mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #8847 from nextcloud/ocs-api-userdetails 

Add userdetails to ocs api
This commit is contained in:
Morris Jobke 2018-03-20 18:12:25 +01:00 committed by GitHub
parent 64feb3f391 24659342f5
commit 29c551c88f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 170 additions and 74 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
apps/provisioning_api/appinfo/routes.php
View File

@ -41,6 +41,7 @@ return [
// Users
['root' => '/cloud', 'name' => 'Users#getUsers', 'url' => '/users', 'verb' => 'GET'],
['root' => '/cloud', 'name' => 'Users#getUsersDetails', 'url' => '/users/details', 'verb' => 'GET'],
['root' => '/cloud', 'name' => 'Users#addUser', 'url' => '/users', 'verb' => 'POST'],
['root' => '/cloud', 'name' => 'Users#getUser', 'url' => '/users/{userId}', 'verb' => 'GET'],
['root' => '/cloud', 'name' => 'Users#getCurrentUser', 'url' => '/user', 'verb' => 'GET'],

90
apps/provisioning_api/lib/Controller/UsersController.php
View File

@ -124,7 +124,7 @@ class UsersController extends OCSController {
* @param int $offset
* @return DataResponse
*/
public function getUsers(string $search = '', $limit = null, $offset = null): DataResponse {
public function getUsers(string $search = '', $limit = null, $offset = 0): DataResponse {
$user = $this->userSession->getUser();
$users = [];
@ -139,16 +139,10 @@ class UsersController extends OCSController {
$subAdminOfGroups[$key] = $group->getGID();
}
if($offset === null) {
$offset = 0;
}
$users = [];
foreach ($subAdminOfGroups as $group) {
$users = array_merge($users, $this->groupManager->displayNamesInGroup($group, $search));
$users = array_merge($users, $this->groupManager->displayNamesInGroup($group, $search, $limit, $offset));
}
$users = array_slice($users, $offset, $limit);
}
$users = array_keys($users);
@ -158,6 +152,47 @@ class UsersController extends OCSController {
]);
}
/**
* @NoAdminRequired
*
* returns a list of users and their data
*/
public function getUsersDetails(string $search = '', $limit = null, $offset = 0): DataResponse {
$user = $this->userSession->getUser();
$users = [];
// Admin? Or SubAdmin?
$uid = $user->getUID();
$subAdminManager = $this->groupManager->getSubAdmin();
if($this->groupManager->isAdmin($uid)){
$users = $this->userManager->search($search, $limit, $offset);
} else if ($subAdminManager->isSubAdmin($user)) {
$subAdminOfGroups = $subAdminManager->getSubAdminsGroups($user);
foreach ($subAdminOfGroups as $key => $group) {
$subAdminOfGroups[$key] = $group->getGID();
}
$users = [];
foreach ($subAdminOfGroups as $group) {
$users = array_merge($users, $this->groupManager->displayNamesInGroup($group, $search, $limit, $offset));
}
}
$users = array_keys($users);
$usersDetails = [];
foreach ($users as $key => $userId) {
$userData = $this->getUserData($userId);
// Do not insert empty entry
if(!empty($userData)) {
$usersDetails[$userId] = $userData;
}
}
return new DataResponse([
'users' => $usersDetails
]);
}
/**
* @PasswordConfirmationRequired
* @NoAdminRequired
@ -232,6 +267,10 @@ class UsersController extends OCSController {
*/
public function getUser(string $userId): DataResponse {
$data = $this->getUserData($userId);
// getUserData returns empty array if not enough permissions
if(empty($data)) {
throw new OCSException('', \OCP\API::RESPOND_UNAUTHORISED);
}
return new DataResponse($data);
}
@ -277,17 +316,18 @@ class UsersController extends OCSController {
throw new OCSException('The requested user could not be found', \OCP\API::RESPOND_NOT_FOUND);
}
// Admin? Or SubAdmin?
if($this->groupManager->isAdmin($currentLoggedInUser->getUID())
// Should be at least Admin Or SubAdmin!
if( $this->groupManager->isAdmin($currentLoggedInUser->getUID())
|| $this->groupManager->getSubAdmin()->isUserAccessible($currentLoggedInUser, $targetUserObject)) {
$data['enabled'] = $this->config->getUserValue($targetUserObject->getUID(), 'core', 'enabled', 'true');
$data['enabled'] = $this->config->getUserValue($targetUserObject->getUID(), 'core', 'enabled', 'true');
} else {
// Check they are looking up themselves
if($currentLoggedInUser->getUID() !== $targetUserObject->getUID()) {
throw new OCSException('', \OCP\API::RESPOND_UNAUTHORISED);
return $data;
}
}
// Get groups data
$userAccount = $this->accountManager->getUser($targetUserObject);
$groups = $this->groupManager->getUserGroups($targetUserObject);
$gids = [];
@ -297,6 +337,10 @@ class UsersController extends OCSController {
// Find the data
$data['id'] = $targetUserObject->getUID();
$data['storageLocation'] = $targetUserObject->getHome();
$data['lastLogin'] = $targetUserObject->getLastLogin() * 1000;
$data['backend'] = $targetUserObject->getBackendClassName();
$data['subadmin'] = $this->getUserSubAdminGroupsData($targetUserObject->getUID());
$data['quota'] = $this->fillStorageInfo($targetUserObject->getUID());
$data[AccountManager::PROPERTY_EMAIL] = $targetUserObject->getEMailAddress();
$data[AccountManager::PROPERTY_DISPLAYNAME] = $targetUserObject->getDisplayName();
@ -779,10 +823,10 @@ class UsersController extends OCSController {
* Get the groups a user is a subadmin of
*
* @param string $userId
* @return DataResponse
* @return array
* @throws OCSException
*/
public function getUserSubAdminGroups(string $userId): DataResponse {
protected function getUserSubAdminGroupsData(string $userId): array {
$user = $this->userManager->get($userId);
// Check if the user exists
if($user === null) {
@ -796,11 +840,19 @@ class UsersController extends OCSController {
$groups[] = $group->getGID();
}
if(!$groups) {
throw new OCSException('Unknown error occurred', 102);
} else {
return new DataResponse($groups);
}
return $groups;
}
/**
* Get the groups a user is a subadmin of
*
* @param string $userId
* @return DataResponse
* @throws OCSException
*/
public function getUserSubAdminGroups(string $userId): DataResponse {
$groups = $this->getUserSubAdminGroupsData($userId);
return new DataResponse($groups);
}
/**

153
apps/provisioning_api/tests/Controller/UsersControllerTest.php
View File

@ -139,7 +139,7 @@ class UsersControllerTest extends TestCase {
$this->userManager
->expects($this->once())
->method('search')
->with('MyCustomSearch', null, null)
->with('MyCustomSearch')
->will($this->returnValue(['Admin' => [], 'Foo' => [], 'Bar' => []]));
$expected = ['users' => [
@ -662,6 +662,9 @@ class UsersControllerTest extends TestCase {
$loggedInUser = $this->getMockBuilder(IUser::class)
->disableOriginalConstructor()
->getMock();
$subAdminManager = $this->getMockBuilder('OC\SubAdmin')
->disableOriginalConstructor()
->getMock();
$loggedInUser
->expects($this->once())
->method('getUID')
@ -671,15 +674,15 @@ class UsersControllerTest extends TestCase {
->getMock();
$targetUser->expects($this->once())
->method('getEMailAddress')
->willReturn('demo@owncloud.org');
->willReturn('demo@nextcloud.com');
$this->userSession
->expects($this->once())
->method('getUser')
->will($this->returnValue($loggedInUser));
$this->userManager
->expects($this->once())
->expects($this->exactly(2))
->method('get')
->with('UserToGet')
->with('UID')
->will($this->returnValue($targetUser));
$this->groupManager
->expects($this->once())
@ -690,6 +693,14 @@ class UsersControllerTest extends TestCase {
->expects($this->any())
->method('getUserGroups')
->willReturn([$group, $group, $group]);
$this->groupManager
->expects($this->once())
->method('getSubAdmin')
->will($this->returnValue($subAdminManager));
$subAdminManager
->expects($this->once())
->method('getSubAdminsGroups')
->willReturn([$group]);
$group->expects($this->at(0))
->method('getDisplayName')
->willReturn('group0');
@ -699,6 +710,9 @@ class UsersControllerTest extends TestCase {
$group->expects($this->at(2))
->method('getDisplayName')
->willReturn('group2');
$group->expects($this->at(3))
->method('getGID')
->willReturn('group3');
$this->accountManager->expects($this->any())->method('getUser')
->with($targetUser)
->willReturn(
@ -713,7 +727,7 @@ class UsersControllerTest extends TestCase {
->expects($this->at(0))
->method('getUserValue')
->with('UID', 'core', 'enabled', 'true')
->will($this->returnValue('true'));
->will($this->returnValue('true'));
$this->config
->expects($this->at(1))
->method('getUserValue')
@ -729,15 +743,31 @@ class UsersControllerTest extends TestCase {
->method('getDisplayName')
->will($this->returnValue('Demo User'));
$targetUser
->expects($this->exactly(4))
->expects($this->once())
->method('getHome')
->will($this->returnValue('/var/www/newtcloud/data/UID'));
$targetUser
->expects($this->once())
->method('getLastLogin')
->will($this->returnValue(1521191471));
$targetUser
->expects($this->once())
->method('getBackendClassName')
->will($this->returnValue('Database'));
$targetUser
->expects($this->exactly(5))
->method('getUID')
->will($this->returnValue('UID'));
$expected = [
'id' => 'UID',
'enabled' => 'true',
'storageLocation' => '/var/www/newtcloud/data/UID',
'lastLogin' => 1521191471000,
'backend' => 'Database',
'subadmin' => ['group3'],
'quota' => ['DummyValue'],
'email' => 'demo@owncloud.org',
'email' => 'demo@nextcloud.com',
'displayname' => 'Demo User',
'phone' => 'phone',
'address' => 'address',
@ -746,7 +776,7 @@ class UsersControllerTest extends TestCase {
'groups' => ['group0', 'group1', 'group2'],
'language' => 'de',
];
$this->assertEquals($expected, $this->invokePrivate($this->api, 'getUserData', ['UserToGet']));
$this->assertEquals($expected, $this->invokePrivate($this->api, 'getUserData', ['UID']));
}
public function testGetUserDataAsSubAdminAndUserIsAccessible() {
@ -763,15 +793,15 @@ class UsersControllerTest extends TestCase {
$targetUser
->expects($this->once())
->method('getEMailAddress')
->willReturn('demo@owncloud.org');
->willReturn('demo@nextcloud.com');
$this->userSession
->expects($this->once())
->method('getUser')
->will($this->returnValue($loggedInUser));
$this->userManager
->expects($this->once())
->expects($this->exactly(2))
->method('get')
->with('UserToGet')
->with('UID')
->will($this->returnValue($targetUser));
$this->groupManager
->expects($this->once())
@ -790,8 +820,12 @@ class UsersControllerTest extends TestCase {
->method('isUserAccessible')
->with($loggedInUser, $targetUser)
->will($this->returnValue(true));
$this->groupManager
$subAdminManager
->expects($this->once())
->method('getSubAdminsGroups')
->willReturn([]);
$this->groupManager
->expects($this->exactly(2))
->method('getSubAdmin')
->will($this->returnValue($subAdminManager));
$this->config
@ -814,7 +848,19 @@ class UsersControllerTest extends TestCase {
->method('getDisplayName')
->will($this->returnValue('Demo User'));
$targetUser
->expects($this->exactly(4))
->expects($this->once())
->method('getHome')
->will($this->returnValue('/var/www/newtcloud/data/UID'));
$targetUser
->expects($this->once())
->method('getLastLogin')
->will($this->returnValue(1521191471));
$targetUser
->expects($this->once())
->method('getBackendClassName')
->will($this->returnValue('Database'));
$targetUser
->expects($this->exactly(5))
->method('getUID')
->will($this->returnValue('UID'));
$this->accountManager->expects($this->any())->method('getUser')
@ -831,8 +877,12 @@ class UsersControllerTest extends TestCase {
$expected = [
'id' => 'UID',
'enabled' => 'true',
'storageLocation' => '/var/www/newtcloud/data/UID',
'lastLogin' => 1521191471000,
'backend' => 'Database',
'subadmin' => [],
'quota' => ['DummyValue'],
'email' => 'demo@owncloud.org',
'email' => 'demo@nextcloud.com',
'displayname' => 'Demo User',
'phone' => 'phone',
'address' => 'address',
@ -841,7 +891,7 @@ class UsersControllerTest extends TestCase {
'groups' => [],
'language' => 'da',
];
$this->assertEquals($expected, $this->invokePrivate($this->api, 'getUserData', ['UserToGet']));
$this->assertEquals($expected, $this->invokePrivate($this->api, 'getUserData', ['UID']));
}
@ -887,7 +937,7 @@ class UsersControllerTest extends TestCase {
->method('getSubAdmin')
->will($this->returnValue($subAdminManager));
$this->invokePrivate($this->api, 'getUserData', ['UserToGet']);
$this->invokePrivate($this->api, 'getUser', ['UserToGet']);
}
public function testGetUserDataAsSubAdminSelfLookup() {
@ -906,9 +956,9 @@ class UsersControllerTest extends TestCase {
->method('getUser')
->will($this->returnValue($loggedInUser));
$this->userManager
->expects($this->once())
->expects($this->exactly(2))
->method('get')
->with('subadmin')
->with('UID')
->will($this->returnValue($targetUser));
$this->groupManager
->expects($this->once())
@ -923,8 +973,12 @@ class UsersControllerTest extends TestCase {
->method('isUserAccessible')
->with($loggedInUser, $targetUser)
->will($this->returnValue(false));
$this->groupManager
$subAdminManager
->expects($this->once())
->method('getSubAdminsGroups')
->willReturn([]);
$this->groupManager
->expects($this->exactly(2))
->method('getSubAdmin')
->will($this->returnValue($subAdminManager));
$this->groupManager
@ -943,11 +997,23 @@ class UsersControllerTest extends TestCase {
$targetUser
->expects($this->once())
->method('getEMailAddress')
->will($this->returnValue('subadmin@owncloud.org'));
->will($this->returnValue('subadmin@nextcloud.com'));
$targetUser
->expects($this->exactly(4))
->expects($this->exactly(5))
->method('getUID')
->will($this->returnValue('UID'));
$targetUser
->expects($this->once())
->method('getHome')
->will($this->returnValue('/var/www/newtcloud/data/UID'));
$targetUser
->expects($this->once())
->method('getLastLogin')
->will($this->returnValue(1521191471));
$targetUser
->expects($this->once())
->method('getBackendClassName')
->will($this->returnValue('Database'));
$this->config
->expects($this->at(0))
->method('getUserValue')
@ -966,8 +1032,12 @@ class UsersControllerTest extends TestCase {
$expected = [
'id' => 'UID',
'storageLocation' => '/var/www/newtcloud/data/UID',
'lastLogin' => 1521191471000,
'backend' => 'Database',
'subadmin' => [],
'quota' => ['DummyValue'],
'email' => 'subadmin@owncloud.org',
'email' => 'subadmin@nextcloud.com',
'displayname' => 'Subadmin User',
'phone' => 'phone',
'address' => 'address',
@ -976,7 +1046,7 @@ class UsersControllerTest extends TestCase {
'groups' => [],
'language' => 'ru',
];
$this->assertEquals($expected, $this->invokePrivate($this->api, 'getUserData', ['subadmin']));
$this->assertEquals($expected, $this->invokePrivate($this->api, 'getUserData', ['UID']));
}
public function testEditUserRegularUserSelfEditChangeDisplayName() {
@ -1034,13 +1104,13 @@ class UsersControllerTest extends TestCase {
$targetUser
->expects($this->once())
->method('setEMailAddress')
->with('demo@owncloud.org');
->with('demo@nextcloud.com');
$targetUser
->expects($this->any())
->method('getUID')
->will($this->returnValue('UID'));
$this->assertEquals([], $this->api->editUser('UserToEdit', 'email', 'demo@owncloud.org')->getData());
$this->assertEquals([], $this->api->editUser('UserToEdit', 'email', 'demo@nextcloud.com')->getData());
}
@ -2783,33 +2853,6 @@ class UsersControllerTest extends TestCase {
$this->assertEquals(['TargetGroup'], $this->api->getUserSubAdminGroups('RequestedUser')->getData());
}
/**
* @expectedException \OCP\AppFramework\OCS\OCSException
* @expectedExceptionCode 102
* @expectedExceptionMessage Unknown error occurred
*/
public function testGetUserSubAdminGroupsWithoutGroups() {
$targetUser = $this->getMockBuilder(IUser::class)->disableOriginalConstructor()->getMock();
$this->userManager
->expects($this->once())
->method('get')
->with('RequestedUser')
->will($this->returnValue($targetUser));
$subAdminManager = $this->getMockBuilder('OC\SubAdmin')
->disableOriginalConstructor()->getMock();
$subAdminManager
->expects($this->once())
->method('getSubAdminsGroups')
->with($targetUser)
->will($this->returnValue([]));
$this->groupManager
->expects($this->once())
->method('getSubAdmin')
->will($this->returnValue($subAdminManager));
$this->api->getUserSubAdminGroups('RequestedUser');
}
public function testEnableUser() {
$targetUser = $this->getMockBuilder(IUser::class)->disableOriginalConstructor()->getMock();
$targetUser->expects($this->once())
@ -2896,7 +2939,7 @@ class UsersControllerTest extends TestCase {
'id' => 'UID',
'enabled' => 'true',
'quota' => ['DummyValue'],
'email' => 'demo@owncloud.org',
'email' => 'demo@nextcloud.com',
'displayname' => 'Demo User',
'phone' => 'phone',
'address' => 'address',
@ -2909,7 +2952,7 @@ class UsersControllerTest extends TestCase {
'id' => 'UID',
'enabled' => 'true',
'quota' => ['DummyValue'],
'email' => 'demo@owncloud.org',
'email' => 'demo@nextcloud.com',
'phone' => 'phone',
'address' => 'address',
'website' => 'website',
@ -2956,7 +2999,7 @@ class UsersControllerTest extends TestCase {
'id' => 'UID',
'enabled' => 'true',
'quota' => ['DummyValue'],
'email' => 'demo@owncloud.org',
'email' => 'demo@nextcloud.com',
'phone' => 'phone',
'address' => 'address',
'website' => 'website',