codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-11-24 08:09:13 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #1250 from sir-pinecone/display-group-save-errors

Browse source 
display group validation errors in alert modal
This commit is contained in:
Sam 2013-07-23 22:39:23 -07:00
commit a09c9f08a8
2 changed files with 15 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
app/assets/javascripts/admin/models/group.js
View file

@ -60,13 +60,20 @@ Discourse.Group = Discourse.Model.extend({
name: this.get('name'), name: this.get('name'),
usernames: this.get('usernames') usernames: this.get('usernames')
} }
}}).then(function(r){ }}).then(function(resp) {
group.set('disableSave', false); group.set('disableSave', false);
group.set('id', r.id); group.set('id', resp.id);
}, function (error) {
group.set('disableSave', false);
if (error && error.responseText) {
bootbox.alert($.parseJSON(error.responseText).errors);
}
else {
bootbox.alert(I18n.t('generic_error'));
}
}); });
}, },
save: function(){ save: function(){
var group = this; var group = this;
group.set('disableSave', true); group.set('disableSave', true);

7
app/controllers/admin/groups_controller.rb
View file

@ -33,8 +33,11 @@ class Admin::GroupsController < Admin::AdminController
group = Group.new group = Group.new
group.name = params[:group][:name] group.name = params[:group][:name]
group.usernames = params[:group][:usernames] if params[:group][:usernames] group.usernames = params[:group][:usernames] if params[:group][:usernames]
group.save! if group.save
render_serialized(group, BasicGroupSerializer) render_serialized(group, BasicGroupSerializer)
else
render_json_error group
end
end end
def destroy def destroy