rails/PaperTrail+ActiveScaffold の履歴(No.17)

Rails の管理画面と言えば rails_admin†

rails_admin を使うと、Rails で動く Web アプリケーションに、データ管理画面を簡単に作成できます。

データの更新履歴を残したい paper_trail + paranoia†

一方で、

• paper_trail: データの更新履歴を取る
• paranoia: 削除済みデータを閲覧・復帰可能にする

を用いると、レコードの各カラムの変更履歴を残し、さらに削除済みのデータを閲覧可能になります。

データ更新履歴を見られる rails_admin ってないの？†

paper_trail + paranoia に対応した rails_admin （みたいなもの）って無いんでしょうか？

見つけられなかったため、そういう管理画面を active_scaffold で作ろうともがいているのがこの記事です。

active_scaffold†

active_scaffold を使うことで、レコードに対する更新履歴のような has_many 子要素まで対応可能な閲覧・編集画面を簡単に作成できます。

ということで†

paper_trail + paranoia でデータの更新履歴を残しつつ、 そのデータを active_scaffold で閲覧・編集できるようにしようともがいているのが以下の記事です。

そこそこ動くようになったところから読みたい方は、無理矢理ライブラリ化してみた からどうぞ。

なんかいろいろ車輪の再発明をしている気がするのですが、 こういう機能を簡単に持たせることって、もっと簡単にできたりしないんでしょうか？？？

考えてみると†

active_scaffold で１から作るという方向の他に、

http://qiita.com/joker1007/items/4ac31f081c44634a5e90 などを参考にして rails_admin を paper_trail + paranoia に対応させるという方向もあって、

考えてみるとそちらの方が需要多かったのかもしれないです？いまさらですが orz

一応、active_scaffold の知識があれば管理画面のカスタマイズがいろいろ捗るというあたりが自分で作るメリットということになるのでしょうか？？？

モデルの作成†

Active Scaffold で管理される Test というモデルを作ります。

$ rails g active_scaffold Test name:string email:string
       invoke  active_record
       create    db/migrate/20140424103245_create_tests.rb
       create    app/models/test.rb
       invoke    test_unit
       create      test/models/test_test.rb
       create      test/fixtures/tests.yml
        route  resources :tests do as_routes end
       invoke  active_scaffold_controller
       create    app/controllers/tests_controller.rb
       create    app/helpers/tests_helper.rb
       invoke    test_unit
       create      test/controllers/tests_controller_test.rb
       create    app/views/tests
$ rake db:migrate
 == 20140424103245 CreateTests: migrating ======================================
 -- create_table(:tests)
    -> 0.0043s
 == 20140424103245 CreateTests: migrated (0.0044s) =============================

編集可能になった†

Paper Trail による履歴管理を追加†

このモデルに has_paper_trail を追加して、履歴情報を保存すると共に、 ActiveScaffold からその履歴情報へアクセスするための histories というプロパティを定義します。

app/models/test.rb

class Test < ActiveRecord::Base
  has_paper_trail :ignore => [:id, :created_at, :updated_at]
end

ただし、id, created_at, updated_at の各カラムは履歴管理の必要がないため除外します。

編集不可能になった†

ActionView::Template::Error (Could not find ::PaperTrail::VersionsController or ::PaperTrail::VersionController or PaperTrail::VersionsController or PaperTrail::VersionController):

Tests コントローラを編集†

app/controllers/tests_controller.rb

class TestsController < ApplicationController
  active_scaffold :"test" do |conf|
    conf.columns.exclude :versions
  end
end

ActiveScaffold が versions を表示できないので除外しました。

編集可能になった†

１つレコードを追加した後、編集中。

モデルに履歴情報へのリンクを持たせます†

http://kingyo-bachi.blogspot.jp/2012/10/activescaffoldpapertrail.html を参考にしつつ、 多数のモデルの履歴を取るのに便利なように工夫しています。

class Test < ActiveRecord::Base
  has_paper_trail

  class History < ItemHistory
  end
  has_many :histories, -> { where 'item_type = "Test"' }, foreign_key: 'item_id', class_name: 'Test::History'
end

app/models/item_history.rb

class ItemHistory < PaperTrail::Version

  def method_missing(action)
    unless self.reify.attributes.keys.index(action.to_s).nil?
      self.reify[action]
    else
      super
    end
  end

end

PaperTrail::Version には、複数のモデルの履歴がまぜこぜに追加されるので、 自分の履歴情報のみを取り出すため、item_type="Test" を指定しています。

