Merge pull request #8502 from nextcloud/properly-show-hints-in-ocs
Show hint in OCS API for user creation
This commit is contained in:
commit
98baf8915d
|
@ -31,6 +31,7 @@
|
||||||
namespace OCA\Provisioning_API\Controller;
|
namespace OCA\Provisioning_API\Controller;
|
||||||
|
|
||||||
use OC\Accounts\AccountManager;
|
use OC\Accounts\AccountManager;
|
||||||
|
use OC\HintException;
|
||||||
use OC\Settings\Mailer\NewUserMailHelper;
|
use OC\Settings\Mailer\NewUserMailHelper;
|
||||||
use OC_Helper;
|
use OC_Helper;
|
||||||
use OCA\Provisioning_API\FederatedFileSharingFactory;
|
use OCA\Provisioning_API\FederatedFileSharingFactory;
|
||||||
|
@ -193,15 +194,22 @@ class UsersController extends OCSController {
|
||||||
|
|
||||||
try {
|
try {
|
||||||
$newUser = $this->userManager->createUser($userid, $password);
|
$newUser = $this->userManager->createUser($userid, $password);
|
||||||
$this->logger->info('Successful addUser call with userid: '.$userid, ['app' => 'ocs_api']);
|
$this->logger->info('Successful addUser call with userid: ' . $userid, ['app' => 'ocs_api']);
|
||||||
|
|
||||||
if (is_array($groups)) {
|
if (is_array($groups)) {
|
||||||
foreach ($groups as $group) {
|
foreach ($groups as $group) {
|
||||||
$this->groupManager->get($group)->addUser($newUser);
|
$this->groupManager->get($group)->addUser($newUser);
|
||||||
$this->logger->info('Added userid '.$userid.' to group '.$group, ['app' => 'ocs_api']);
|
$this->logger->info('Added userid ' . $userid . ' to group ' . $group, ['app' => 'ocs_api']);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return new DataResponse();
|
return new DataResponse();
|
||||||
|
} catch (HintException $e ) {
|
||||||
|
$this->logger->logException($e, [
|
||||||
|
'message' => 'Failed addUser attempt with hint exception.',
|
||||||
|
'level' => \OCP\Util::WARN,
|
||||||
|
'app' => 'ocs_api',
|
||||||
|
]);
|
||||||
|
throw new OCSException($e->getHint(), 107);
|
||||||
} catch (\Exception $e) {
|
} catch (\Exception $e) {
|
||||||
$this->logger->logException($e, [
|
$this->logger->logException($e, [
|
||||||
'message' => 'Failed addUser attempt with exception.',
|
'message' => 'Failed addUser attempt with exception.',
|
||||||
|
|
Loading…
Reference in New Issue