なるようになるブログ

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

rails commit log流し読み(2016/10/29)

2016/10/29分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activerecord/CHANGELOG.md


Hide commands from API site.

railtiesのdocの修正です。

各command用のクラス(e.g. ApplicationCommandConsoleCommand等)に:nodoc:を設定し、API docに表示されないようにしています。


Merge pull request #23698 from meinac/add_missing_types_into_ar_model_generator_usage

railties/lib/rails/generators/rails/model/USAGEの修正です。

USAGEに表示する使用出来るfieldの一覧をべたに書いていたのを、DBから取得(ActiveRecord::Base.connection.native_database_typesを使用)するよう修正しています。

が、generator実行時点でDBに接続するのはよろしくないので、後ほどrevertされています。


Merge PR #19759

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

partial_writesをfalseにしていた場合に、has_and_belongs_to_many、及びhas_many_through associationのrecordの保存に失敗するバグがあったのを修正しています。


Fix the variable scoping issue I introduced in 007e50d8e5a900547471b6c4ec79d9d217682c5d

activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rbの修正です。

Permit loads while queries are runningの対応の際に、ActiveRecord::Resultインスタンス生成を行うscopeが、本来行っていたscopeとずれてしまったのを修正しています。


Revert "Merge pull request #23698 from meinac/add_missing_types_into_ar_model_generator_usage"

USAGEに表示する使用出来るfieldの一覧をDBから取得するようにした、Merge pull request #23698 from meinac/add_missing_types_into_ar_model_generator_usage をrevertしています。理由は先に書いた通り。


Merge pull request #26926 from gsamokovarov/debug-exceptions-plain-loggers

actionpack/lib/action_dispatch/middleware/debug_exceptions.rbの修正です。

ActionDispatch::DebugExceptions::DebugView#renderメソッドで、logger(`ActionView::Base.logger)のsilenceメソッドを使用する前に、silenceメソッドがloggerに定義されているかどうか確認するよう修正しています。

Ruby標準のLoggerクラスのように、loggerにsilenceメソッドが定義されていない場合があるため。


Add more rubocop rules about whitespaces

.rubocop.ymlにwhitespaceに関するルールの追加(Style/SpaceAfterColonStyle/SpaceAfterCommaStyle/SpaceAroundEqualsInParameterDefaultStyle/SpaceAroundKeywordStyle/SpaceAroundOperatorsStyle/SpaceBeforeFirstArgStyle/SpaceInsideParens)、及び各ファイルにそのルールの適用を行っています。