なるようになるブログ

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

rails commit log流し読み(2015/03/25)

2015/03/25分のコミットです。

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

activerecord/CHANGELOG.md


Update ActiveSupport::Subscriber docs

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

ActiveSupport::Subscriber.attach_toメソッドの使い方が変更になっていた(Allow attaching to AS::Notifications namespace up front by dasch · Pull Request #12285 · rails/railsのが、docに反映されていなかったので、修正内容に合わせて、attach_toメソッドの設定方法を修正しています。


DRY up STI subclass logic

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

STIに関するメソッド(find_sti_classtype_condition)のリファクタリングを行っています。

が、テストが壊れてしまったらしく、直ぐrevertされています。


Revert "Merge pull request #19500 from ccutrer/dry_sti_subclass_finding"

DRY up STI subclass logic のrevertです。テストが壊れてしまった為。


Add changelog entry for #19271

actionpack/CHANGELOG.mdの修正です。

HTTPのstatus codeが304の場合に設定してはならないContent-Type ヘッダーが設定されてしまっていたバグを修正したコミット(304 response should not include Content-Type header by eagletmt · Pull Request #19271 · rails/rails)についてCHANGELOGに追記しています。


[ci skip] Update 4.2 Release notes with more assert_select information.

rails guideのRuby on Rails 4.2 Release Notesの修正です。

assert_selectメソッドの変更について、正規表現を使用している場合のexampleを追加しています。


Avoid loading user's psqlrc when loading test structure

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

database structure ファイルを読み込む際、個人のpsqlrcファイルを読み込まないよう修正しています。

-        Kernel.system("psql -q -f #{Shellwords.escape(filename)} #{configuration['database']}")
+        Kernel.system("psql -X -q -f #{Shellwords.escape(filename)} #{configuration['database']}")

psqlrcに\timing等を設定していた場合に、行読む込む度にTimeが表示されてしまい、邪魔な為との事です。


update config.assets.digest default value in guides [ci skip]

rails guideのThe Asset Pipelineの修正です。

config.assets.digest のdepelopment環境でのデフォルトが、Rails 4.2からtrueになったのですが、guideではfalseである旨説明がされてしまっていたので、修正しています。


[ci skip] Active Record Validations guide fixes

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

以下を行っています。

  • deprecatedになったerrors#[]=メソッドの削除
  • 重複していたerrors#addメソッドの整理
  • errors#adderrors#full_messagesについての説明の改善

Documenting remove_column as a reversible migration method [ci skip]

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

remove_columnメソッドの第三引数に型情報を指定してあげる事で、rollback実行時に、元のカラムを復元する事が出来る旨説明を追加しています。

例。

remove_column :posts, :slug, :string, null: false, default: '', index: true