なるようになるブログ

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

rails commit log流し読み(2015/09/30)

2015/09/30分のコミットです。

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

activerecord/CHANGELOG.md


Fix documentation for has_many dependant options.

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

Options for belongs_toの項、has_manydependentオプションに指定出来る値が誤っていたのを修正、及び、指定可能な全ての値についての説明を追加しています。


Stop modifying global variables

railtiesの修正です。

middlewareの管理をするのに、グローバル変数である(@middleware)を直接操作していたのをやめて、application毎のconfigurationで管理するよう修正しています。

が、middlewareのマージ処理に誤りがあった為、後ほど修正が行われています。


Make app building threadsafe

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

middlewareのbuild処理をmutexを使用してスレッドセーフになるよう修正しています。


Eliminate overlapping app/assets load path https://github.com/rails/rails/commit/20ec1e922cc141364881b17e2509c4d7dccca695

railtiesの修正です。

app/assets のロード処理の改善として、manifest.jsをassets直下からconfig配下に移動し、link_directoryでstylesheets / javascripts 配下のファイルを全て読み込むよう修正しています。これは stylesheets配下のファイルをrequireする際に、stylesheetsのパス部分を指定せずに済むように、との事です。


Engines get different middleware than apps

railties/lib/rails/application.rbrailties/lib/rails/configuration.rbrailties/lib/rails/engine.rbの修正です。

middlewareのマージ処理で、Rails EngineとRails Appとの場合で参照するmiddlewareを分けなければならないのを、誤って同じmiddlewareを参照しまっていた為、それぞれ適切な値を参照するよう修正しています。


call default_middleware_stack before merging stacks

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

middlewareのマージを行う前に、default_middleware_stackメソッドを呼び出してstackを設定するよう修正しています。

default_middleware_stackメソッドon_loadメソッドを実行する為、middleware stack自身の状態を変更させてしまう可能性がある為、先に処理を呼び出すようにしたようです。


Bump concurrent-ruby to shush circular require warning

Gemfile.lockの修正です。

concurrent-rubyを最新のmasterを参照するよう修正しています。最新のコードでは、circular require consideredのwarningが表示されないよう対応されている為、との事です。


Fixed concurrent-ruby warnings.

Gemfileactivesupport/activesupport.gemspecの修正です。

concurrent-rubyのバージョンを1.0.0.pre2 -> 1.0.0.pre3に更新しています。


Merge pull request #20574 from repinel/fix-db-fixtures-load

Active Recordのfixturesの修正です。

fixturesファイルに、model class名を指定出来るよう対応しています。

_fixture:
  model_class: User
david:
  name: David

db:fixtures:loadのような、set_fixture_classが使用出来ない場合に便利そうです。


run against edge sass to eliminate circular require warnings

Gemfileの修正です。

sass gemの参照先を、githubstableブランチに変更しています。gtihub上では、circular require warningが除去されている為、との事です。