なるようになるブログ

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

rails commit log流し読み(2017/08/24)

2017/08/24分のコミットです。

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

actionview/CHANGELOG.md

activerecord/CHANGELOG.md

Merge pull request #30213 from ckundo/ccundiff-alt-text-default

Action Viewの修正です。

imageタグを生成する際、デフォルトではalt属性を生成しないよう修正しています。

いまだと、例えばimage_tag "logo.png"を呼び出すと、下記のようなHTMLが生成されます。

<img src="/images/logo.png" alt="Logo" />

上記の場合、スクリーンリーダー(コンピュータの画面読み上げソフトウェア)は、"Logo"というテキストをを読み上げます。しかしこれは実際は何も意味もありません(logo.pngが会社のロゴなら、その会社名が読み上げらないと意味が無い)。

というわけで、デフォルトで生成されるalt属性には意味がない、どころが邪魔になってしまっている、という事でデフォルトではalt属性を生成しないようにしたとの事です。合わせて、image_altメソッドをdeprecateにしています。

詳細は説明はalt text generated from filename can be harmful for screen reader users を参照。

Remove alt generation from image_submit_tag

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

先の対応同様、image_submit_tagメソッドでもデフォルトではalt属性を生成しないよう修正しています。

Make sure image_alt is deprecated

actionview/test/template/asset_tag_helper_test.rbの修正です。

image_altメソッドを呼び出したらdeprecateメッセージが表示される事を確認するようにしています。

Merge pull request #30366 from MrOutis/action-cable-env-var

railties/lib/rails/generators/rails/app/templates/config/cable.ymlの修正です。

redisのURLを、ENVから取得するようデフォルトのテンプレートを修正しています。

production:
   adapter: redis
-  url: redis://localhost:6379/1
+  url: <%%= ENV.fetch("REDIS_URL") { "redis://localhost:6379/1" } %>
   channel_prefix: <%= app_name %>_production

Fix broken link for Java xml [ci skip]

activesupport/lib/active_support/xml_mini/jdom.rbのdocの修正です。

Java xmlについてのリンク先ページがもう無くなってしまっていたので、代わりにarchive(https://archive.is/9xcQQ)へのリンクに修正しています。

Remove alt text from image_tag example [ci skip]

actionview/lib/action_view/helpers/asset_tag_helper.rb、 actionview/lib/action_view/helpers/asset_url_helper.rbのdocの修正です。

image_tagの実行例からalt属性を削除しています。

[ci skip]Revert commits changing wrong place

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

Options for belongs_toの項、:dependentオプションについて説明している箇所に、belongs_toでは使えないオプション(has_many じゃないと使えない)が記載されいたのを削除しています。

Update azure-core to 0.1.11 to fix rails-dev-box

Gemfile.lockの修正です。

azure-core gemのバージョンを0.1.11に更新しています。

0.1.10だとrails-dev-boxでgemのインストールに失敗する、という問題があるため、との事です。

参考：Changed ‘nokogiri’ to runtime dependency with version >= 1.6, and add…

Update active_support_core_extensions.md [ci skip]

rails guideのActive Support Core Extensionsの修正です。

オプション、メソッド名の+で囲んでいたんのを、バッククォートで囲むよう修正しています。guideはマークダウンな為。

Fix indentation [ci skip]

rails guideの修正です。

各guide内のコード例のインデントを調整しています。

Remove unnecessary fixture loading

activerecord/test/cases/relation/delegation_test.rbの修正です。

使用していないfixtureのloadingがあったのを削除しています。

Merge pull request #30337 from kamipo/refactor_schema_dumper

Active Recordの修正です。

deprecateになっていたColumnDumper#migration_keysメソッドの削除、及び、SchemaDumperをadapter毎にカスタマイズ出来るようリファクタリングを行っています。

元々SchemaDumperはcolumn optionしかカスタマイズしか出来ないようになっていました。しかし、oracle-enhancedのような3rd partyのconnection adapterではtableやindexのdump処理のカスタマイズがしたい事がある為、各connection adapterでカスタマイズしたSchemaDumperを使用出来るようリファクタリングしています。

create_schema_dumperメソッドを使用してSchemaDumperを取得するようになっており、create_schema_dumperメソッドでカスタマイズしたSchemaDumperクラスを返せばよいようになっています。

参考：

Remove unused returning value stream

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

SchemaDumper#tableメソッドから使用していない戻り値(stream)を削除しています。

Widen blob size column

activestorage/db/migrate/20170806125915_create_active_storage_tables.rbの修正です。

active_storage_blobsテーブルのbyte_sizeカラムの型をintegerからbigintに変更しています。