rails commit log流し読み(2016/01/18) - なるようになるブログ

2016/01/18分のコミットです。

CHANGELOGへの追加はありませんでした。

Spare to_sym call in ==.

actionview/lib/action_view/template/types.rbの修正です。

ActionView::Template::Types:Type#==メソッドから不要なsymbolへの変換処理を削除しています。

        def ==(type)
-          return false if type.blank?
-          symbol.to_sym == type.to_sym
+          @symbol == type.to_sym unless type.blank?
         end

initializeの時点で@symbolにto_symした値を保持している為。

Replace delegate calls with standard method defs.

actionview/lib/action_view/template/types.rbの修正です。

ActionView::Template::Types::Typeでdelegateメソッドでto_s、to_symをsymbolにdelegateしていたのを、defメソッドで該当のメソッドを呼び出すよう修正しています。

-        delegate :to_s, :to_sym, :to => :symbol
+        def to_s
+          @symbol.to_s
+        end
         alias to_str to_s

         def ref
           @symbol
         end
+        alias to_sym ref

to_symはrefのaliasとした為、delegateするメソッドが一つだけになってしまい、それなら通常通りdefで定義した方がわかりやすいだろう、という判断のようです。

Remove register abstraction.

actionview/lib/action_view/template/types.rbの修正です。

ActionView::Template::Types::Type.registerでmime typeの設定を行っていたのを、メソッドを削除し、変数作成時に値を設定しまうよう修正しています。

元々Action Pack経由で値を設定する為にメソッド経由で行っていたようなのですが、Action ViewとAction Packは分離しており、メソッドを使用する必要がなくなっている為、削除したようです。

Replace class attribute with SET constant.

actionview/lib/action_view/template/types.rbの修正です。

mime typesの保持にクラスアトリビュートを使用していたのを、定数を使用するよう修正しています。

Don't bother looking up the types.

actionview/lib/action_view/template/types.rbの修正です。

ActionView::Template::Types::Type.[]で引数がSymbolかどうか、及び値がmime typeがどうかのチェック処理をしていたのを削除しています。

仮に引数がSymbolで無かった場合、どちらにしろ値は設定されないので、チェックは不要だろう、という事のようです。

Enrich the SET constant to respond to symbols.

actionview/lib/action_view/template/types.rbの修正です。

ActionView::Template::Types::Type#symbolがMime::SET.symbols同様のsymbolの値を返すよう修正しています。

Store the symbols as an array.

actionview/lib/action_view/template/types.rbの修正です。

ActionView::Template::Types::Type#SET変数で、symbolをArrayとして保持するよう修正しています。Mime::SET.symbolsがArrayを返す為、それに合わせる為のようです。

Remove ActionView dependence on ActionPack's Mime implementation

Action Viewの修正です。

Mimeの参照にAction PackのMimeではなく、ActionView::Template::Typesを使用するよう修正しています。

Action ViewからAction Packの依存を削除する為の対応との事です。

Pass through AP CHANGELOG [ci skip]

actionpack/CHANGELOG.mdの修正です。

entry内でタイポしている箇所が複数あったのをまとめて修正しています。

Changed options for find_each and variants to have options start/finish instead of start_at/end_at based on comments

Active Recordの修正です。

ActiveRecord::Relation#in_batchesメソッド及び、#find_in_batchesメソッドのデータの開始 / 終了位置を渡すパラメータ名が、begin_at / end_at になっていたのを、それぞれ start/ finish に変更しています。

末尾に_atがついているのは、日を表す意味があるように思え、紛らわしい為修正したとの事です。

詳細はこちら：Add an option end to find_in_batches by vipulnsward · Pull Request #12257 · rails/rails

Explain how HStore columns can be queried [ci skip]

rails guideのActive Record and PostgreSQLの修正です。

HStoreカラムを使用した場合に#whereに指定するqueryのexampleを追加しています。

Updated the guides for having a distinct has_many through at the database level

rails guideのActive Record Associationsの修正です。

Scopes for has_manyの項、distinctオプションについて説明している箇所でuniqueオプションを使用したmigrationのexampleコードが、実際は動作しないコードになってしまっていた為、動作するコードに修正しています。

Merge pull request #23096 from HayleyCAnderson/ha-action-cable-docs

Action CableのREADMEの修正です。

全体の言い回しの修正、log_tagsオプションの設定方法についての説明を追加、等をまとめて行っています。

Merge pull request #23088 from prathamesh-sonpatki/update-gemfile-lock