なるようになるブログ

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

rails commit log流し読み(2020/06/21)

2020/06/21分のコミットです。

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


Remove reference to rendering absolute files

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

現在出来ないアプリの外側にあるファイルの参照方法についての言及している箇所が残っていたのを修正しています。


AV Helper guide fixes- (#39679)

rails guideのAction View Helpersの修正です。

exampleの実行結果からスペースを削除、略語を使っている箇所を正式名称を使用するよう修正、等を行っています。


move code example right after colon

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

patchメソッドについて説明している箇所について、exampleコードの後に説明を行うよう修正しています。


Fix code example in upgrading guide [ci skip] (#39682)

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

patchメソッドのexampleコードが実行エラーになってしまうコードになっていたのを修正しています。


Remove less interested where_clause_factory

activerecord/lib/active_record/relation/query_methods.rbの修正です。

where clauseを構築するのを専用のクラス(WhereClauseFactory)で行っていたのを、同等の処理をbuild_where_clauseメソッド内で直接行うようにしてクラスは使用しないよう修正しています。


Restore referencing BaseHandler in legacy_relation.dump

先のコミットの際に誤って関係無いファイル(legacy_relation.dump)を削除してしまっていたのを戻しています。


Implement equality? rather than inheriting Equality for In node

activerecord/lib/arel/nodes/in.rbの修正です。

In nodeがEqualityを継承していたのをやめて、独自にequality?メソッドを定義するよう修正しています。

元々は、Equalityを継承する事でequalityかどうかをチェック出来る、というようになっていました。しかし現在は例外となるクラスがあり、そもそもIn nodeが何故Equalityを継承しているかが明確ではない、というような理由もあった為、継承はやめてequality?メソッドを直接定義するよう修正しています。


Move set_owner_attributes into ForeignAssociation

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

ForeignAssociationでだけ使用しているset_owner_attributesメソッドをForeignAssociation moduleに移動しています。


Move belongs_to's association_primary_key into BelongsToReflection

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

AssociationReflectionクラスに定義されていたassociation_primary_keyメソッドをBelongsToReflectionクラスに移動しています。

AssociationReflectionに定義されているassociation keyメソッドはhas_manyhas_one associationの為のメソッドが殆どで、association_primary_keyだけがbelongs_to用、という状態になっていました。これは紛らわしいという事で、association_primary_keyメソッドをBelongsToReflectionに移動しています。


[ci skip] Increase info on validate presence for association

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

presence validationについて説明している箇所に、associationがpresentかつvalidである事をチェックしたいなら、validates_associatedを使うよう説明を追加しています。