Solve zad6 #265
Solve zad6 #265
Conversation
| end | ||
| def show | ||
| @customer = Customer.find(params[:id]) | ||
| @products = @customer.products.order(price: :desc) |
There was a problem hiding this comment.
order(price: :desc) można przerzucić do modelu jako scope, zgodnie z ostatnim wykładem 😉
| class Product < ApplicationRecord | ||
| has_many :product_categories | ||
| has_many :categories, through: :product_categories | ||
| belongs_to :customer |
There was a problem hiding this comment.
brak opcji optional: true sprawia, że nie można stworzyć produktu, który nie jest jeszcze przypisany do żadnego customera 😢
| def index | ||
| @customers = Customer.all | ||
| end | ||
| def show |
There was a problem hiding this comment.
Zgodnie z zadanie, produkty należące do customera powinny być wyświetlane pod adresem /customers/:customer_id/products. Teraz jest to /customers/:customer_id. Zauważ, że ten show jest trochę mylący. Oczekiwalibysmy, że dostaniemy tutaj profil pojedynczego customera a dostajemy produkty 🤔 Żeby to poprawić należałoby:
- zanestować odpowiednio ścieżki
# config/routes.rb
resources :customers, only: :index do
resources :products, only: :index
endTaki zapis rzeczywiście tworzy ścieżkę /customers/:customer_id/products, która odpowiada akcji index w kontrolerze ProductsController, czyli wszystko jest na swoim miejscu - CustomersController zajmuje się customerami a ProductsController produktami. 😉
| </tr> | ||
| </thead> | ||
| <tbody> | ||
| <% @products.each do |product| %> |
There was a problem hiding this comment.
Wyświetlanie pojedynczego produktu nadaje się na oddzielny plik, z którego można wielokrotnie skorzystać 😉
| @@ -1,3 +1,5 @@ | |||
| Rails.application.routes.draw do | |||
| # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html | |||
| resources :customers | |||
| resources :products | |||
There was a problem hiding this comment.
tak jak pisałem wyżej, żeby otrzymać odpowiednie ścieżki trzeba użyć:
resources :customers do
resources :products
endZauważ też, że niesprecyzowanie o jakie dokładnie akcje nam chodzi(resources :customers, only: :index) powoduje powstanie całej masy niepotrzebnych i niezaimplementowanych routes:
No description provided.