なるようになるブログ

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

rails commit log流し読み(2018/10/17)

2018/10/17分のコミットです。

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


Refactored abstract MySQL adapter to support lazy version check.

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

AbstractMysqlAdapter#initializeで行っていたMySQLバージョンチェック処理をメソッド(check_version)に切り出しています。

サブクラスで処理をオーバーライド出来るようにする為、との事です。


Merge pull request #34220 from bogdanvlviv/follow-up-33571

activejob/lib/active_job/test_helper.rbのdocの修正です。

ActiveJob::TestHelper moduleの各メソッドのdocのグラマーの修正を行っています。


Fix mapping of content

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

Naming Conventionsの項、Model ClassとDatabase Tableについて説明しているリストの、説明の順番をその後にあるテーブルと同じ順になるよう修正しています。


Consistently extract checking version for all adapters

Active Recordの修正です。

Refactored abstract MySQL adapter to support lazy version check.で、MySQL adapterでだけDBのバージョンチェック処理をメソッドで行うようにしたのですが、他の全てのadapterでも同様にバージョンチェック処理を同名のメソッド(check_version)で行うよう修正しています。


Refactor Chars#reverse and Chars#grapheme_length

activesupport/lib/active_support/multibyte/chars.rbの修正です。

Chars#reverseChars#grapheme_lengthメソッドでgrapheme clustersを取得するのに\Xメタ文字を使用するよう取得しています。


Consolidate duplicated code that initializing an empty model object

activerecord/lib/active_record/core.rbactiverecord/lib/active_record/persistence.rbの修正です。

ActiveRecord::Core moduleのinit_withメソッド、init_from_dbメソッドの内容がほぼ同じだった為、init_withメソッドはinit_from_dbメソッドを呼び出すよう修正しています。

合わせて、init_from_dbメソッドをinit_with_attributesメソッドにリネームしています。引数に指定した値からmodel objectの初期化を行うメソッドなのですが、引数に指定する値はattributes objectであり、databasではない為。


Merge pull request #34233 from lucasprag/update_guide

rails guideのLayouts and Rendering in Railsの修正です。

Rendering Collectionsの項、_counter変数について説明している箇所に、asオプションを指定している場合、_counter変数も同様に名前が変わる旨説明を追加しています。


Merge pull request #32146 from abhikanojia/association_guide_fix

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

migrationファイルのexampleで、belongs_toメソッドにindex: trueが指定されていたのを削除しています。

belongs_toはデフォルトでindex: trueが指定されるようになっており、generatorが生成するmigrationファイルにindex: trueは指定されない為。


Remove and flip index: true for references in the doc [ci skip]

activerecord/lib/active_record/connection_adapters/abstract/schema_definitions.rbactiverecord/lib/active_record/connection_adapters/abstract/schema_statements.rbのdocの修正です。

references / belongs_toメソッドのexampleでから不要なindex: trueを削除、及び、indexが不要な場合のexampleにindex: falseを追加しています。


Merge pull request #34197 from schneems/schneems/symbol-hash-respond_to

Active Recordの修正です。

ActiveRecord#respond_to?メソッドで引数のnameをStringに変換して値のチェックをしていたのを、変換せずにチェック出来るようリファクタリングしています。不要なString Objectの生成を減らせるようにする為。