なるようになるブログ

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

rails commit log流し読み(2016/05/28)

2016/05/28分のコミットです。

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


Merge pull request #25160 from kamipo/remove_association_for_table

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

使用していないassociation_for_tableメソッドを削除しています。


cometic updates to security guide - fixes #25058 [ci skip]

rails guideのRuby on Rails Security Guideの修正です。

Privilege Escalationの項で、Web Developer Toolbar -> Developer toolsに修正及び言い回しの修正を行っています。


[ci skip] Reword doc for around_enqueue callback

activejob/lib/active_job/callbacks.rbのdocの修正です。

around_enqueueメソッドのdoc内のメソッドについての説明を、around_performと合わせるよう修正しています。


Use add_index instead of execute in guides [ci skip]

rails guideのActive Record and PostgreSQLの修正です。

Full Text Searchの項で、exampleコードでINDEXを生成するのに生のSQLを使用していたのを、add_indexメソッドを使用するよう修正しています。


fix incorrect class name [ci skip]

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

ActiveModel::ForbiddenAttributesErrorActiveModel::ForbiddenAttributesにタイポしていたのを修正しています。


Boil down custom Action Cable configuration to a single block

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

デフォルトのproduction.rbでAction Cable用の設定を一つのブロックにまとめています。


Remove public/ files for API apps

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

API-only Applicationsを生成した際、不要なpublicファイル(public/404.htmlpublic/favicon.ico等)を削除するよう修正しています。


The first comment does not explain anything that the code doesnt and the second is now irrelevant with the dedicated API mode

railties/lib/rails/generators/rails/app/templates/app/controllers/application_controller.rb.ttの修正です。

デフォルトで生成されるapplication_controller.rbからprotect_from_forgeryメソッドについての説明コメントを削除しています。

<%- unless options[:api] -%>
-  # Prevent CSRF attacks by raising an exception.
-  # For APIs, you may want to use :null_session instead.
   protect_from_forgery with: :exception
 <%- end -%>

最初のコメントはコードについて何も説明していないからで、二つめのコメントはRails 5からはAPI only Applicationsが生成出来るようになり、普通のRailsアプリの方にはは不要な説明だろうとの事のようです。


Redirects are to urls, not paths

railties/lib/rails/generators/test_unit/scaffold/templates/functional_test.rbの修正です。

デフォルトで生成されるcontrollers用のテストファイルで、redirectの確認をする際に、xxx_pathの値のチェックをしていたのを、xxx_urlの値のチェックをするよう修正しています。