子モデルのデータのテーブルのカラムをソートしたい

Question

前提・実現したいこと

楽曲とその音源を登録するアプリを作っています。1つの曲に対し複数の演奏音源がつきます。親モデルはpostで、子モデルはrecordingです。

楽曲(親モデル)を表示するページにて、Sortable Table Columnsおよび[Railsで並べ替え可能なテーブルの実装手順](a href="https://www.virment.com/rails-sortable-table/)を参考にして音源(子モデル)の情報をカラム別にソートできるテーブルを作っています。

発生している問題・エラーメッセージ

テーブルに表示されたリンクをクリックしても何も起こりません。
URL自体はhttp://localhost:3000/posts/60?direction=asc&sort=Mov2という風に機能します。

該当のソースコード

モデル
Post.rb
class Post < ApplicationRecord
  has_many :recordings, dependent: :destroy
  has_many :post_instruments
  has_many :instruments, :through => :post_instruments, dependent: :destroy
  accepts_nested_attributes_for :recordings, {allow_destroy: true}
end

recording.rb
class Recording < ApplicationRecord
  belongs_to :post
  has_many :durations, dependent: :destroy
  accepts_nested_attributes_for :durations
  mount_uploader :image, ImageUploader
end

コントローラおよびヘルパー
application_controller.rb

ApplicationController
class ApplicationController < ActionController::Base
    include SessionsHelper
    include RecordingsHelper
end


recordings_helper.rb

module RecordingsHelper
    def sortable(column, title = nil)
        title ||= column.titleize
        css_class = (column == sort_column) ? "current #{sort_direction}" : nil
        direction = (column == sort_column && sort_direction == "asc") ? "desc" : "asc"
        link_to title, {:sort => column, :direction => direction}, {:class => css_class}
    end
  
    def sort_direction 
        %w[asc desc].include?(params[:direction]) ? params[:direction] : "asc"
    end

    def sort_column
        Recording.column_names.include?(params[:sort]) ? params[:sort] : "name"
    end
end

posts_controller
helper_method :sort_column, :sort_direction

  def show
    @post = Post.includes(:instruments, :recordings=>:durations).find(params[:id])
    #@post.recordings.order(sort_column+ "" + sort_direction)
    @post.recordings.order("recordings + sort_column+  ' ' + sort_direction")
    @instruments = @post.instruments.order('created_at desc')
  end

ビュー(show.html.erb)

<table>

  <tr><%# Start of th　%>
    <% @instruments.each do |instrument| %>%# Start of instruments th　%><%# 楽曲編成に応じてカラムを追加　%>
      <% if instrument.name.include? 'orchestra'  %><%# 管弦楽曲の場合、指揮者カラムを追加　%>
        <th><%= sortable "conductor", "Conductor" %></th>
      <% end %>

      <% if instrument.name.include? "#{instrument.name}" %>
        <th><%= sortable "#{instrument.name}","#{instrument.name}" %></th>
      <% end %>
    <% end %><%# End of instruments th　%>

    <% ([e-mail]).each do |movnum| %><%# 楽章数に応じてカラム数が変化　%>
        <th width="70"><%= sortable "Mov#{movnum}" %></th>
    <% end %>
  </tr><%# End of th　%>


<% @post.recordings.each do |recording| %>

  <tr>
  <% @instruments.each do |instrument| %>

      <% if instrument.name.include? 'orchestra'  %>
        <td><%= recording.conductor %></td>
      <% end %>

      <% if instrument.name.include? "#{instrument.name}" %>
        <td><%=recording.send("#{instrument.name}") %></td>
      <% end %>

  <% end %><%# End of the loop of instruments　%>

  <% recording.durations.each do |duration| %><%# 楽章ごとの演奏時間　%>
       <td><%=Time.at(duration.total_seconds).utc.strftime("%M:%S") %></td>
  <% end %><%# End of durations　%>

  </tr>

  <% end %><%# End of the loop of each recordings　%>
  
</table>

試したこと

親モデルにおいて子モデルのデータをソートするには
親モデル.includes(:子モデル).oder("子モデル名の複数形.カラム名 DESC")
とすればいいらしいですが、それと今回のことをどう組み合わせればいいか分かりません。

Helperを親モデルにすることなどを試しましたが結果は変わりませんでした。

補足情報（FW/ツールのバージョンなど）

Ruby on Rails 6.0

Comments

[winterboum]

viewを載せてください

[winterboum]

そのviewには「テーブルに表示されたリンク」が見当たらないんですが？

[kawa0tatsu]

<th><%= sortable... %>の部分のことを言っているつもりなのですが･･･。

[winterboum]

ああ、そこか。
この　link_to がよろしくないように見えます。
どのURLになることを期待してますか？

[kawa0tatsu]

クリックして表示されるURL自体は正しいように見えます。例：http://localhost:3000/posts/56?direction=asc&sort=Mov1

クリックしたら格納されている文字列がアルファベット順、そして演奏時間(time型)の数字が昇順・降順で並び替えられるようにしたいですが、そうなりません。あと、sortがascだけで、クリックしてもdescになりません。

Answer 1

@post.recordings.order("recordings + sort_column+ ' ' + sort_direction")
を
@post.recordings.order(ort_column+ ' ' + sort_direction)
にしてみてください

ただ、、、direction=asc&sort=Mov2 の Mov2　が気になります。
ここは Recording のカラム名でなければならないのですが、そういうのあります？
まず、captalで始まってるのも？？ですし

Comments

[kawa0tatsu]

(1)@post.recordings.order(sort_column+ ' ' + sort_direction)にしてみてください＞

最初そうしたら効かなかったので、今回質問しました。

子モデルのソートは 親モデル名.includes(:子モデル名).order("子モデル名の複数形.カラム名 desc")
で出来るらしいので、それを組み合わせるべきかと思って
@post.recordings.order('recordings.' + sort_column + ' ' + sort_direction)
としたのですが、出来ませんでした。

そもそもこれらの文言があってもなくても挙動に影響がないので、ヘルパーとかメソッドの置き方にも問題があるのかも知れません。いろいろ試してますが出来ていません。

(2)まず、captalで始まってるのも？？ですし＞
今回のコードは参考もとのサイトのコピペですが、sortableメソッドにおけるtitlizeで自動的に大文字になるらしいです。ただ、なぜこれを使う必要があるのかはよく分かりません。

(3)確かに言われてようやく気づきましたが、mov1, mov2などというカラムは確かにRecordingモデルには存在しません。

末尾の番号はDurationモデル(recordingモデルの子モデル、postモデルからみて孫モデル)においてmovカラムに格納しています。

また、Durationモデルに格納した分と秒をもとに、こちら(https://teratail.com/questions/252331)で書いたように、自作関数で出した時間を表示しています。これもソートしたいですが、別の複雑な問題になるので、これについてはまた質問を立て直そうかと思います。