Merge branch 'stable-3.0' into stable-3.1
* stable-3.0:
Revert "Check if full name and email edit are allowed in realm"
Change-Id: Iea5e80134a0a92066da647d080c7e0a31f6d5eb4
diff --git a/src/main/resources/static/gr-serviceuser-detail.html b/src/main/resources/static/gr-serviceuser-detail.html
index e515ee6..41e4e8d 100644
--- a/src/main/resources/static/gr-serviceuser-detail.html
+++ b/src/main/resources/static/gr-serviceuser-detail.html
@@ -101,7 +101,6 @@
on-keyup="_computePrefsChanged">
</iron-input>
</span>
- <span class="value" hidden$="[[_allowFullName]]">[[_serviceUser.name]]</span>
</section>
<section>
<span class="title">Email Address</span>
diff --git a/src/main/resources/static/gr-serviceuser-detail.js b/src/main/resources/static/gr-serviceuser-detail.js
index 53ec980..95aded2 100644
--- a/src/main/resources/static/gr-serviceuser-detail.js
+++ b/src/main/resources/static/gr-serviceuser-detail.js
@@ -25,7 +25,6 @@
_restApi: Object,
_serviceUserId: String,
_serviceUser: Object,
- _serverConfig: Object,
_loading: {
type: Boolean,
value: true,
@@ -50,10 +49,6 @@
type: Boolean,
value: false,
},
- _allowFullName: {
- type: Boolean,
- value: false,
- },
_allowOwner: {
type: Boolean,
value: false,
@@ -80,10 +75,8 @@
],
attached() {
- this._getPermissions().then(() => {
- this._extractUserId();
- this._loadServiceUser();
- });
+ this._extractUserId();
+ this._loadServiceUser();
},
_loadServiceUser() {
@@ -93,7 +86,6 @@
promises.push(this._getPluginConfig());
promises.push(this._getServiceUser());
- promises.push(this._getServerConfig());
Promise.all(promises).then(() => {
this.$.sshEditor.loadData(this._restApi, this._serviceUser);
@@ -104,11 +96,6 @@
this._loading = false;
this._newFullName = this._serviceUser.name;
this._newEmail = this._serviceUser.email;
- this._allowFullName = this._serverConfig.auth.editable_account_fields
- .includes('FULL_NAME');
- this._allowEmail = this._allowEmail &&
- this._serverConfig.auth.editable_account_fields
- .includes('REGISTER_NEW_EMAIL');
});
},
@@ -128,25 +115,17 @@
},
_getPluginConfig() {
- return this.plugin.restApi('/config/server/serviceuser~config/').get('')
- .then(config => {
- if (!config) {
- return;
- }
- this._allowEmail = config.allow_email || this._isAdmin;
- this._allowOwner = config.allow_owner || this._isAdmin;
- this._allowHttpPassword = config.allow_http_password
- || this._isAdmin;
- });
- },
-
- _getServerConfig() {
- return this.plugin.restApi().getConfig().then(cfg => {
- if (!cfg) {
- return;
- }
-
- this._serverConfig = cfg;
+ return Promise.resolve(this._getPermissions()).then(() => {
+ this.plugin.restApi('/config/server/serviceuser~config/').get('')
+ .then(config => {
+ if (!config) {
+ return;
+ }
+ this._allowEmail = config.allow_email || this._isAdmin;
+ this._allowOwner = config.allow_owner || this._isAdmin;
+ this._allowHttpPassword = config.allow_http_password
+ || this._isAdmin;
+ });
});
},