なるようになるブログ

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

rails commit log流し読み(2015/04/27)

2015/04/27分のコミットです。

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

activerecord/CHANGELOG.md

actionpack/CHANGELOG.md


Don't reference sprockets assets on action pack

actionpack/lib/action_dispatch/routing/inspector.rbの修正です。

RouteWrapper#internal?メソッドからassets.prefixを無視するコードを削除しています。

rake routesコマンドのようなroutingの情報を出力する際assets_prefixの情報を無視する必要があり、RouteWrapper#internal?メソッドのその為のコードが入っていたのですが、 sprockets-rails側で考慮する対応が入り、ActionPack側で対応する必要が無くなったので、コードを削除しています。


Use latest sprockets-rails on internal dev

Gemfileの修正です。

sprockets-railsのmasterを使用する修正しています。


assets:cache:clean wont clean cache anymore

railties/test/application/assets_test.rbの修正です。

assets:clobberタスクがtmp/cache/assetsを削除する事を確認するテストを削除しています。

clobberタスクでtmp/cache/assetsを削除しないよう、sprockets-rails側が修正されたとの事で、それに合わせた対応したようです。


Need to add config.assets.compile=true to access .assets

railties/test/application/assets_test.rbの修正です。

assets pipelineのテストの際に、config.assets.compile = trueを設定するよう修正しています。

compilenilの場合、Rails.application.assetsが設定されないので、Rails.application.assetsが参照出来るようcompiletrueにするよう修正したようです。これもsprockets-rails修正による影響の為との事です。


[ci skip] Update ActiveSupport::Rails and ActiveSupport::Rescuable docs

activesupport/lib/active_support/rails.rbactivesupport/lib/active_support/rescuable.rbのdocの修正です。

各種グラマーの修正しています。


Merge pull request #19904 from zzak/rm_route_wrapper_internal_hacks

actionpack/test/dispatch/routing/inspector_test.rbの修正です。

assetsがdisbleになっている場合のrouteの実行結果についてのテストを追加しています。


Don't document private internal constant [ci skip]

activemodel/lib/active_model/validations.rbのdocの修正です。

VALID_OPTIONS_FOR_VALIDATE定数に:nodoc:を追加しています。


Don't document internal monkey patch for mysql gem [ci skip]

activerecord/lib/active_record/connection_adapters/mysql_adapter.rbのdocの修正です。

Mysql#to_dateメソッドのdocに、mysql gemへのモンキーパッチのコミットIDが記載されていたのを削除しています。


nodoc private internal constants from ActiveJob::Arguments

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

Arguments module配下のprivate定数に:nodoc:を追加しています。


Revert "Merge pull request #17920 from calebthompson/dont-rely-on-environment-task-for-schema-load"

activerecord/lib/active_record/railties/databases.rakeenvironmentに依存させないようにしたRemove environment dependency for db:schema:loadのコミットをrevertしています。

environmentに依存しなくなってしまった為、 initializersが実行されてしまくなってしまい、ActiveSupport::Inflector.inflectionsを使用していた場合に、正しくでテーブル名・カラム名の変換が行われなくなってしまった為、revertしたとのこ事です。


test, for create_table and foreign_key: true no-op. Closes #19794.

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

foreign_key: trueの場合にcreate_tableを行うテストを追加しています。


Add note about sti column value to becomes method [ci skip]

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

becomesメソッドのdocに、STIについての説明を追加しています、


Merge pull request #18392 from brainopia/fix_route_requirements

actionpack/lib/action_dispatch/journey/route.rbの修正です。

scopeメソッドdefaultオプションを指定している場合に、rake routesコマンドでscopeのブロック内で指定したresourceformatが正しく表示されないバグがあったのを修正しています。

issueより。

  scope constraints: {format: :html}, defaults: {format: 'html'} do
    resource  :search,    :only => :show
    resource  :dashboard, :only => :show, constraints: {format: /html|atom/}
  end
# `rake routes`実行結果
dashboard GET    /dashboard(.:format)             dashboards#show {:format=>"html"}

Add changelog for rake routes default fix

actionpack/CHANGELOG.mdの修正です。

上記routeの対応についてCHANGELOGに追記しています。


remove trailing whitespace. [ci skip]

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

touchメソッドのdocから不要な終端のスペースを削除しています。