なるようになるブログ

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

rails commit log流し読み(2017/05/11)

2017/05/11分のコミットです。

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


Fix a typo in guide [ci skip]

rails guideのWorking with JavaScript in Railsの修正です。

Dealing with Ajax eventsの項でexampleexempleにタイポしていたのを修正しています。


Defer loading each DB Tasks class from AR DatabaseTasks

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

DB task用のクラスをの読み込み処理を遅延ロードするよう修正しています。

-      register_task(/mysql/,        ActiveRecord::Tasks::MySQLDatabaseTasks)
-      register_task(/postgresql/,   ActiveRecord::Tasks::PostgreSQLDatabaseTasks)
-      register_task(/sqlite/,       ActiveRecord::Tasks::SQLiteDatabaseTasks)
+      register_task(/mysql/,        'ActiveRecord::Tasks::MySQLDatabaseTasks')
+      register_task(/postgresql/,   'ActiveRecord::Tasks::PostgreSQLDatabaseTasks')
+      register_task(/sqlite/,       'ActiveRecord::Tasks::SQLiteDatabaseTasks')

アプリで使用しているDB以外のクラスの読み込みを行わないようにする為。

rails commit log流し読み(2017/05/10)

2017/05/10分のコミットです。

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


Allow to receive arbitrary arguments in aggregated_results

railties/lib/rails/test_unit/minitest_plugin.rbの修正です。

Minitest::SuppressedSummaryReporter#aggregated_resultsメソッドに任意の引数を指定出来るよう修正しています。

-    def aggregated_results
+    def aggregated_results(*)
       super unless options[:output_inline]
     end

minitest側(SuppressedSummaryReporterの親クラスであるSummaryReporter)でaggregated_resultsに引数が追加されており、オーバーライドしているこちらのメソッドでも引数を受け取れるようにするために対応しています。

参考:- Write aggregated_results directly to the IO object to avoid mixed e…

rails commit log流し読み(2017/05/09)

2017/05/09分のコミットです。

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


Merge pull request #28240 from kreintjes/fix/autoloading-sti-guide

rails guideのAutoloading and Reloading Constantsの修正です。

Autoloading and STIの項、 STIを使用している場合のsubclassesのload方法についての説明に誤りがあったのを修正しています。


Merge pull request #28995 from jcoyne/update_capybara

Gemfilerailties/lib/rails/generators/rails/app/templates/Gemfileの修正です。

Capybaraのバージョンロックを緩めて、bundle updateでminor releaseを含められるよう修正しています。

-  gem 'capybara', '~> 2.13.0'
+  gem 'capybara', '~> 2.13'

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

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

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


Fix link to assert_nothing_raised doc [ci skip]

rails guideのA Guide to Testing Rails Applicationsの修正です。

assert_nothing_raisedメソッドのdocへのリンクが誤っていたのを修正しています。

rails commit log流し読み(2017/05/07)

2017/05/07分のコミットです。

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


Should escape meta characters in regexp

テストでassertionに正規表現を使用している箇所で、正規表現内のメタ文字(.)をエスケープしていない箇所があったのを、まとめてエスケープするよう修正しています。

rails commit log流し読み(2017/05/06)

2017/05/06分のコミットです。

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


Remove unused DelegationTest#call_method

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

使用していないActiveRecord::DelegationTest#call_methodメソッドを削除しています。


Merge pull request #28989 from matthewd/quoted_id-deprecation

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

quoted_idを使用した場合に出力するdeprecateメッセージで、quoted_idメソッドを定義した箇所を表示するよう修正しています。

現状、quoted_idを使用しているのは独自に定義した場合だけ(の筈)なので、定義した箇所があった方がわかりやすいだろう、という事で対応されたようです。


Fix warning: ambiguous first argument

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

quotingのテストでRubyのwarning(ambiguous first argument; put parentheses or a space even after/‘ operator`)が出ていたのを対応しています。

rails commit log流し読み(2017/05/05)

2017/05/05分のコミットです。

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


Assorted delegate_missing_to doc fixes

activesupport/lib/active_support/core_ext/module/delegation.rbのdocの修正です。

delegate_missing_toメソッドのdocについて、フォーマットの修正、delegateされたメソッドはpublicメソッドである必要がある旨説明を追加、等を行っています。


Update test names to match method name

activesupport/test/core_ext/module_test.rbの修正です。

delegate_missing_toメソッドのテストのテスト名がtest_delegate_to_missing_xxxになってしまっていたのを、test_delegate_missing_to_xxxに修正しています。


Merge pull request #28983 from kamipo/remove_useless_target_records_from_association

activerecord/lib/active_record/associations/preloader/through_association.rbの修正です。

不要なassocationがload済みかどうかのチェックを行わないよう修正しています。


Don’t pass arel.engine to Arel::SelectManager.new

Active Recordの修正です。

Arel::SelectManager.newarel.engineを渡さないよう修正しています。

Remove `engine` from `TreeManager` and subclassesの対応でArel::SelectManager.newの引数からare.engineが削除されており、その対応に合わせる為。