diff --git a/app/controllers/admin/environment_role_manager_controller.rb b/app/controllers/admin/environment_role_manager_controller.rb index 6de4887..fef08a8 100644 --- a/app/controllers/admin/environment_role_manager_controller.rb +++ b/app/controllers/admin/environment_role_manager_controller.rb @@ -1,6 +1,6 @@ class EnvironmentRoleManagerController < AdminController protect 'manage_environment_roles', :environment - + def index @admins = Person.find(:all, :conditions => ['role_assignments.resource_type = ?', 'Environment'], :include => :role_assignments ) end @@ -8,7 +8,7 @@ class EnvironmentRoleManagerController < AdminController def change_roles @admin = Person.find(params[:id]) @roles = Role.find(:all).select{ |r| r.has_kind?(:environment) } - end + end def update_roles @roles = params[:roles] ? Role.find(params[:roles]) : [] @@ -20,7 +20,7 @@ class EnvironmentRoleManagerController < AdminController end redirect_to :action => :index end - + def change_role @roles = Role.find(:all).select{ |r| r.has_kind?(:environment) } @admin = Person.find(params[:id]) diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 469a9ac..9524548 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -805,8 +805,7 @@ module ApplicationHelper fields_for(name, object, { :builder => NoosferoFormBuilder }.merge(options), &proc) end - def labelled_form_for(name, object = nil, options = {}, &proc) - # FIXME remove the =object= argument and adapt the calling code + def labelled_form_for(name, options = {}, &proc) form_for(name, { :builder => NoosferoFormBuilder }.merge(options), &proc) end diff --git a/app/views/account/_login_form.html.erb b/app/views/account/_login_form.html.erb index f58f2fd..a145e92 100644 --- a/app/views/account/_login_form.html.erb +++ b/app/views/account/_login_form.html.erb @@ -1,4 +1,4 @@ -<%= labelled_form_for :user, @user, +<%= labelled_form_for :user, :url => { :controller => 'account', :action => (params[:enterprise_code] ? 'activate_enterprise' : 'login') } do |f| %> <%= f.text_field :login, diff --git a/app/views/account/_signup_form.html.erb b/app/views/account/_signup_form.html.erb index 58d51dc..2b6af52 100644 --- a/app/views/account/_signup_form.html.erb +++ b/app/views/account/_signup_form.html.erb @@ -11,7 +11,7 @@ <%= error_messages_for :user, :person, :header_message => _('The account could not be created') %> -<%= labelled_form_for :user, @user, :html => { :multipart => true, :id => 'signup-form', :honeypot => true } do |f| %> +<%= labelled_form_for :user, :html => { :multipart => true, :id => 'signup-form', :honeypot => true } do |f| %>