Попытка просмотра коллекции продуктов, созданной пользователем, и ссылки/отображения в отдельном файле вида

Question

Мои пользователи создают коллекции и добавляют списки в каждую коллекцию. У меня есть главная страница профиля для каждого пользователя, показывающая все коллекции, созданные этим пользователем. Теперь я хочу связать каждую коллекцию с отдельной страницей, содержащей все списки в этой коллекции.

Мой файл представления пользователю показать все коллекции:

<% @collections.each do |collection| %> 
    <%= image_tag listing.image.url(:thumb) %> 
    <%= link_to "#{collection.name}", shopcollected_path(collection_id: @collection) %> 
<% end %> 

Когда я нажимаю на моей ссылке я доставлен в отдельную страницу коллекции (shopcollected.html.erb) просто отлично, но результаты для этого коллекция не распознается, а страница не содержит списков. Я знаю, что я пропустил одну небольшую часть, но снова застрял на этом. Я все еще изучаю извлечение объектов из баз данных и привязки.

Мой listings_controller:

def shopcollections 
    @user = User.find(params[:id]) 
    @collections = Collection.where(user: User.find(params[:id])).order("created_at DESC") 
end 

def shopcollected 
    @user = User.find(params[:id]) 
    @listings = Listing.where(collection: :collection_id) 
end 

Мой индивидуальный пользователь 'shopcollected' Вид файла:

<% @listings.each do |listing| %> 
    <%= image_tag listing.image.url(:medium) %> 
    <%= listing.name %>` 
<% end %> 

Что мне не хватает? Я считаю, что мне нужно вызвать все списки по коллекции, но для этого, где мне нужно внести изменения в свой код?

Answer 1

Так как мы обсуждали в комментариях, были две проблемы, от которых страдает приложение:

В представлении пользователя, когда вы были итерация коллекции @collections, вы использовали неправильную переменную внутри цикла для обозначения к текущему объекту Collection (в частности, @collection вместо collection). Должно выглядеть так:

<% @collections.each do |collection| %> 
    <%= image_tag listing.image.url(:thumb) %> 
    <%= link_to "#{collection.name}", shopcollected_path(collection_id: collection) %> 
<% end %> 

Другой вопрос в методе shopcollected контроллера. Вы просто забыли указать часть params при попытке получить доступ к значению :collection_id из формы. Он должен выглядеть следующим образом:

def shopcollected 
    @user = User.find(params[:id]) 
    @listings = Listing.where(collection: params[:collection_id]) 
end 

Одно незначительное неродственного предложение, которое не будет иметь никакого влияния на производительность, но будет немного очистить свой код и сделать его более Rails-у смотреть; в случае shopcollections, если вы создали соответствующие belongs_to и has_many ассоциаций в моделях, есть немного короче способ отрываясь коллекции для конкретного пользователя, например, так:

def shopcollections 
    @user = User.find(params[:id]) 
    @collections = @user.collections.order("created_at DESC") 
end 

Это будет имеют такое же влияние, что и на предыдущую версию, поскольку Rails загрузит все коллекции для , которые пользователь, потому что вы вызываете метод отношений has_many. Поскольку большинство активных методов записи возвращают прокси-сервер, вы можете дополнительно вызвать другие методы, такие как order, которые теперь будут влиять на фильтрацию и упорядочение вычисленной коллекции has_many (в данном случае объекты Collection). Надеюсь, что это имеет смысл,