Rails 4 scope не работает

Question

В приложении Rails 4 область не работает для меня.

Модель:

class Todo < ActiveRecord::Base 
    ... 

    scope :completed, -> { where(completed: true) } 
    scope :notcompleted, -> { where(completed: false) } 
end 

Маршруты:

get 'todos/completed' 

Контроллер:

class TodosController < ApplicationController 
    before_action :set_todo, only: [:show, :edit, :update, :destroy] 

def completed 
    @todos = current_user.todos.completed 
end 
... 
def set_todo 
    @todo = Todo.find(params[:id]) 
end 

Вид:

... 
<% @todos.each do |todo| %> 
... 

Я получаю:

Couldn't find Todo with 'id'=completed 
in the set_todo logic 

Я не понимаю, почему это ударяет set_todo логику. Он работает set_todo, даже если я использую что-то вроде before_action :set_todo, except: [:index, :create, :completed]

Как-то, код set_todo не должен выполняться!

Answer 1

#config/routes.rb 
resources :todos do 
    get :completed, on: :collection #-> url.com/todos/completed 
end 

---

Разъяснение

Ваши старые маршруты, вероятно, выглядели так:

#config/routes.rb 
resources :todos 
get "todos/complete" 

Когда вы отправляете запрос в Rails, он просматривает маршруты и соответствует той, которую вы запрашиваете.

С resources :todos вы получаете маршрут /todos/:id.

Это означает, что в любое время, когда вы запрашиваете адрес url.com/todos/x, Rails будет ссылаться на действие todos#show, которое заполнено set_todo.

Вот почему вы получаете ошибку Couldn't find Todo with 'id'=completed. Rails отправляет ваш запрос на шоу.

Что вам нужно либо определить get 'todos/complete'вышеresources :todos:

#config/routes.rb 
get "todos/complete" 
resources :todos 

... ИЛИ (рекомендуется), create an extra route:

#config/routes.rb 
resources :todos do 
    get :completed, on: :collection 
end 

---

Как и в сторону, ваши области может быть очищены немного:

#app/models/todo.rb 
class Todo < ActiveRecord::Base 
    scope :completed, ->(trigger = true) { where completed: trigger } 
end 

Это позволит использовать единую completed области с прошедшим аргументом:

@todos.completed 
@todos.completed(false) 

Answer 2

Я не уверен, что области работают над сборником, созданным current_user.todos. Как видно из the docs, область действия - метод класса, который действует на модель.

Если вы отмените запрос, все должно быть хорошо.

Todo.completed.where(user_id: current_user.id)

Answer 3

метод использования класса

def self.completed 
    where(completed: true) 
end 

http://api.rubyonrails.org/classes/ActiveRecord/Scoping/Named/ClassMethods.html

Answer 4

Попробуйте эту область:

scope :completed, proc{ |todo| where('completed = ?', true) } 

Answer 5

Ошибка вы получаете нет в объеме, но и в ваша логика .find. Ваши set_todo работает Todo.find(params[:id]). Нет никаких причин для этого для действия контроллера completed, так как он имеет всю необходимую информацию без использования хэша params (пользовательский ввод не требуется, чтобы узнать, какие записи завершены).

Другая проблема заключается в том, что ваш params[:id], вероятно, вообще не должен заселяться в этом случае, поскольку он выглядит /todos/completed выглядит как маршрут сбора.

Вместо того, чтобы get todos/complete, я бы гнездо под вашим todo ресурс:

resources :todos do 
    collection do 
    get 'completed' 
    end 
end 

Или явно продиктовать отображение маршрута

get '/todos/completed', to: 'todos#completed' 

Как это, похоже get todos/completed обдувает неправильно часть таблицы маршрутизации.

Наконец, вы можете просто создать «completed_todos» ассоциации по User, которая будет двигаться логику в модели (где я бы утверждать, что это принадлежит)

class User < ActiveRecord::Base 

    has_many :todos 
    has_many :completed_todos, -> { where(completed: true) } 
end