show間で移動する際に、ransackが解除される問題の解決

Question

こんにちは。

現在プログラミングの構築を行なっており、その中で現在showからshowに移動するプログラムを書いています。

<% if prev_customer = @customer.prev_customer %>
    <%= link_to "前へ", customer_path(prev_customer), class: "prev btn btn-danger" %>
      <% end %>
<% if next_customer = @customer.next_customer %>
    <%= link_to "次へ", customer_path(next_customer), class: "next btn btn-danger" %>
      <% end %>

具体的なプログラムは上記となります。

modelでは

  def next_customer
    Customer.where("id > ?", id).first
  end

  def prev_customer
    Customer.where("id < ?", id).last
  end

と記載しております。

これでも
show/1, show/2, show/3と移動するのですが、これから導入したいこととして、ransackで検索されたindexを元にしてshowを反映させたいと考えております。

例えばransackで検索して、
show/5, show/19, show/23
がindexに抽出された場合、showから直接移動しても上記のように移動するようにしたいのですが、現行のプログラムでは、
show/5, show/6, show/7
と次の数字に移動してしまいます。

要はindexで検索された内容がshowで解除されてしまうため、ransackの条件をshowでも残したいと考えております。

ここを解決する方法は何かないでしょうか？

  def index
    @type = params[:type]
    @q = Customer.ransack(params[:q])
    @customers = @q.result.page(params[:page]).per(100)
  end

  def show
    @q = Customer.ransack(params[:q])
    @customer = @q.result.find(params[:id])
  end

indexの検索条件は以下のようになります。

<div  class="heading"><h2>リスト検索</h2></div>

<!--検索機能実装-->
<%= search_form_for @q do |f| %>
    <table width = "90%">
      <col width="20%">
      <col width="30%">
      <col width="20%">
      <col width="30%">
      <tbody>
          <tr>
            <th colspan = "4">検索</th>
          </tr>
          <tr>
            <th>会社名</th>
            <td><%= f.search_field :company_cont, type: "text" %></td>
            <th>店舗名</th>
		    <td><%= f.search_field :store_cont, type: "text" %> </td>
	      </tr>
	      <tr>
            <th>代表者</th>
            <td><%= f.search_field :first_name_cont, type: "text" %></td>
            <th>ダイヒョウ</th>
            <td><%= f.search_field :first_kana_cont, type: "text" %></td>
          </tr>
          <tr>
            <th>電話番号1</th>
            <td><%= f.search_field :tel_cont, type: "text" %></td>
            <th>電話番号2</th>
            <td><%= f.search_field :tel2_cont, type: "text" %></td>
          </tr>
          <tr>
            <th>FAX番号</th>
            <td><%= f.search_field :fax_cont, type: "text" %></td>
            <th>業種</th>
            <td><%= f.search_field :industry_cont, type: "text" %></td>
          </tr>
          <tr>
            <th>メール</th>
            <td><%= f.search_field :mail_cont, type: "text" %></td>
            <th>URL</th>
            <td><%= f.search_field :url_cont, type: "text" %></td>
          </tr>
          <tr>
            <th>人数</th>
            <td><%= f.search_field :people_cont, type: "text" %></td>
            <th>住所</th>
            <td><%= f.search_field :address_cont, type: "text" %></td>
          </tr>
          <!-- #TPD #2018.12.14 -->
          <tr>
            <th>最終コール状態</th>
            <td><%= select_tag "last_call[statu]", options_for_select( [""] + Call.StatuItems, @last_call_params[:statu] ) %></td>
            <th>再コール日時</th>
            <td></td>
          </th>
          <tr>
            <th>最終コール日時（最初）</th>
            <td>
              <div class='input-group date' id='created_at_from_datetimepicker'>
                    <%= text_field_tag "last_call[created_at_from]", @last_call_params[:created_at_from], class: "form-control" %>
                    <span class="input-group-addon">
                        <span class="glyphicon glyphicon-calendar"></span>
                    </span>
               </div>
             </td>
             <th>最終コール日時（最後）</th>
            <td>
              <div class='input-group date' id='created_at_to_datetimepicker'>
                    <%= text_field_tag "last_call[created_at_to]", @last_call_params[:created_at_to], class: "form-control" %>
                    <span class="input-group-addon">
                        <span class="glyphicon glyphicon-calendar"></span>
                    </span>
               </div>
                <%# text_field_tag "last_detail[created_at_from]", @last_detail_params[:created_at_from], class: "datetimepicker" %>
                <%# text_field_tag "last_detail[created_at_to]", @last_detail_params[:created_at_to], class: "datetimepicker" %>
            </td>
          </tr>
          <tr>
            <th colspan = "4" ><%= f.submit '検索' %></th>
          </tr>
        <% end %>
      </tbody>
    </table>

追加修正

<% if prev_customer = @customer.prev_customer %>
    <%= link_to "前へ", customer_path(@prev_customer, q: params[:q]), class: "prev btn btn-danger" %>
      <% end %>
