読者です 読者をやめる 読者になる 読者になる

なるようになるブログ

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

rails commit log流し読み(2016/08/13)

2016/08/13分のコミットです。

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


Make cache max-age durations self-explanatory

railties/lib/rails/generators/rails/app/templates/config/environments/development.rb.ttrailties/lib/rails/generators/rails/app/templates/config/environments/test.rb.ttの修正です。

cache max-ageを指定するexampleに、普通に数字を指定していたのを、ActiveSupport::Duartionの値を指定するよう修正しています。

    config.public_file_server.headers = {
-      'Cache-Control' => 'public, max-age=172800'
+      'Cache-Control' => "public, max-age=#{2.days.seconds.to_i}"
     }

こちらの方が意図が伝わりやすい為との事です。


Given a hash (Rails 5) .from_hash must be used

actionpack/lib/action_dispatch/http/headers.rbのdocの修正です。

ActionDispatch::Http::Headersクラスのdocで、ActionDispatch::Http::Headersクラスのインスタンスを生成するのに、.newを使用していたのを、.from_hashを使用するよう修正しています。

pass a request object to the headers objectでenv hashを使用してActionDispatch::Http::Headersインスタンスを生成する場合.from_hashを使用するよう変更になった為。


Mark method as nodoc

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

public APIでないメソッドに:nodoc:を設定しています。


Make private method private

actionpack/lib/action_dispatch/http/parameters.rbの修正です。

クラス内でだけ使用するメソッドの可視性をprivateに変更しています。


Merge pull request #26100 from vipulnsward/changelogs

CHANGELOGの修正です。

各エントリーのグラマー、フォーマットの修正等をまとめて行っています。


Merge pull request #26113 from kamipo/use_ar_test_case_instead_of_as_test_case

activerecord/test/cases/fixtures_test.rbactiverecord/test/cases/relation/mutation_test.rbの修正です。

各テストの親クラスをActiveSupport::TestCaseからActiveRecord::TestCaseに修正しています。 

Active Recordのテストなので、ActiveRecord::TestCaseを使用する方が適切な為。


Add documentation about ActiveSupport.on_load

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

ActiveSupport::LazyLoadHooks#on_loadメソッドにdocを追加しています。


Ensure app/mailers is removed when skipping action-mailer

railties/lib/rails/generators/rails/app/app_generator.rbの修正です。

skip-action-mailerオプションを指定した際に、app/mailersディレクトリを削除するよう修正しています。