@membership = Member.edit_membership(params[:membership_id], params[:membership], @group)
@membership.save if request.post?
respond_to do |format|
- format.html { redirect_to :controller => 'groups', :action => 'edit', :id => @group, :tab => 'memberships' }
- format.js {
- render(:update) {|page|
- page.replace_html "tab-content-memberships", :partial => 'groups/memberships'
- page.visual_effect(:highlight, "member-#{@membership.id}")
- }
- }
- end
+ if @membership.valid?
+ format.html { redirect_to :controller => 'groups', :action => 'edit', :id => @group, :tab => 'memberships' }
+ format.js {
+ render(:update) {|page|
+ page.replace_html "tab-content-memberships", :partial => 'groups/memberships'
+ page.visual_effect(:highlight, "member-#{@membership.id}")
+ }
+ }
+ else
+ format.js {
+ render(:update) {|page|
+ page.alert(l(:notice_failed_to_save_members, :errors => @membership.errors.full_messages.join(', ')))
+ }
+ }
+ end
+ end
end
def destroy_membership
@membership = Member.edit_membership(params[:membership_id], params[:membership], @user)
@membership.save if request.post?
respond_to do |format|
- format.html { redirect_to :controller => 'users', :action => 'edit', :id => @user, :tab => 'memberships' }
- format.js {
- render(:update) {|page|
- page.replace_html "tab-content-memberships", :partial => 'users/memberships'
- page.visual_effect(:highlight, "member-#{@membership.id}")
- }
- }
- end
+ if @membership.valid?
+ format.html { redirect_to :controller => 'users', :action => 'edit', :id => @user, :tab => 'memberships' }
+ format.js {
+ render(:update) {|page|
+ page.replace_html "tab-content-memberships", :partial => 'users/memberships'
+ page.visual_effect(:highlight, "member-#{@membership.id}")
+ }
+ }
+ else
+ format.js {
+ render(:update) {|page|
+ page.alert(l(:notice_failed_to_save_members, :errors => @membership.errors.full_messages.join(', ')))
+ }
+ }
+ end
+ end
end
def destroy_membership