routes の設定†

上記履歴情報へ /tests/:test_id/histories というパスでアクセスできるようにします。

コントローラ名はモデル名 Test::History に合わせなければならないため、 Test::HistoriesController とします。

config/routes.rb

resources :tests do 
  as_routes
  resources :histories, controller: 'test/histories' do
    as_routes
  end
end

履歴情報のコントローラ†

app/controllers/test/histories_controller.rb

class Test::HistoriesController < ApplicationController
  active_scaffold :"test/history" do |conf|
  end
end

履歴情報へのリンクが追加されました†

クリックすると、

ちゃんと履歴情報が表示されます。

履歴情報の見た目を変更†

上記のままだと何が変更されたか見にくく、 また、履歴情報の編集ができてしまうので、いろいろ直します。

app/controllers/test/histories_controller.rb

# coding:utf-8
class Test::HistoriesController < ApplicationController
  active_scaffold :"test/history" do |conf|
    # 編集を不可にする
    conf.actions.exclude :create, :update, :delete

    # 履歴に日時を表示する
    conf.columns.add :created_at
    conf.columns[:created_at].label = "Time"

    # リストに表示する内容と順番を指定
    conf.list.columns= [:item, :event, :created_at, :object_changes]

    # 検索をしやすくする
    conf.actions.exclude :search
   conf.actions.add :field_search
    conf.field_search.columns = [:item_id,:event,:created_at]
  end
end

app/helpers/test/histories_helper.rb

module Test::HistoriesHelper

  def test_history_created_at_column(record, column)
    # record.created_at.in_time_zone(@user.time_zone) のようにしても良い
    record.created_at ? record.created_at.localtime.strftime('%m/%d %H:%M') : ''
  end
  
  def test_history_object_changes_column(record, column)
    # まるっきり美しくないけれどとりあえず
    result = "<table>"
    record.changeset.each do |k,v|
      result += "<tr><td style=\"text-align:right;border:0px;width:100px;\">"+
                "<b>#{k}</b><td style=\"border:0px;\">#{v}</tr>\n"
    end
    result += "</table>"
    result.html_safe
  end
  
  def test_history_item_column(record, column)
    "#"+record.item_id.to_s
  end
end

履歴部分が分かりやすくなりました†

モデル側の調整†

• histories を編集不能にする
• created_at, updated_at を表示する
• histories の見た目を調整

app/controllers/tests_controller.rb

class TestsController < ApplicationController
  active_scaffold :"test" do |conf|
    conf.columns.exclude :versions
    
    conf.show.columns.add :created_at, :updated_at

    conf.create.columns.exclude :histories
    conf.update.columns.exclude :histories
  end
end

app/helpers/tests_helper.rb

module TestsHelper
 
  def test_updated_at_column(record, column)
    record.updated_at ? record.updated_at.localtime.strftime('%m/%d') : ''
  end

  def test_created_at_column(record, column)
    record.created_at ? record.created_at.localtime.strftime('%m/%d') : ''
  end

  def test_histories_column(record, column)
    "# " + record.versions.size.to_s
  end

end

だいたいＯＫ†

/tests/1/histories への参照がエラーになるなど腑に落ちない点は残りますが、 履歴情報を閲覧するという目的に限れば、これでほぼ問題なく使えると思います。

/tests/1/histories への参照は、

No route matches {:action=>"edit", :association=>:item, :controller=>"test/tests", :id=>"--CHILD_ID--", :parent_scaffold=>"test/histories", :test_id=>"1", :version_id=>"--ID--"}

というエラーを生じました。{:controller=>"test/tests"} がまずいのですが、 正攻法での回避方法は分かりませんでした。

この問題はクラス階層を正しく扱えないことが原因なので、

• 履歴モデルを Test::History ではなく TestHistory にする
• コントローラ等を Test::Histories ではなく TestHistories にする

ことにより解決されるようです。

削除したレコードの履歴がたどれない†

削除したレコードの履歴もたどりたければ、上記では対応できません。

ではどうするかということで考えたのが以下になります。

歴代レコードのモデル†

Test と Test::History への参照を持たせます。

app/models/historical_test.rb

class HistoricalTest < ActiveRecord::Base
  belongs_to :test, foreign_key: 'item_id'
  has_many :histories, -> { where 'item_type = "Test"' }, 
      primary_key: 'item_id', foreign_key: 'item_id', class_name: 'Test::History'
