How to handle additional param in controller

Question

I'm learning Rails and I have now a good knowledge about controllers. By the way I always have some issues and I don't know whats the best way to solve them.

One of these is the search: I have a search in my website and I must reorder results for relevance and date.

My search controller

def show
    @query = params[:query]
    @contents  = Content.published.search_by_text(@query).page(params[:page]).per(12)
  end

This is the default search. I have to implement also the "data order" search and I think to do something like this:

 def show
    @query = params[:query]
    @contents  = Content.published.search_by_text(@query).page(params[:page]).per(12)
    if params[:order]
       @contents = Content.published.search_by_text(@query).reorder("created_at DESC").page(params[:page]).per(12)
    end
  end

Is there a better way to obtain my result?

Intrepidd · Accepted Answer · 2014-03-31 08:18:28Z

2

Fortunately, rails allows us to chain calls when using Active Record (Rails' ORM)

Here is a possibility :

def show
  @query = params[:query]
  @contents  = Content.published.search_by_text(@query)
  @contents = @contents.reorder("created_at DESC") if params[:order]
  @contents = @contents.page(params[:page]).per(12)
end

answered Mar 31, 2014 at 8:18

Intrepidd

21.2k7 gold badges59 silver badges64 bronze badges

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

Collectives™ on Stack Overflow

How to handle additional param in controller

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related