wbrawner/server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use RESPOND_NOT_FOUND for non existing user

Browse source 
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
This commit is contained in:
Morris Jobke 2017-04-21 11:30:00 -05:00
parent 9a0280f1c8
commit 4dd72e9eb9
No known key found for this signature in database
GPG key ID: 9CE5ED29E7FCD38A
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
apps/provisioning_api/lib/Controller/UsersController.php
View file

@ -790,7 +790,7 @@ class UsersController extends OCSController {
$targetUser = $this->userManager->get($userId);
if($targetUser === null) {
throw new OCSException('', \OCP\API::RESPOND_UNAUTHORISED);
throw new OCSException('', \OCP\API::RESPOND_NOT_FOUND);
}
// Check if admin / subadmin

2
apps/provisioning_api/tests/Controller/UsersControllerTest.php
View file

@ -2707,7 +2707,7 @@ class UsersControllerTest extends TestCase {
/**
* @expectedException \OCP\AppFramework\OCS\OCSException
* @expectedExceptionCode 997
* @expectedExceptionCode 998
*/
public function testResendWelcomeMessageWithNotExistingTargetUser() {
$this->userManager