Filtering nested attributes in rails 3

Question

My application is composed of Category that has_many products. Category as well as products have a publication date.

The show page of category display the product from that category.

The show and index actions of Category_controller are very simple (published is a scope):

def index
    @categories = Category.published
end

def show
    @categories = Category.find(params[:id])
    unless @categories.datepublication <= Time.now
        redirect_to categories_path
    end
end

As you can see it display published category and redirect you to category index when you are being too curious for your own sake.

Now it would be great to display only published products in the show view of a category. I do not know how to do that except with a if in the view, but it seems to me that filtering elements is not the job of the view. What is the correct MVC way of filtering a nested element ?

Peter Duijnstee · Accepted Answer · 2012-10-02 15:45:29Z

1

You can chain scopes, even through associations. From your example I gather that your categories have a publication date but your products also have their own publication date, is that right?

In that case you could set up a scope on your products similar to the one you have on your categories and just use it the same way, i.e.:

@products = @category.products.published

Then whether you want to preload the @products instance variable in the controller or just use @category.products.published in the view is up to you. If the application is simple enough I usually just use the scoped query in the view myself.

answered Oct 2, 2012 at 15:45

Peter Duijnstee

3,7792 gold badges23 silver badges31 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Syl Over a year ago

Is there a way to keep a stupid generic view where i loop around @category.products and not around @products? With products being filtered of course. My real app use quite a lot of partials.

Peter Duijnstee Over a year ago

Yep, that's fine. Just use the scoped method in your view, so for example: <%= @category.products.published do |product| %><%= product.name %><% end -%>

rewritten · Accepted Answer · 2012-10-02 15:45:43Z

1

This is the job of the ORM (also for categories)

 begin
   @category = Category.published.find(params[:id])
 rescue ActiveRecord::RecordNotFound
   redirect_to categories_path unless @category
 end

 @products = @category.products.published

answered Oct 2, 2012 at 15:45

rewritten

16.5k2 gold badges49 silver badges50 bronze badges

Collectives™ on Stack Overflow

Filtering nested attributes in rails 3

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related