2016-07-21 6 views
0

Ich richte eine App (Rails 5) ein, in der Benutzer von einem Admin genehmigt werden müssen. Ich habe die Anweisungen unter https://github.com/plataformatec/devise/wiki/How-To%3a-Require-admin-to-activate-account-before-sign_in bei der Einrichtung von alles befolgt, aber ich bekomme immer eine NoMethodError undefined Methode für 'nil: NilClass, wenn ich versuche, @users zu durchlaufen. Hier ist mein users_controller.rbDevise für Ruby on Rails: @users ist Null, wenn eine Admin-Genehmigung erforderlich ist

def index 
    if params[:approved] == "false" 
    @users = User.find_by_approved(false) 
    else 
    @users = User.all 
    end 
end 

Benutzermodell user.rb

class User < ApplicationRecord 
    # Include default devise modules. Others available are: 
    # :confirmable, :lockable, :timeoutable and :omniauthable 
    devise :database_authenticatable, :registerable, 
    :recoverable, :rememberable, :trackable, :validatable, :confirmable 

    def active_for_authentication? 
    super && approved? 
    end 

    def inactive_message 
    if !approved? 
     :not_approved 
    else 
     super 
    end 
    end 
end 

routes.rb

Rails.application.routes.draw do 
    devise_for :admins 
    devise_for :users, except: [:index] 
    resources :users, only: [:index] 

    get 'home/index' 

    get 'home/about' 

    get 'home/products' 

    get 'home/agents' 

    get 'home/contact' 

    get 'home/documents' 

    get 'users/admin_page' 

    root 'home#index' 

end 

und der Tisch aus meiner Sicht auf Benutzer/admin_page.html.erb

<%= link_to "All Users", :action => "index" %>| 
<%= link_to "Users awaiting approval", :action => "index", :approved => "false" %> 
<table> 
    <% @users.each do |user| %> 
    <tr> 
     <td><%= user.email %></td> 
     <td><%= user.approved %></td> 
     <td><%= link_to "Edit", edit_user_path(user) %></td> 
    </tr> 
    <% end %> 
</table> 

Ich habe mich umgesehen andere Lösungen oder Gründe warum @users nicht definiert wäre, aber ich kann nichts finden.

+0

Versuchen '@users = User.find_by_approved (false)' auf '@users = User.where (genehmigt: false) ändert' und sehen, ob es – mrvncaragay

+0

Ihr Problem behebt Ich habe gerade versucht, dass und es zeigt immer noch als Null auf. Ich habe auch versucht, diesen Teil komplett zu entfernen und nur als @ users = User.all zu definieren, und das hat sich auch als undefiniert gezeigt. –

Antwort

0

Die Methoden find_by_ sollen ein einzelnes Objekt zurückgeben, oder nil, falls keines existiert. Sie möchten die gesamte Sammlung von Benutzern, die noch nicht genehmigt wurden.

@users = User.where(approved: false) 
0

Ich fand es heraus. Ich habe meine Methode in meinem UsersController falsch benannt. Meine Ansicht heißt admin_page.rb, aber ich habe meinen Methodenindex genannt. Danke an alle, die geantwortet haben.