diff options
| author | David Pursehouse <dpursehouse@collab.net> | 2019-03-12 07:05:58 +0000 |
|---|---|---|
| committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2019-03-12 07:05:58 +0000 |
| commit | 55e5067038822522e98e7e2c2512fd5f4808fcdb (patch) | |
| tree | d66e16c5b937ad93fa742fadc1bba85fa5b5127c | |
| parent | 39f960842501dfe0f66795fb2c56425f33a3c12b (diff) | |
| parent | b7b5cf0067ffa6a4c54c7791e4ee67b5146f9214 (diff) | |
Merge "Fix setting groups visibility" into stable-2.16
| -rw-r--r-- | polygerrit-ui/app/elements/admin/gr-group/gr-group.js | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/polygerrit-ui/app/elements/admin/gr-group/gr-group.js b/polygerrit-ui/app/elements/admin/gr-group/gr-group.js index c66d86057a..e2ecfbac35 100644 --- a/polygerrit-ui/app/elements/admin/gr-group/gr-group.js +++ b/polygerrit-ui/app/elements/admin/gr-group/gr-group.js @@ -180,13 +180,10 @@ }, _handleSaveOptions() { - let options; - // The value is in string so we have to convert it to a boolean. - if (this._groupConfig.options.visible_to_all) { - options = {visible_to_all: true}; - } else if (!this._groupConfig.options.visible_to_all) { - options = {visible_to_all: false}; - } + const visible = this._groupConfig.options.visible_to_all; + + const options = {visible_to_all: visible}; + return this.$.restAPI.saveGroupOptions(this.groupId, options).then(config => { this._options = false; |