end

routes の設定†

config/routes.rb

resources :tests do 
  as_routes
end

resources :historical_tests do 
  as_routes 
  resources :histories, controller: 'test/histories' do
    as_routes
  end
end

とりあえず表示できるようになりました†

ここまでで以下が可能です。

• destroy されたレコードの履歴をたどる
• 存在するレコードを更新

表示を整える†

app/controller/historical_tests_controller.rb

class HistoricalTestsController < ApplicationController
  active_scaffold :"historical_test" do |conf|
    conf.actions.exclude :search,:create,:update,:delete,:show

    conf.columns.exclude 'id:1'

    conf.actions.add :field_search
    conf.field_search.columns.exclude :test,:histories

    conf.columns[:test].actions_for_association_links = [:show]
    conf.columns[:last_event_at].label = 'Modified'

    conf.list.columns= [:item_id, :test, :last_event_at, :histories]
  end
end

app/helper/historical_tests_helper.rb

module HistoricalTestsHelper

  def historical_test_last_event_at_column(record, column)
    record.last_event_at ? record.last_event_at.localtime.strftime('%y/%m/%d %H:%M') : ''
  end

  def historical_test_test_column(record, column)
    if record.last_event.to_s == 'destroy'
      "-- destroyed --"
    else
      "#{record.test.name} <#{record.test.email}>"
    end
  end

  def historical_test_histories_column(record, column)
      "# #{record.histories.count}"
  end

  def last_event_search_column(record, input_name)
    select :record, :last_event, options_for_select(['create','update','destroy']), 
        {:include_blank => as_(:_select_)}, input_name
  end
 
end

良い感じ†

これで削除済みのレコードを含めて、すべての履歴を参照できるようになりました。

acts_as_paranoid の設定†

$ rails generate migration AddDeletedAtToTests deleted_at:datetime:index
$ rake db:migrate

app/models/test.rb

class Test < ActiveRecord::Base
  acts_as_paranoid
  validates :email, presence: true, uniqueness: {scope: :deleted_at}

  ...

end

acts_as_paranoid と validates uniqueness を付け加え、 削除と、削除済みを考慮した重複検出が行えていることを確認できました。

削除済みレコードを表示する†

このままだと削除されたレコードが一覧に出てこないので、

app/controllers/tests_controller.rb

class TestsController < ApplicationController

  ...
  
  def beginning_of_chain
    super.unscoped
  end

end

とすることで、削除済みのアイテムを表示できました。

    super.unscoped

    nested? ? super : super.unscoped

histories を一覧†

上記だけだと histories のリンクで Request Failed となってしまいます。

Started GET "/tests/4/histories?association=histories&parent_scaffold=tests&adapter=_list_inline_adapter" for 192.168.133.1 at 2014-04-27 14:50:44 +0900
Processing by Test::HistoriesController#index as JS
  Parameters: {"association"=>"histories", "parent_scaffold"=>"tests", "adapter"=>"_list_inline_adapter", "test_id"=>"4"}
  Test Load (0.2ms)  SELECT "tests".* FROM "tests" WHERE "tests"."deleted_at" IS NULL AND "tests"."id" = ? LIMIT 1  [["id", "4"]]
Completed 404 Not Found in 3ms

