チケット一覧画面のステータスにカスタムデータ属性をつけたい

Question

いつも勉強させていただいております。
redmineのチケット一覧で表示される「ステータス」にカスタムデータ属性を付け、属性値にはステータス名がそのまま入るようにしたいと思っております。
rubyやredmineの知識は少なく、なんとなく該当しそうな部分を探し、編集し再起動させてみましたが、
発行されるHTMLをみても変更されておらず、皆様にご指導賜りたく、質問いたしました。

仕様バージョン：3.3.0-1

どこを変更すればいいのかを特定するために下記の部分のクラスを編集してみました。

issues_helper.rb




def render_descendants_tree(issue)
   s = '<form><table class="list issues">'
   issue_list(issue.descendants.visible.preload(:status, :priority, :tracker, :assigned_to).sort_by(&:lft)) do |child, level|
     css = "issue issue-#{child.id} hascontextmenu #{issue.css_classes}"
     css << " idnt idnt-#{level}" if level > 0
     s << content_tag('tr',
            content_tag('td', check_box_tag("ids[]", child.id, false, :id => nil), :class => 'checkbox') +
            content_tag('td', link_to_issue(child, :project => (issue.project_id != child.project_id)), :class => 'subject', :style => 'width: 50%') +
            content_tag('td', h(child.status), :class => 'status') +
            content_tag('td', link_to_user(child.assigned_to), :class => 'assigned_to') +
            content_tag('td', child.disabled_core_fields.include?('done_ratio') ? '' : progress_bar(child.done_ratio), :class=> 'done_ratio'),
            :class => css)
   end
   s << '</table></form>'
   s.html_safe
 end

上記のコードの

content_tag('td', h(child.status), :class => 'status') 

クラス名を

content_tag('td', h(child.status), :class => 'test') 

に変更し、apacheを再起動してみました。
しかし、HTMLは変更されていませんでした。

まず、どの部分を変更すればよいのかを教えてもらえないでしょうか？
よろしくお願いいたします。

追記

最終的に発行したいHTMLの抜粋を記載します

<div class="autoscroll">
<table class="list issues sort-by-id sort-desc">
  #thead省略
  <tbody>
  <tr id="issue-29" class="hascontextmenu odd issue tracker-4 status-1 priority-2 priority-default created-by-me">
    <td class="checkbox hide-when-print"><input type="checkbox" name="ids[]" value="29"></td>
    <td class="id"><a href="/redmine/issues/29">29</a></td><td class="tracker">デザイン</td><td class="status" data-color="新規">新規</td><td class="priority">通常</td><td class="subject"><a href="/redmine/issues/29">04_メイキング</a></td><td class="assigned_to"></td><td class="updated_on">2019/10/27 11:30</td><td class="spent_hours">0.00</td><td class="total_spent_hours">0.00</td>
  </tr>

  </tbody>
</table>
</div>

<td class="status" data-color="新規">という風にカスタムデータ属性を付けたいと思っています

Comments

[nandymak]

＞チケット一覧で表示される「ステータス」にカスタム属性を付け、属性値にはステータス名がそのまま入るようにしたい

イメージが湧かないので手書きでも良いので画面のサンプルイメージを提示しては如何でしょうか？

後、最新バージョンは4.1だったと思いますが、かなり機能改善が適用されているので、リリース内容をチェックしてみれば案外実現されているかもしれません。

[beginner_t]

すいません。誤記がありましたので修正いたしました。正しくは「カスタムデータ属性」です。内容に追記いたします。

[beginner_t]

gitにある最新バージョンのソースを確認しましたが、該当部分の記載内容は同じでした。content_tag('td', h(child.status), :class => 'status') とあったので、同じようにclassのみ出力されるようになっているようです。

[nandymak]

ご覧になったソースの箇所が間違っているとは考えられませんか？ソースをすべて見るのは大変なので、外形の動きから類する方法もあるかなぁと思います。

[beginner_t]

HTMLから付与されているclass名でappsディレクトリ内をgrep検索をかけていたのですが、該当しそうな部分が上記のファイルくらいでした。なので、該当ファイルであっているかを含めて質問させていただいております。

Answer 1

td.statusの配下にp.statusを作るやり方しかできなかったのですが、この方法で代用になりませんか。

../app/helpers/queries_helper.rb

def column_value(column, item, value)
    case column.name
    when :status
      content_tag('p', value, :class => "status", 'data-color' => "#{value}")
    when :id
      link_to value, issue_path(item)
    when :subject

Comments

[beginner_t]

ご回答いただきありがとうございます。
有用な情報ありがとうございます。今回自己解決とさせていただきましたが、
必要に応じて教えていただいたものを活用していきたいと思います。

[takashikawai]

Viewで対応したんですね。もし、不必要な部分にカスタムデータ属性が付かないようにしたければ、

<tr id="issue-<%= issue.id %>" class="hascontextmenu <%= cycle('odd', 'even') %> <%= issue.css_classes %> <%= level > 0 ? "idnt idnt-#{level}" : nil %>">
<td class="checkbox hide-when-print"><%= check_box_tag("ids[]", issue.id, false, :id => nil) %></td>
<% query.inline_columns.each do |column| %>
<% if column.name.to_s == 'status' %>
<%= content_tag('td', column_content(column, issue), :class => column.css_classes, 'data-color'=> "#{column_content(column, issue)}") %>
<% else %>
<%= content_tag('td', column_content(column, issue), :class => column.css_classes) %>
<% end %>
<% end %>
</tr>

