[opensuse-buildservice] Re: [PATCH] [api] Fix active_rbac/user/delete.rhtml to post to destroy method. Otherwise validation is broken in controllers/active_rbac/user_controller.rb and we can't delete users at all in the admin interface.

Hi, On 2010-11-30 00:05:48 +0100, OBS build-service wrote:

From: Jan-Simon Möller

Signed-off-by: Jan-Simon Möller --- src/api/app/views/active_rbac/user/delete.rhtml | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/api/app/views/active_rbac/user/delete.rhtml b/src/api/app/views/active_rbac/user/delete.rhtml index 90025b2..df89d43 100644 --- a/src/api/app/views/active_rbac/user/delete.rhtml +++ b/src/api/app/views/active_rbac/user/delete.rhtml @@ -7,7 +7,7 @@ depending on this user (articles, for example). <strong>This action cannot be reverted!</strong> </p>

-<% form_for :group, :url => { :action => :delete, :id => @user } do |form| %> +<% form_for :group, :url => { :action => :destroy, :id => @user } do |form| %> <%= submit_tag 'Yes', :name => 'yes' %> <%= submit_tag 'No', :name => 'no' %> <% end %> \ No newline at end of file Hmm I'm not sure if it's a good idea to "enable/fix" the user deletion. It was already discussed some time ago (see http://lists.opensuse.org/opensuse-buildservice/2010-08/msg00057.html).

Marcus -- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org