なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2015/12/21)

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


mysql2 adapter instead of mysql [ci skip]

各docの修正です。

各docから、mysql adapterについての記載を削除しています。


Upgrade Guide: Add missing period, :scissors: whitespace [skip ci]

rails guideのA Guide for Upgrading Ruby on Railsの修正です。

終端にピリオドが無い箇所があったのを追加、及び不要な空白の削除を行っています。


Add missing @claudiob credit to change log [skip ci]

activemodel/CHANGELOG.mdの修正です。

callback chainの終了条件を変更した対応について、対応者の名前が記載されていなかったので追記しています。


Remove legacy mysql adapter

Active Recordの修正です。

rails本体から削除されたmysql adapterについてのdoc、テストが残っている箇所があったのをまとめて削除しています。


Update README extension in Getting Started guide

rails guideのGetting Started with Railsの修正です。

Creating the Blog Applicationの項、生成されるファイル一覧のREADMEの拡張子が古いまま(rdoc)になっていたのを、新しい形式(md)に修正しています。


Merge pull request #22706 from habermann24/fix_mattr_accessor_docs

activesupport/lib/active_support/core_ext/module/attribute_accessors.rbのdocの修正です。

#mattr_accessorのexampleを記載している箇所が、記載してあるexampleコードに問題があった(#newメソッドの呼び出しが必要なのが漏れていた、classmoduleを間違えていた等)のをまとめて修正しています。


Merge pull request #22486 from methyl/fix-includes-for-groupped-association

activerecord/lib/active_record/associations/preloader/association.rbの修正です。

preloading scoped associations + groupを使用した場合に、groupに指定した値が使用されない、という問題があったのを修正しています。

が、PostgreSQLでテストが壊れてしまった為、後ほどrevertされています。そもそも生成されるSQLがおかしくなってしまっている模様。


Merge pull request #22581 from hirocaster/fix-expect-sample-code https://github.com/rails/rails/commit/e65680ee746877c9c028658c6e3c8f684c857bc9 activesupport/lib/active_support/cache.rbのdocの修正です。

activesupport#Cache#fetchメソッドのdoc内のexampleコードが、記載されている出力通りになっていなかった(値の表示タイミングがおかしかった)のを、正しく動作するよう修正しています。


No more no changes entries in the CHANGELOGs

CHANGELOGの修正です。

CHANGELOGの先頭にあった不要なNo changes.の文字を削除しています。


[ci skip] rails new project generates README.md now

rails guideのThe Rails Command Lineの修正です。

生成されるファイル一覧のREADMEの拡張子が古いまま(rdoc)になっていたのを、新しい形式(md)に修正しています。


Revert "Merge pull request #22486 from methyl/fix-includes-for-groupped-association"

preloading scoped associations + groupを使用した場合にgroupに指定した値が使用されない問題があったのを修正した、Merge pull request #22486 from methyl/fix-includes-for-groupped-associationをrevertしています。


Merge pull request #22698 from y-yagi/display_detail_info_on_inline_report

railties/lib/rails/test_unit/reporter.rbの修正です。

rails test runnerで、inlineでfailuresメッセージを表示する際、テストクラス名及びメソッド名を表示するよう修正しています。

# before
Failed assertion, no message given.

bin/rails test test/models/user_test.rb:5
# after
Failure:
UserTest#test_the_truth:
Failed assertion, no message given.

bin/rails test test/models/user_test.rb:5

Use CGI.escapeHTML for html escape https://github.com/rails/rails/commit/51152fc0f8517b24af4a619faa9df9879920f5d1

activesupport/lib/active_support/core_ext/string/output_safety.rbの修正です。

html escapeにString#gsubを使用していたのを、CGI.escapeHTMLを使用するよう修正しています。

-        ActiveSupport::Multibyte::Unicode.tidy_bytes(s).gsub(HTML_ESCAPE_REGEXP, HTML_ESCAPE)
+        CGI.escapeHTML(ActiveSupport::Multibyte::Unicode.tidy_bytes(s))

Ruby 2.3では、gsubよりCGI.escapeHTMLの方が7倍位早くなっている為、との事です。参考:Optimize CGI.escapeHTML for ASCII-compatible encodings by k0kubun · Pull Request #1164 · ruby/ruby。 すごい。


Merge pull request #22692 from ryohashimoto/22669_api_generator

actioncable/lib/rails/generators/channel/channel_generator.rbの修正です。

assets 生成の設定がfalseになっている場合、Action Cableのchannel用JSファイルを生成しないよう修正しています。


Merge pull request #22717 from nning/actioncable-readme-fixes

Action Cableのreadmeの修正です。

exampleコードで使用している変数、パス等に誤りがあったのを修正、及びaction_cable_meta_tagメソッドが設定されていた場合の挙動についての説明を追加しています。