Skip to content

Commit ea40345

Browse files
committed
makes users management page aware of the introduced options
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
1 parent 457d746 commit ea40345

File tree

7 files changed

+14
-9
lines changed

7 files changed

+14
-9
lines changed

settings/Controller/UsersController.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -246,6 +246,8 @@ public function usersList() {
246246
// Settings
247247
$serverData['defaultQuota'] = $defaultQuota;
248248
$serverData['canChangePassword'] = $canChangePassword;
249+
$serverData['newUserGenerateUserID'] = (bool)$this->config->getAppValue('settings', 'newUser.generateUserID', false);
250+
$serverData['newUserRequireEmail'] = (bool)$this->config->getAppValue('settings', 'newUser.requireEmail', false);
249251

250252
return new TemplateResponse('settings', 'settings-vue', ['serverData' => $serverData]);
251253
}

settings/js/vue-6.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

settings/js/vue-6.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

settings/js/vue-settings-apps-users-management.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

settings/js/vue-settings-apps-users-management.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

settings/src/components/userList.vue

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,12 @@
4949
<div :class="loading.all?'icon-loading-small':'icon-add'"></div>
5050
<div class="name">
5151
<input id="newusername" type="text" required v-model="newUser.id"
52-
:placeholder="t('settings', 'Username')" name="username"
53-
autocomplete="off" autocapitalize="none" autocorrect="off"
54-
ref="newusername" pattern="[a-zA-Z0-9 _\.@\-']+">
52+
:placeholder="this.settings.newUserGenerateUserID
53+
? t('settings', 'Will be autogenerated')
54+
: t('settings', 'Username')"
55+
name="username" autocomplete="off" autocapitalize="none"
56+
autocorrect="off" ref="newusername" pattern="[a-zA-Z0-9 _\.@\-']+"
57+
:disabled="this.settings.newUserGenerateUserID">
5558
</div>
5659
<div class="displayName">
5760
<input id="newdisplayname" type="text" v-model="newUser.displayName"
@@ -67,7 +70,7 @@
6770
</div>
6871
<div class="mailAddress">
6972
<input id="newemail" type="email" v-model="newUser.mailAddress"
70-
:required="newUser.password===''"
73+
:required="newUser.password==='' || this.settings.newUserRequireEmail"
7174
:placeholder="t('settings', 'Email')" name="email"
7275
autocomplete="off" autocapitalize="none" autocorrect="off">
7376
</div>

settings/src/store/users.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -429,7 +429,7 @@ const actions = {
429429
addUser({commit, dispatch}, { userid, password, displayName, email, groups, subadmin, quota, language }) {
430430
return api.requireAdmin().then((response) => {
431431
return api.post(OC.linkToOCS(`cloud/users`, 2), { userid, password, displayName, email, groups, subadmin, quota, language })
432-
.then((response) => dispatch('addUserData', userid))
432+
.then((response) => dispatch('addUserData', userid || response.data.ocs.data.UserID))
433433
.catch((error) => {throw error;});
434434
}).catch((error) => {
435435
commit('API_FAILURE', { userid, error });

0 commit comments

Comments
 (0)