end
if @user.update_attributes(params[:user])
flash[:notice] = l(:notice_successful_update)
- redirect_to :action => 'list'
+ # Give a string to redirect_to otherwise it would use status param as the response code
+ redirect_to(url_for(:action => 'list', :status => params[:status], :page => params[:page]))
end
end
@auth_sources = AuthSource.find(:all)
[l(:status_registered), 2],
[l(:status_locked), 3]], selected)
end
+
+ def change_status_link(user)
+ url = {:action => 'edit', :id => user, :page => params[:page], :status => params[:status]}
+
+ if user.locked?
+ link_to l(:button_unlock), url.merge(:user => {:status => User::STATUS_ACTIVE}), :method => :post, :class => 'icon icon-unlock'
+ elsif user.registered?
+ link_to l(:button_activate), url.merge(:user => {:status => User::STATUS_ACTIVE}), :method => :post, :class => 'icon icon-unlock'
+ else
+ link_to l(:button_lock), url.merge(:user => {:status => User::STATUS_LOCKED}), :method => :post, :class => 'icon icon-lock'
+ end
+ end
end
<td align="center"><%= image_tag('true.png') if user.admin? %></td>
<td class="created_on" align="center"><%= format_time(user.created_on) %></td>
<td class="last_login_on" align="center"><%= format_time(user.last_login_on) unless user.last_login_on.nil? %></td>
- <td>
- <small>
- <% if user.locked? -%>
- <%= link_to l(:button_unlock), {:action => 'edit', :id => user, :user => {:status => User::STATUS_ACTIVE}}, :method => :post, :class => 'icon icon-unlock' %>
- <% elsif user.registered? -%>
- <%= link_to l(:button_activate), {:action => 'edit', :id => user, :user => {:status => User::STATUS_ACTIVE}}, :method => :post, :class => 'icon icon-unlock' %>
- <% else -%>
- <%= link_to l(:button_lock), {:action => 'edit', :id => user, :user => {:status => User::STATUS_LOCKED}}, :method => :post, :class => 'icon icon-lock' %>
- <% end -%>
- </small>
- </td>
+ <td><small><%= change_status_link(user) %></small></td>
</tr>
<% end -%>
</tbody>