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

AbstractMysqlAdapter#translate_exceptionメソッドでmysql2固有のエラーに関する処理があったのを、mysql2固有のエラーについてはMysql2Adapterクラスで行うよう修正しています。

2019-08-29

rails commit log流し読み(2019/08/28)

2019/08/28分のコミットです。

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

activesupport/CHANGELOG.md

Prevent ActiveSupport::Duration.build(value) from creating instances of ActiveSupport::Duration unless value is of type Numeric.

Disallow all non-numerics as Duration.build input, Fixes #37012

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

ActiveSupport::Duration.buildの引数にNumeric以外の値を指定した場合TypeErrorをraiseするよう修正しています。

元々Stringを指定してもDurationのインスタンスを生成出来るようになっていたのですが、Stringを指定した場合のインスタンスはNumericを指定した場合のインスタンスとの比較処理が正しく行えない、という問題がありました。

それは振る舞いとしてよくないよねという話になり、という事でそもそもNumeric以外の値は指定出来ないよう修正されました。

Changed controller name (Typo fix) in rails engines documentation [ci skip]

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

Using a Class Provided by the Applicationの項のexampleコードのコントローラ名をタイポしていたのを修正しています。

Add doc about I18n support for :placeholder option [ci skip]

actionview/lib/action_view/helpers/form_tag_helper.rbのdocの修正です。

text_field_tagメソッドのdocに:placeholderオプションについての説明を追加しています。

Fix a document to update postgresql version for rails6 [ci skip]

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

Rails with Databases and SCMの項にある、rails newで生成されるconfig/database.ymlの内容が実際の内容と異なっていたのを修正しています。

Clarify severe security issues maintenance policy [ci skip]

rails guideのMaintenance Policy for Ruby on Railsの修正です。

Severe Security Issuesの項の説明の言い回しを修正しています。

Update stale issue comment to mention 6-0-stable

.github/stale.ymlの修正です。

stale botがコメントする、まだissueが再現するかどうかを確認するブランチを5-2-stableから6-0-stableに修正しています。

Merge pull request #37047 from dvandersluis/37044-fix-exception-message

actionpack/lib/action_controller/metal/exceptions.rb、 activerecord/lib/active_record/associations.rbの修正です。

MethodNotAllowed、HasManyThroughSourceAssociationNotFoundError exceptionでエラーメッセージを構築する際に、to_sentenceメソッドでen localeを強制的に使用していたのを、使用しないよう修正しています。

en localeの設定が無い環境でエラー(I18n::InvalidLocale`)になってしまう為。

2019-08-28

rails commit log流し読み(2019/08/27)

2019/08/27分のコミットです。

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

actionmailer/CHANGELOG.md

Added email_address_with_name to properly escape addresses with names.

activesupport/CHANGELOG.md

Add ActiveSupport::Cache::Store#delete_multi method to delete multiple keys from the cache store.

Escape email addresses with name

actionmailer/lib/action_mailer/base.rbの修正です。

名前付きのemailを作成する為のemail_address_with_nameメソッドを追加しています・

email_address_with_name("sunny@example.org", "Sunny") #=> "Sunny <sunny@example.com>"

Add delete_multi method

activesupport/lib/active_support/cache.rb、 activesupport/lib/active_support/cache/redis_cache_store.rbの修正です。

cacheから複数データをまとめて削除する為のdelete_multiメソッドを追加しています。

Reduce Hash creation in FormHelper

actionview/lib/action_view/helpers/form_helper.rbの修正です。

FormHelper#objectify_optionsでmergeメソッドを使用するのを避けて、Hashオブジェクトの生成を減らすようにしています。

No need to count an Array because Array knows its length

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

SafeBuffer#[]=メソッドで引数の数を数えるのにcountを使用していたのをlengthを使用するよう修正し、不要なループが実行されないよう修正していｍす。

Skip tidying bytes for ascii_only strings

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

Unicode#tidy_bytesメソッドでStringがASCIIだけの場合不要な変換処理を行わないよう修正しています。

No need to dup && freeze an already frozen sql string

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

DatabaseStatements#to_sql_and_bindsメソッドでSQL Stringがfreezeな場合、不要なdup + freeze処理を実行しないよう修正しています。

Giving a Symbol here might be a very little bit efficient than giving a String

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

connection handlerを保持するスレッドローカル変数に使用するkeyをStringからSymbolに変更しています。keyは後の処理でSymbolに変更される為、最初からSymbolを使用した方が不要な変換が行われない為。

Fix rdoc of ActiveRecord::Associations::CollectionProxy#replace [ci skip]

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

ActiveRecord::Associations::CollectionProxy#replaceメソッドのdoc内のexampleコードで、括弧が不足している箇所があったのを修正しています。

Fix assertion in test_delete_multi

activesupport/test/cache/behaviors/cache_store_behavior.rbの修正です。

delete_multiメソッドのテストでチェックするkey名に誤りがあったのを修正しています。

Ignore SQLite3 database files generated by parallel testing

gitignoreの対象にparallel testingで生成するSQLite3のDBファイル(sqlite3- + 数字)を追加しています。

2019-08-27

rails commit log流し読み(2019/08/26)

2019/08/26分のコミットです。

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

Merge pull request #28862 from georgemillo/show_detailed_exceptions

rails guideのConfiguring Rails Applicationsの修正です。

config.consider_all_requests_localについて説明している箇所に、より細かな制御が必要な場合、コントローラでshow_detailed_exceptions?を実装する必要がある旨説明を追加しています。

Always load validator class to verify it exists

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

optionの内容に関わらずvalidation classを必ずロードするよう修正しています。不正なvalidationが指定された場合に気付けるようにする(ロード時にエラーになるようにする)為。

Fix docs to replace http with https in activerecord [ci skip]

Active Recordのdocの修正です。

外部リンクへのURLをhttp -> httpsに修正しています。

なるようになるブログ

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

rails commit log流し読み(2019/09/01)

rails commit log流し読み(2019/08/31)

rails commit log流し読み(2019/08/30)

rails commit log流し読み(2019/08/29)

rails commit log流し読み(2019/08/28)

rails commit log流し読み(2019/08/27)

rails commit log流し読み(2019/08/26)