ActiveRecord::RecordNotFound (Couldn't find Test with id=4 [WHERE "tests"."deleted_at" IS NULL]):
  activerecord (4.0.4) lib/active_record/relation/finder_methods.rb:199:in `raise_record_not_found_exception!'
  activerecord (4.0.4) lib/active_record/relation/finder_methods.rb:285:in `find_one'
  activerecord (4.0.4) lib/active_record/relation/finder_methods.rb:269:in `find_with_ids'
  activerecord (4.0.4) lib/active_record/relation/finder_methods.rb:36:in `find'
  activerecord-deprecated_finders (1.0.3) lib/active_record/deprecated_finders/relation.rb:122:in `find'
  activerecord (4.0.4) lib/active_record/querying.rb:3:in `find'
  /home/osamu/.bundler/ruby/2.0.0/active_scaffold-6b895cd14a2a/lib/active_scaffold/finder.rb:325:in `find_if_allowed'
  /home/osamu/.bundler/ruby/2.0.0/active_scaffold-6b895cd14a2a/lib/active_scaffold/actions/nested.rb:97:in `nested_parent_record'

lib/active_scaffold/actions/nested.rb:97:in `nested_parent_record' で find_if_allowed に渡されている nested.parent_model にスコープが付いているのがいけないようなので、

class Test::HistoriesController < ApplicationController

  ...

  def nested_parent_record(crud = :read)
#   @nested_parent_record ||= find_if_allowed(nested.parent_id, crud, nested.parent_model)
    @nested_parent_record ||= find_if_allowed(nested.parent_id, crud, nested.parent_model.unscoped)
  end

end

としたところ、ちゃんと表示されるようになりました。

delete 済みのレコードに edit と delete のアクションが表示される件†

２つの方法があるようです。

class Test < ActiveRecord::Base
  ...
   
  def authorized_for_delete?
    not deleted_at
  end
  
   def authorized_for_update?
     not deleted_at
  end

end

module TestsHelper

  ...
   
  def skip_action_link?(link, *args)
    (args.size > 0) and args[0].respond_to?(:deleted_at) and args[0].deleted_at and (link.crud_type!=:read)
  end

end

削除時にレコードが消えてしまう†

削除したレコードも表示して欲しいので、 削除成功時には行を消去する代わりに表示し直すことにします。

削除失敗時にはエラーメッセージを表示するため、普通に 'destroy' を呼びます。

app/controllers/tests_controller.rb

class TestsController < ApplicationController
  ...

  protected

  def destroy_respond_to_js
    if successful?
      render :action => 'update_row'
    else
      # エラーメッセージを表示する
      render :action => 'destroy'
    end
  end

end

削除済みのレコードに Restore アクションを追加する†

app/controllers/tests_controller.rb

class TestsController < ApplicationController
  active_scaffold :"test" do |conf|
    ...
 
    conf.action_links.add 'restore', :label => 'Restore', :type => :member, 
            :confirm => 'Are you sure you want to restore the record?', 
            :method => :put, :position => :false
  end
  ...

  def restore

  end
 
end

config/routes.rb

  resources :tests do 
    member do
      get 'restore'
    end
    as_routes
    resources :histories, controller: 'test/histories' do
      as_routes
    end
  end

app/helpers/tests_helper.rb

module TestsHelper
  ...

  def skip_action_link?(link, *args)
    return false unless (args.size > 0) and args[0].respond_to?(:deleted_at)
    
    return true if args[0].deleted_at and (link.action=='destroy' or link.action=='edit')
    return true if not args[0].deleted_at and (link.action=='restore')
  end
end

これで、削除済みのレコードにのみ Restore が表示されました。

TestsController::restore の中身は active_scaffold/lib/active_scaffold/actions/update.rb を参考にして、以下のようにすれば良いようでした。

app/controllers/tests_controller.rb

class TestsController < ApplicationController
  ...

  def restore
    do_restore
    respond_to_action(:restore)
  end
 
  protected

  def do_restore
    do_edit
    @record.deleted_at = nil
    update_save(no_record_param_update: true)
  end

  def restore_respond_to_js
    update_row
  end

  def destroy_respond_to_js
    # Update row instead of deleting
    update_row
  end

  def update_row
    if successful?
      render :action => 'update_row'
    else
      # エラーメッセージを表示する
      render :action => 'destroy'
    end
  end

end

これで、

• acts_as_paranoid による restore 可能な削除を実現
• 作成・編集・削除・リストアすべてを paper_trail で追跡
• 作成・編集・削除・リストア・履歴閲覧が ActiveScaffold 上で可能

になりました。

リストア時に uniqueness 制約に引っかかれば、ちゃんとエラーメッセージも表示されます。

想定する利用方法†

config/application.rb

    # lib 以下のモジュールを自動ロードする
    config.autoload_paths += %W(#{config.root}/lib)

app/models/some_model.rb

# coding:utf-8
class SomeModel < ActiveRecord::Base

  # 必要に応じて、paper_trail で無視する column を指定する
  # 指定しなければ [:id, :created_at, :updated_at] になる
  @paper_trail_ignore = [:id, :created_at, :updated_at, :some_other_column]

  # 論理削除＋履歴情報 を持たせる
  include TrailedParanoidScaffold::Model 

  # もし uniqueness が必要な場合には必ず scope: :deleted_at を付ける
  validates :some_unique_column, presence: true, uniqueness: {scope: :deleted_at}
 
end

SomeModel に act_as_paranoid 用のための deleted_at カラムを追加する

$ rails generate migration AddDeletedAtToSomeModels deleted_at:datetime:index
$ rake db:migrate

config/routes.rb

  resources :some_models do
    # TrailedParanoidScaffold 用のルート指定
    as_tpsf_routes
  end

app/controllers/some_models_controller.rb

# coding:utf-8
class SomeModelsController < ApplicationController
  # 上記の機能を取り込む
  include TrailedParanoidScaffold::Controller

  active_scaffold :"some_model" do |conf|
    # 追加で必要な設定があればここで行う
  end
  # 必要ならここでも
end

app/helpers/some_models_helper.rb

# coding:utf-8
module SomeModelsHelper
  # 上記の機能を取り込む
  include TrailedParanoidScaffold::Helper

  # 日付表示の設定などがあればここで行う
  def test_updated_at_column(record, column)
    record.updated_at ? record.updated_at.localtime.strftime('%m/%d') : ''
  end

end 

app/controllers/some_model/histories_controller.rb

# coding:utf-8
# ソースファイル名を元に自動でコントローラを設定する
# 親にしたいコントローラを第２引数として与えることができる
# 第２引数を省略すれば ApplicationController が親になる
class SomeModel::HistoriesController < ApplicationController
  # 上記の機能を取り込む
  include TrailedParanoidScaffold::HistoriesController

  active_scaffold :"some_model/history" do |conf|
    # 追加の設定があればここで
  end
  # 必要ならここでも
end

app/helpers/some_model/histories_helper.rb

# coding:utf-8
module SomeModel::HistoriesHelper
  # 上記の機能を取り込む
  include TrailedParanoidScaffold::HistoriesHelper

  # 日付表示の設定など必要があれば別途
  def some_model_history_created_at_column(record, column)
    record.created_at ? record.created_at.localtime.strftime('%m/%d %H:%M') : ''
  end
  
end

AdminController の下で動かす場合†

この機能を使うのは主に管理者画面だと思うので、

• /admin/some_models
• /admin/another_models

のように、admin パスの下で動かす需要も多いかもしれません。

その場合には以下のようにします。

namespace 'admin' do
  resources :some_models do 
    as_tpsf_routes
  end
  resources :another_models do
    as_tpsf_routes
  end
end

class AdminController < ApplicationController
  # 管理者のみがアクセス可能なようにする
  before_filter :assert_admin
end

削除済みレコードを淡色表示†

削除済みレコードを見やすく（見にくく）するために、 ApplicationHelper に column_attributes を仕込み、

app/controllers/application_controller.rb

module ApplicationHelper
  ...

  def column_attributes(column, record)
    if record.respond_to?(:deleted_at) and record.deleted_at
      super
      column.css_class << "record_deleted"
#      { :record_deleted => true }.merge(super)  # attribute を設定する場合
    else
      super
    end
  end
  ...

end

css で薄くできました。

app/assets/stylesheets/application.css.less

div.as_content table {

  .record_deleted {
    opacity: 0.3;
  }

/* attribute を設定する場合
  [record_deleted="true"] {
    opacity: 0.3;
  }
*/
}

ただこれだと、データ管理画面以外で active_scaffold を使っていると バッティングする可能性があります。

例えば AdminHelper に column_attributes を置いて、 取り込んで使えればよいと思うのですが、 include などで取り込んだ際の method_missing の動作を良く理解していないため、 正しいやり方を見つけられていません。

子要素の数でソートする†

has_many: subitems を持つモデルについて、list の表示を subitems.count でソートしたい場合、 sort_by の method オプションを使えば次のように簡単にできるのだけれど、

  active_scaffold tpsf.model_id do |conf|
    conf.columns[:subitems].sort_by :method => 'subitems.count'
  end

これでは無駄なクエリが乱発されるので、大きなテーブルに対してはパフォーマンスに問題が生じそうです。

これを１回の sql 発行で済ますには、 beginning_of_chain をいじって LEFT OUTER JOIN で subitem 数を求める方法があるようです。

app/controllers/some_models_controller.rb

TrailedParanoidScaffold::build(__FILE__, AdminController) do

  def beginning_of_chain
    # subitems_count というカラムを追加しておく
   super().unscoped.joins(:presentations).select('accounts.*').
           select('COUNT(presentations.id) AS presentations_count')
  end

  active_scaffold @active_scaffold_model do |conf|
    # 追加した subitems_count でソートする
    conf.columns[:subitems].sort_by :sql => 'subitems_count'
  end

end

同様に、beginning_of_chain にて .select を使えば、 テーブルにはないカラムを好きなだけ追加できるので、 いろいろ凝ったこともできそうです。

削除済みレコードから親レコードへのリンクを表示しない†

削除済み deleted_at!=NULL のレコードから belongs_to 経由で親へのリンクをたどろうとすると、 自身がすでに削除されていることから、親レコードが見付からないというエラーになってしまいます。

そこで以下のように、削除済みレコードからは親へのリンクを張らないようにしました。 削除されていなければ、表示用のリンクを張ります。

app/helpers/subitems_helper.rb

 def subitem_parent_model_column do |rec, col|
   if rec.deleted_at
     # 一時的に無効にする良い方法が分からなかったため
     # このようにしている
     col.set_link ->(dummy){ nil }
   else
     col.set_link :show
#     col.set_link :edit  # 編集リンクを張るならこちら
   end

   rec.to_s # 表示内容
 end

リンクをしない設定はこれで正しいんだろうか？
ソースコード を読んでも、どうするのが正式なのかが分からない？？？

histories のリンクが機能していない†

500 Internal Server Error が出てしまいました。

Processing by Admin::Account::HistoriesController#index as JS
  Parameters: {"association"=>"histories", "parent_scaffold"=>"admin/accounts", "adapter"=>"_list_inline_adapter", "account_id"=>"1"}
  Account Load (0.3ms)  SELECT  "accounts".* FROM "accounts" WHERE "accounts"."deleted_at" IS NULL AND "accounts"."id" = ? LIMIT ?  [["id", 1], ["LIMIT", 1]]
  Account Load (0.2ms)  SELECT  "accounts".* FROM "accounts" WHERE "accounts"."id" = ? LIMIT ?  [["id", 1], ["LIMIT", 1]]
Completed 500 Internal Server Error in 6ms (ActiveRecord: 0.5ms)

NoMethodError (undefined method `[]' for nil:NilClass):

/home/takeuchi/.rvm/gems/ruby-2.4.0@ICSPM25/bundler/gems/active_scaffold-ab5840e57e91/lib/active_scaffold/data_structures/nested_info.rb:109:in `default_sorting'
/home/takeuchi/.rvm/gems/ruby-2.4.0@ICSPM25/bundler/gems/active_scaffold-ab5840e57e91/lib/active_scaffold/data_structures/nested_info.rb:102:in `sorted?'
/home/takeuchi/.rvm/gems/ruby-2.4.0@ICSPM25/bundler/gems/active_scaffold-ab5840e57e91/lib/active_scaffold/actions/nested.rb:39:in `configure_nested'
activesupport (5.1.2) lib/active_support/callbacks.rb:413:in `block in make_lambda'
activesupport (5.1.2) lib/active_support/callbacks.rb:197:in `block (2 levels) in halting'
...

https://github.com/osamutake/active_scaffold/blob/master/lib/active_scaffold/data_structures/nested_info.rb#L109

    def sorted?(chain)
      default_sorting(chain).present?
    end

    def default_sorting(chain)
      return @default_sorting if defined? @default_sorting
      if association.scope.is_a?(Proc) && chain.respond_to?(:values)
        @default_sorting = chain.values[:order]
        @default_sorting = @default_sorting.map(&:to_sql) if @default_sorting[0].is_a? Arel::Nodes::Node
        @default_sorting = @default_sorting.join(', ')
      end
    end

の真ん中の @default_sorting[0].is_a? Arel::Nodes::Node で、 @default_sorting が nil になって失敗しているみたい

上記の chain は

https://github.com/osamutake/active_scaffold/blob/master/lib/active_scaffold/actions/nested.rb#L39

によると

https://github.com/osamutake/active_scaffold/blob/master/lib/active_scaffold/actions/nested.rb#L82

の beginning_of_chain で作られているのだけれど、 必ずしも :order が設定されていない可能性がある、ということなのかも？

byebug 仕込んで見てみると、

 LANG:console
[103, 112] in /home/takeuchi/.rvm/gems/ruby-2.4.0@ICSPM25/bundler/gems/active_scaffold-ab5840e57e91/lib/active_scaffold/data_structures/nested_info.rb
   103:     end
   104:
   105:     def default_sorting(chain)
   106:       return @default_sorting if defined? @default_sorting
   107: byebug
=> 108:       if association.scope.is_a?(Proc) && chain.respond_to?(:values)
   109:         @default_sorting = chain.values[:order]
   110:         @default_sorting = @default_sorting.map(&:to_sql) if @default_sorting[0].is_a? Arel::Nodes::Node
   111:         @default_sorting = @default_sorting.join(', ')
   112:       end
(byebug) chain
 CACHE Account::History Load (0.0ms)  SELECT  "versions".* FROM "versions" WHERE "versions"."item_id" = ? AND (item_type = "Account") LIMIT ?  [["item_id", 1], ["LIMIT", 11]]

#<ActiveRecord::Associations::CollectionProxy [#<Account::History id: 1, item_type: "Account", item_id: 1, event: "create", whodunnit: nil, object: nil, created_at: "2017-07-13 00:39:52", object_changes: "---\nemail:\n- \n- takeuchi@bk.tsukuba.ac.jp\npassword...", transaction_id: 1>]>

なので、確かに ORDER BY 節が存在しない。

えーと・・・これ、以前のバージョンだとどうして動いてたんだ？？？

あれ、ruby や Rails のバージョン違いじゃなくて、 active_scaffold 自体のバージョンが上がっている問題なのかも？？？

以前使っていた active_scaffold (3.4.39) だと、

https://github.com/activescaffold/active_scaffold/blob/5c5c18ccaf28ef7ab6a4f0564912bb4e0fbb8e4f/lib/active_scaffold/data_structures/nested_info.rb#L121

のようになっていて、全然違うコードだ。

# TODO: remove when rails 3 compatibility is removed

というコメントが付いてある部分を何とかしようとして、バグが混入したのかもしれない？

たぶん、以下のようにすれば良さそう。

    def default_sorting(chain)
      return @default_sorting if defined? @default_sorting
-     if association.scope.is_a?(Proc) && chain.respond_to?(:values)
+     if association.scope.is_a?(Proc) && chain.respond_to?(:values) && chain.values[:order]
        @default_sorting = chain.values[:order]
        @default_sorting = @default_sorting.map(&:to_sql) if @default_sorting[0].is_a? Arel::Nodes::Node
        @default_sorting = @default_sorting.join(', ')
      end
    end

とりあえず、それらしく動くことを確認。

これでいいのかどうかよく分からないけど、一応、報告してみました。
https://github.com/activescaffold/active_scaffold/issues/560

edit → update で失敗する†

まず、なぜか更新ボタンを１回押しただけで同時に２つのリクエストが飛んでいる。

Started PATCH "/event/ICSPM25/ja/admin/invited_speakers/13?utf8=%E2%9C%93" for 127.0.0.1 at 2017-07-21 11:35:30 +0900
Started PATCH "/event/ICSPM25/ja/admin/invited_speakers/13?utf8=%E2%9C%93" for 127.0.0.1 at 2017-07-21 11:35:30 +0900
Processing by Admin::InvitedSpeakersController#update as JS
Processing by Admin::InvitedSpeakersController#update as JS

form の submit に jquery_ujs のイベント

function(e) {
  var form = $(this),
    remote = rails.isRemote(form),
    blankRequiredInputs,
    nonBlankFileInputs;

  if (!rails.allowAction(form)) return rails.stopEverything(e);

  // Skip other logic when required values are missing or file upload is present
  if (form.attr('novalidate') === undefined) {
    if (form.data('ujs:formnovalidate-button') === undefined) {
      blankRequiredInputs = rails.blankInputs(form, rails.requiredInputSelector, false);
      if (blankRequiredInputs && rails.fire(form, 'ajax:aborted:required', [blankRequiredInputs])) {
        return rails.stopEverything(e);
      }
    } else {
      // Clear the formnovalidate in case the next button click is not on a formnovalidate button
      // Not strictly necessary to do here, since it is also reset on each button click, but just to be certain
      form.data('ujs:formnovalidate-button', undefined);
    }
  }

  if (remote) {
    nonBlankFileInputs = rails.nonBlankInputs(form, rails.fileInputSelector);
    if (nonBlankFileInputs) {
      // Slight timeout so that the submit button gets properly serialized
      // (make it easy for event handler to serialize form without disabled values)
      setTimeout(function() {
        rails.disableFormElements(form);
      }, 13);
      var aborted = rails.fire(form, 'ajax:aborted:file', [nonBlankFileInputs]);

      // Re-enable form elements if event bindings return false (canceling normal form submission)
      if (!aborted) {
        setTimeout(function() {
          rails.enableFormElements(form);
        }, 13);
      }

      return aborted;
    }

    rails.handleRemote(form);
    return false;

  } else {
    // Slight timeout so that the submit button gets properly serialized
    setTimeout(function() {
      rails.disableFormElements(form);
    }, 13);
  }
}

form の 更新ボタン の click に jquery_ujs の

function(event) {
  var button = $(this);

  if (!rails.allowAction(button)) return rails.stopEverything(event);

  // Register the pressed submit button
  var name = button.attr('name'),
    data = name ? {
      name: name,
      value: button.val()
    } : null;

  var form = button.closest('form');
  if (form.length === 0) {
    form = $('#' + button.attr('form'));
  }
  form.data('ujs:submit-button', data);

  // Save attributes from button
  form.data('ujs:formnovalidate-button', button.attr('formnovalidate'));
  form.data('ujs:submit-button-formaction', button.attr('formaction'));
  form.data('ujs:submit-button-formmethod', button.attr('formmethod'));
}

あれ？jquery_ujs と rails-ujs ってのが両方入ってるけど、 これっていいのか？

http://qiita.com/itkrt2y/items/7e999836f460fb9c005d

jquery_ujs を削ったらそもそも form が表示されなくなってしまった。

rails-ujs を削ったら正しく動いた。

えーと・・・どうなってるんだっけ？

Gemfile を見直す必要がありそう。

モデルの作成†

$ rails g active_scaffold:resource Test name:string email:string
 Running via Spring preloader in process 13486
       invoke  active_record
       create    db/migrate/20171110143107_create_tests.rb
       create    app/models/test.rb
       invoke    test_unit
       create      test/models/test_test.rb
       create      test/fixtures/tests.yml
       route  resources :tests, concerns: :active_scaffold
       invoke  active_scaffold:controller
       create    app/controllers/tests_controller.rb
       create    app/helpers/tests_helper.rb
       invoke    test_unit
       create      test/controllers/tests_controller_test.rb
       create    app/views/tests
$ rake db:migrate
 == 20171110143107 CreateTests: migrating ======================================
 -- create_table(:tests)
    -> 0.0052s
 == 20171110143107 CreateTests: migrated (0.0056s) =============================
$ git commit -a -m "active_scaffold を master ブランチの最新版にした"
$ rails s 

http://localhost:3000/tests

へのアクセスで一覧画面が出た。

Create New でエラーが出る†

[Create New] を押したら、

Uncaught SyntaxError: Unexpected token <
    at processResponse (rails-ujs.self-817d9a8cb641f7125060cb18fefada3f35339170767c4e003105f92d4c204e39.js?body=1:246)
    at rails-ujs.self-817d9a8cb641f7125060cb18fefada3f35339170767c4e003105f92d4c204e39.js?body=1:173
    at XMLHttpRequest.xhr.onreadystatechange (rails-ujs.self-817d9a8cb641f7125060cb18fefada3f35339170767c4e003105f92d4c204e39.js?body=1:230)

エラーが出たのは、

         } else if (type.match(/\b(?:java|ecma)script\b/)) {
           script = document.createElement('script');
           script.text = response;
           document.head.appendChild(script).parentNode.removeChild(script);

appendChild の所なので、javascript を期待してたのに html が帰ってきてしまったという感じ。

えー、そんなぁ。

javascript の require を直す†

$ jed app/assets/javascripts/application.js
 - // require rails-ujs
 + //# require rails-ujs
 + //= require jquery
 + //= require jquery_ujs
 + //= require active_scaffold
$ git commit -a -m "active_scaffold が動くように application.js の require を修正"

Create New が動くようになった。

時刻の表示方法を設定†

https://ja.stackoverflow.com/questions/8161/created-at%E3%81%AE%E3%82%BF%E3%82%A4%E3%83%A0%E3%82%BE%E3%83%BC%E3%83%B3%E3%81%A8%E3%83%95%E3%82%A9%E3%83%BC%E3%83%9E%E3%83%83%E3%83%88%E3%82%92%E5%A4%89%E6%8F%9B%E3%81%99%E3%82%8B を参考に

config/application.rb

 　module Album
 　  class Application < Rails::Application
       ...
  
+　    config.time_zone = 'Tokyo'
+　    config.i18n.default_locale = :ja
 　  end
 　end

config/locales/ja.yml

+ ja:
+   time:
+     formats:
+       default: ! '%Y/%m/%d %H:%M:%S'