<% if next_customer = @customer.next_customer %>
    <%= link_to "次へ", customer_path(@next_customer, q: params[:q]), class: "next btn btn-danger" %>
      <% end %>

<% @customers.each do |customer| %>
  <tr>
	 <td><%= link_to customer.company, customer_path(customer, q: params[:q]) %></td>
	 <td><%= link_to customer.first_name, customer_path(customer, q: params[:q]) %></td>
	 <td><%= customer.tel %></td>
	 <td><%= customer.mail %></td>
	 <td><%= link_to '編集', edit_customer_path(customer), class: 'command'%>
			 <%= link_to '削除',
			      customer_path(customer),
			      method: :delete,
			      class: 'command',
			      data: { confirm: '本当に削除しますか？'} %></td>
			 <% end %>

 def before
   @before_customer = Customer.where("id > ?", id).first
 end

 def next
   @next_customer = Customer.where("id < ?", id).last
 end

  def show
    @q = Customer.ransack(params[:q])
    @customer = @q.result.find(params[:id])
  end

Answer 1

prev_customer　達は modelのmethodになっていて、ransacを参照するようになっていないためかと思います。

似たようなことをやっています。

link_to custer_next_customer(@customer, q: params[:q])
とでもし、
method next_customer をコントローラに追加するという方法でどうでしょうか

＃＃＃＃＃
他にもいくつか問題が有ることにきがつきました。あとで時間ができたら追記し

＃＃＃　追記
１） index.html に　<%= link_to "詳細", customer_path(customer), というような記述があると思います。そこを customer_path(customer, q: params[:q]) にしてください。多分今はやっていないのでは？
showのviewとctrlでやり取りが必要ですが、それをindexから渡す必要があります

1. 上にも書きましたが ransackを使うために、next,prevはmodelではなくctrlで行うほうがよいです。

ctrl show で @next_customer、@prev_customer を用意して渡す。
(これですと customer_path(@prev_customer, q: params[:q])でできるので新しいpath作らないで済みます)

以下おせっかい
次前の検索を id: で行ってますがこれ offset で行うと楽かな？と思います。将来　名前順にしたいとかにも対応しやすい。

@offset = params[:id]
@customer = @q.result.offset(@offset).first

viewには@prev_customerでなく、@offsetをわたし、customer_path(@offset-1, q: params[:q])とかする。　id でなく offset を騙して渡すわけです。
ですから、index.htmlでも

@customers.each.with_index( (page - 1) * par_page ){|customer,offset| 
  link_to 表示, customer_path(offset, q: params[:q])
 }

Comments

[KOO_]

お返事ありがとう御座います。細かくご教示いただきありがとう御座います。ただ、私の中で本当にスキル不足もあり、なかなか全てを理解するのが難しいかもしれません。offsetについては特に使ったこともなく更に混乱しそうなので、ご教示頂いた内容をゆくゆく導入して行きたいと思います。
その中で色々やってみたのですが、もしわがままを言って良ければ内容を拝見頂けないでしょうか？よろしくお願い致します。

[KOO_]

ちょっとcontrollerは解決出来なさそうでした。

Answer 2

リクエスト先のparams[:q]に対して元のparams[:q]を渡してやれば良いです。

link_to "前へ", customer_path(prev_customer, q: params[:q]), class: "prev btn btn-danger"

こういうことかな？よくわからないけど。

def prev_customer
  @q.result.where("id < ?", id).last
end

Comments

[KOO_]

ご回答頂き誠にありがとう御座います。上記でparams[:q]を渡しておりますが、結果が変わらないようです。
その他考えられることは御座いますでしょうか？よろしくお願い致します。

[KOO_]

ご回答頂き誠にありがとう御座います。上記でparams[:q]を渡しておりますが、結果が変わらないようです。
その他考えられることは御座いますでしょうか？よろしくお願い致します。

[rhiroe]

結果が変わらないというのは見た目の上だけの話ですか？それともコードをデバッグした上での話ですか？
結果が変わらないという表現だけでは当事者にしかわからないので何も考えられることがないです。
強いて挙げるなら回答自体が間違えていたか、ransackの使い方を間違えているかですかね。
ransackを使ったことがないのでもしかしたら間違えているのかもしれません。

現状あげられているコードのみで考えた場合showからshowへの移動の際
@q = Customer.ransack(params[:q])
の部分はparams[:q]が与えられないので
@q = Customer.ransack(nil)
となっています。なので上記の回答をしただけです。

[KOO_]

ありがとう御座います。おそらくindexでは抽出されている条件がshowでは適応されていないのですね。
ちなみにindexでの条件をshowで引き継ぐ方法は御座いますでしょうか？お手数おかけしますが、よろしくお願い致します。

[rhiroe]

indexでの条件というのは何のことでしょうか？
具体的にコードで示してもらえるとわかりやすいです。

[KOO_]

ご連絡遅くなりました。
indexにあるsearch条件を追加しました。
よろしくお願い致します。

[rhiroe]

回答に追記した通り @q.result から前後のレコードを取得して移動すればいいのでは？と思ったんですが。