【Ruby on Rails】検索画面を1つの画面で実装したい

Question

Ransackを使った検索画面のサンプルがあります。
http://rails.densan-labs.net/form/search_form.html

上記では検索画面(index)と、結果画面(search)が別です。

これを1つの画面(html)にしたいです。

検索画面側は、検索フォームのみ
結果画面側は、検索フォーム＋結果表示部分
で構成されています。

今回は、searchの方だけで実現しようと以下のようにしましたが、
NoMethodErrorで怒られ初期表示できません。

すみませんがご教示いただけないでしょうか？
宜しくお願い致します。

＠＠＠コントローラ

class ProductsController < ApplicationController

  def index

    @q = Product.search

  end

  def search

		if params.nil?

	    @q = Product.search(search_params)

	    @products = @q
	      .result
	      .order(availability: :desc, code: :asc)
	      .decorate

		else

			@q = Product.search

			@products = @q

		end



  end

  private

  def search_params
    search_conditions = %i(
      code_cont name_cont name_kana_cont availability_true
      price_gteq price_lteq purchase_cost_gteq purchase_cost_lteq
    )
    params.require(:q).permit(search_conditions)
  end

end

＠＠＠＠ビュー
下記の<p>カウント:<%= @products.count %></p>でエラー

<% content_for(:title) do %>
  <h1>商品検索</h1>
<% end %>




	<%= search_form_for(@q, url: search_products_path, html: { method: :get, class: 'form-horizontal', role: 'form' }) do |f| %>
	  <%= render 'search_form', f: f %>
	<% end %>


<%= render layout: 'shared/search_result_box' do %>
  <table class="table table-list ">
  <thead>
    <tr>
      <th>販売状況</th>
      <th>商品コード</th>
      <th>商品名</th>
      <th>販売価格</th>
      <th>仕入原価</th>
    </tr>
  </thead>

	<p>カウント:<%= @products.count %></p>
	<% @products.each do |product| %>
	  <tbody>
	  <tr>
	    <td><%= product.sales_condition %> </td>
	    <td><%= product.code %></td>
	    <td><%= product.name %></td>
	    <td><%= product.display_price %></td>
	    <td><%= product.display_purchase_cost %></td>
	  </tr>
	  </tbody>
	<% end %>


	</table>

<% end %>

Answer 1

else以降の@productsが
ransack::searchobjectのままです。
resultをつけてActiveRecord::Relationに戻しましょう

 else

            @q = Product.search

            @products = @q.result

        end

あと、好みの域をでないのですが
できればindexの方だけで実現した方が...
indexはデフォルトで
def search　は余計なrouteを追加することに他ならないので

Comments

[pecchan]

moke様

いつも有難う御座います。

また基礎的な部分が足りないなぁと痛感しました(泣)

Product.searchで受けたのでresultで返す！ですね。
※ちょっと調べました

あとデコレーションしてるのでdecorateも追加しました。

今は、おかげ様でエラーは出なくなりましたが、何故か検索条件が渡らないようになったので
調べています！

有難う御座いました！

あ！完成したらsearch→indexに変えようと思ってました^^

[moke]

えっ検索条件をあえて渡さない処理なのでは?
意図した動きと異なるというのなら
if文が逆ですよ
if params.present?
にするのが正しいのでは？
デフォルトの検索条件をつけないor他で定義するなら
いっそのことif文を取ってしまえばいいと思いますが

[pecchan]

でした！
すいません。。。
いっそif分を無くしました。

有難う御座いました！

[pecchan]

今からアクションをindexに変えるのと、
１つの入力でコード、名前、仮名などを検索できるようにしてみます！！

[pecchan]

ifで分岐したのは、
（serachでもindexでも同じですが）1画面にした場合、検索前の初期表示時、
paramsの:qで「param is missing or the value is empty: q」と怒られるからでした(^^;)
なので1画面にした際、ifでparamsを見てました。
でも何か違う気はしてました・・・orz

[moke]

require(:q)にしてましたか
だと、エラーが出ますね…。
一応こういうのがあるのですが、まあ、わかりやすい方を設定してください。
# `ransackable_attributes` by default returns all column names
# and any defined ransackers as an array of strings.
# For overriding with a whitelist array of strings.
#
def ransackable_attributes(auth_object = nil)
column_names + _ransackers.keys
end

# `ransackable_associations` by default returns the names
# of all associations as an array of strings.
# For overriding with a whitelist array of strings.
#
def ransackable_associations(auth_object = nil)
reflect_on_all_associations.map { |a| a.name.to_s }
end

[pecchan]

moke様

度々有難う御座います。
Productモデルに追加したのですが同じエラーになります。。。

[moke]

search_params
の代わりにparams[:q]
を使うプランの話です。

[pecchan]

moke様

度々すみません、有難う御座います。

コントローラの
@q = Product.search(search_params)
を
@q = Product.search(params)
に変更しました。

初期表示は出来るようになりましたが、検索しても何も表示されなくなりました。
１つ１つ調べてます。
申し訳ございません。

[moke]

@q = Product.search(params[:q])
ですよ

[pecchan]

あぁぁぁぁ　
１から10まですみません。
やっぱり解決できずにおりました(;_;)
有難う御座います！

search_params（ストロングパラメータ）が使えないかわりに、
column_names + _ransackers.keysを使うってことでしょうか？！

[moke]

そちらの方が見通しがよくなると思います。
まあ、好みの問題ですが。
特にassosiationやsortに関してもあるので、より細かく設定できると思います。

[pecchan]

ストロングパラメータ(最近知ったばかり)しかないと思ってました(*_*)
色んな方法があるんですね。
仰る通り見通しが良くて好きです！
有難う御座いました！！