Rozwiązanie #268
Rozwiązanie #268
Conversation
| @@ -0,0 +1,9 @@ | |||
| class CreateCustomers < ActiveRecord::Migration[5.2] | |||
| def change | |||
| create_table :customers do |t| | |||
| @@ -0,0 +1,5 @@ | |||
| module CustomersHelper | |||
| def customers_not_empty? | |||
| @customers && !@customers.empty? | |||
There was a problem hiding this comment.
zamiast !@customers.empty? można napisać @customers.any?
There was a problem hiding this comment.
Właśnie tej metody mi brakowało, dzięki! 👍
| @@ -0,0 +1,3 @@ | |||
| %h1 #{link_to @customer.email, customer_path(@customer)} | |||
| - if customers_not_empty? | |||
There was a problem hiding this comment.
customers_not_empty? w tym miejscu zawsze zwraca nil przez co produkty nigdy nie są wyświetlane 😢
| private | ||
|
|
||
| def products_filtered | ||
| products.where('price > ?', price) |
There was a problem hiding this comment.
tę metodę można przenieśc do modelu jako scope. Dodatkowo w zadaniu była mowa o tym żeby wyświetlić produkty TAŃSZE niż podane price 😉 + co się stanie jeśli params[:price] w ogóle nie będzie podane?
zad6/app/helpers/products_helper.rb
Outdated
|
|
||
| def read_categories product | ||
| result = []; | ||
| product.category_ids.each {|category_id| result << read_category(category_id)} |
There was a problem hiding this comment.
Przy odpowiednio napisanych asocjacjach(a są 👍 ) wystarczy:
product.categories.pluck(:name).join(', ')There was a problem hiding this comment.
Racja nie przemyślałem tego dobrze :/
zad6/app/helpers/products_helper.rb
Outdated
| end | ||
|
|
||
| def read_customer product | ||
| Customer.find(product.customer_id) if has_customer? |
There was a problem hiding this comment.
Tak samo tutaj, asocjacje ułatwiają sprawę i można napisać po prostu product.customer
No description provided.