なるようになるブログ

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

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

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

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


Add :nodoc: for ActiveRecord::AttributeMethods [ci skip]

Active Recordのdocの修正です。

ActiveRecord::AttributeMethods module配下のClassMethods moduleに:nodoc:を指定しています。module配下にpublic methodsが無く、API docにmodule名が表示されないようにする為に:nodoc:を指定したようです。


Enable links in Active Storage docs [ci skip]

activestorage/app/models/active_storage/filename.rbactivestorage/app/models/active_storage/variation.rbのdocの修正です。

メソッド名がリンクになるよう、doc内のメソッド名からエスケープ処理を削除しています。


Use typewriter in doc for Action Cable [ci skip]

actioncable/lib/action_cable/server/base.rbのdocの修正です。

ActionCable::Server::Base#disconnectメソッドのdoc内のidentifiersがタイプライター体で表示されるよう修正しています。


Use tt in doc for action_mailer [ci skip]

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

ActionMailer::Rescuable moduleのdoc内のrescue_fromがタイプライター体で表示されるよう修正しています。


Fix can't modify frozen String error in display_image

actionpack/lib/action_dispatch/system_testing/test_helpers/screenshot_helper.rbの修正です。

System testのスクリーンショット取得処理でcan't modify frozen Stringエラーが発生してしまうバグがあったのを修正しています。


Merge pull request #30403 from yhirano55/fix_broken_doc_for_active_record

Active Recordのdocの修正です。

:nodoc:が設定されているクラスに対するコメントがAPI docに表示されていまっていたのを、表示されないよう修正しています。


Use tt in doc for ActionPack [ci skip]

Action Packのdocの修正です。

doc内でバッククォートを使用していた箇所があったのをttタグまたは+を使用するよう修正しています。


Fix random CI failure due to non-deterministic sorting order

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

テストでassertionを取得する際に、idでsortした値を使用するよう修正しています。sortした値を使用しないと取得出来る値が不定になり、seedによってはテストが失敗してしまう為。


Correct wording and adding info [ci skip]

rails guideのAction Mailer Basicsの修正です。

Caching mailer viewの項のグラマーの修正、及び、Rails caching guideへのリンクを追加しています。


Fix order of Active Storage DiskService URL parameters

activestorage/lib/active_storage/service/disk_service.rbの修正です。

ActiveStorage::Service::DiskService#urlメソッドでdispositionオプションの前にcontent_typeオプションの値を設定するよう修正しています。

-            "/rails/active_storage/disk/#{verified_key_with_expiration}/#{filename}?disposition=#{disposition}&content_type=#{content_type}"
+            "/rails/active_storage/disk/#{verified_key_with_expiration}/#{filename}?content_type=#{content_type}&disposition=#{disposition}"

Rails.applicationが定義されている場合は、Rails.application.routes.url_helpers.rails_disk_service_pathメソッドを使用して上記URLを生成するのですが、その場合と結果が同じようになる為。多分。