Merge pull request #12445 from birarda/feat/content-settings

handle permissions save with no existing permissions
This commit is contained in:
Stephen Birarda 2018-02-21 14:55:37 -07:00 committed by GitHub
commit 6e6f52236e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -396,10 +396,12 @@ function savePermissions() {
var admins = $('#admin-usernames').val().split(',');
var existingAdmins = Settings.data.values.security.permissions.map(function(value) {
return value.permissions_id;
});
admins = admins.concat(existingAdmins);
if (Settings.data.values.security.permissions) {
var existingAdmins = Settings.data.values.security.permissions.map(function(value) {
return value.permissions_id;
});
admins = admins.concat(existingAdmins);
}
// Filter out unique values
admins = _.uniq(admins.map(function(username) {