とすれば良いと思います（対象バージョン redmine 3.4 ）。

[beginner_t]

rubyの書き方が全然わからなかったので、ちょうど調べていました。教えていただきありがとうございます。

Answer 2

views/issues/_list.html.erbを編集することで希望の場所へカスタムデータ属性を追加することはできました。

<%= form_tag({}) do -%>
<%= hidden_field_tag 'back_url', url_for(params), :id => nil %>
<div class="autoscroll">
<table class="list issues <%= sort_css_classes %>">
  <thead>
    <tr>
      <th class="checkbox hide-when-print">
        <%= check_box_tag 'check_all', '', false, :class => 'toggle-selection',
              :title => "#{l(:button_check_all)}/#{l(:button_uncheck_all)}" %>
      </th>
      <% query.inline_columns.each do |column| %>
        <%= column_header(column) %>
      <% end %>
    </tr>
  </thead>
  <tbody>
  <% grouped_issue_list(issues, @query, @issue_count_by_group) do |issue, level, group_name, group_count, group_totals| -%>
  <% if group_name %>
    <% reset_cycle %>
    <tr class="group open">
      <td colspan="<%= query.inline_columns.size + 1 %>">
        <span class="expander" onclick="toggleRowGroup(this);">&nbsp;</span>
        <span class="name"><%= group_name %></span> <span class="count"><%= group_count %></span> <span class="totals"><%= group_totals %></span>
        <%= link_to_function("#{l(:button_collapse_all)}/#{l(:button_expand_all)}",
                             "toggleAllRowGroups(this)", :class => 'toggle-all') %>
      </td>
    </tr>
  <% end %>
  <tr id="issue-<%= issue.id %>" class="hascontextmenu <%= cycle('odd', 'even') %> <%= issue.css_classes %> <%= level > 0 ? "idnt idnt-#{level}" : nil %>">
    <td class="checkbox hide-when-print"><%= check_box_tag("ids[]", issue.id, false, :id => nil) %></td>
    <%= raw query.inline_columns.map {|column| "<td class=\"#{column.css_classes}\" data-color=\"#{column_content(column, issue)}\">#{column_content(column, issue)}</td>"}.join %>
  </tr>
  <% @query.block_columns.each do |column|
       if (text = column_content(column, issue)) && text.present? -%>
  <tr class="<%= current_cycle %>">
    <td colspan="<%= @query.inline_columns.size + 1 %>" class="<%= column.css_classes %>"><%= text %></td>
  </tr>
  <% end -%>
  <% end -%>
  <% end -%>
  </tbody>
</table>
</div>
<% end -%>

編集した部分

<%= raw query.inline_columns.map {|column| "<td class=\"#{column.css_classes}\">#{column_content(column, issue)}</td>"}.join %>

に下記のようにカスタムデータ属性を追加してみました。

<%= raw query.inline_columns.map {|column| "<td class=\"#{column.css_classes}\" data-color=\"#{column_content(column, issue)}\">#{column_content(column, issue)}</td>"}.join %>
  </tr>

この結果

<tr id="issue-29" class="hascontextmenu odd issue tracker-4 status-1 priority-2 priority-default created-by-me">
    <td class="checkbox hide-when-print"><input type="checkbox" name="ids[]" value="29"></td>
    <td class="id" data-color="<a href=" redmine="" issues="" 29"="">29"&gt;<a href="/redmine/issues/29">29</a></td>
    <td class="tracker" data-color="デザイン">デザイン</td><td class="status" data-color="新規">新規</td>
    <td class="priority" data-color="通常">通常</td><td class="subject" data-color="<a href=" redmine="" issues="" 29"="">04_メイキング"&gt;<a href="/redmine/issues/29">04_メイキング</a></td>
    <td class="assigned_to" data-color=""></td>
    <td class="updated_on" data-color="2016/10/27 11:30">2016/10/27 11:30</td><td class="spent_hours" data-color="0.00">0.00</td>
    <td class="total_spent_hours" data-color="0.00">0.00</td>
  </tr>

という風にステータスの変化に応じでカスタムデータ属性を付けることはできましたが、不必要な部分にもカスタムデータ属性がついてしまいました。
しかし、実現したいことの要件は満たしたので一旦解決とさせていただきます。
ご回答いただいた方、大変勉強になりました。ありがとうございました。

Answer 3

View Customize Pluginを使えば簡単にできます。

パスのパターン:/issues$
挿入位置:全ページのヘッダ
種別:JavaScript

$(function() {
    $('td.status').each(function() {
        var text = $(this).text();
        $(this).attr("data-color", text);
    })
})

Comments

[beginner_t]

ご回答ありがとうございます。jsでの付与も考えていましたが、rbファイルで該当部分を変更できるなら余計なリソースを増やさなくてもいいので、rbファイルでの変更ができないのであれば、採用させていただきます。