なるようになるブログ

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

rails commit log流し読み(2015/06/22)

2015/06/22分のコミットです。

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

activerecord/CHANGELOG.md


Remove duplicate private statement

actionview/lib/action_view/renderer/partial_renderer.rb

重複してあったprivate句を削除しています。


remove unused variable in reporter test

railties/test/test_unit/reporter_test.rbの修正です。

使用していない変数を削除しています。


Fix the documentation about ActionController::ParameterMissing [ci skip]

actionpack/lib/action_controller/metal/strong_parameters.rbのdocの修正です。

ActionController::ParameterMissingが発生する条件の説明に誤りがあったのを修正しています。


Fix the documentation about ActionController::UnpermittedParameters [ci skip]

actionpack/lib/action_controller/metal/strong_parameters.rbのdocの修正です。

ActionController::UnpermittedParametersが発生する条件の説明に、action_on_unpermitted_parameters:raiseだった場合に発生する旨説明を追加しています。


Merge pull request #20656 from repinel/move-validations-helper-methods

activemodel/lib/active_model/validations/with.rbの修正です。

activemodel/lib/active_model/validations/with.rb内に定義されてあったActiveModel::Validations::HelperMethods moduleを、専用のファイル(activemodel/lib/active_model/validations/helper_methods.rb)に切り出しています。


AR absence validator respects marked_for_destruction?. Closes #20449.

ActiveRecordの修正です。

ActiveRecordのabsence validatorが、marked_for_destruction?を参照し、marked_for_destruction?が設定されている場合validationを行わないよう修正しています。

元々absence validatiorがmarked_for_destruction?を参照していなかったので、associationにvalidates_absence_ofをvalidationを使用している場合に、_destroyにtrueが設定されていても、validationでエラーになる、というバグがあった為、その対応の為に修正したとの事です。


docs, update Active Model reference for AR length validator. [ci skip]

activerecord/lib/active_record/validations/length.rbのdocの修正です。

LengthValidator#validates_length_ofメソッドにdocを追加しています。


refactor, don't duplicate presence validator logic.

activerecord/lib/active_record/validations/presence.rbリファクタリングです。

PresenceValidatorから重複しているバリデーションロジックを削除しています。


docs, :scissors: wrongly placed heading. [ci skip]

activemodel/lib/active_model/validations/length.rbのdocの修正です。

不要なヘッダー用のdocを削除しています。