2015/08/15分のコミットです。
CHANGELOGへの追加はありませんでした。
Merge pull request #20647 from twalpole/method_source_dependency
activesupport/activesupport.gemspec
の修正です。
Testing::CompositeFilter
クラスで使用しているmethod_source
をgemspecのadd_dependencyに追加しています。
actionpack/lib/action_dispatch/routing/mapper.rb
の修正です。
Mapper
クラスのコンストラクタで行っていたpathのnormalization処理を、クラスメソッドに切り出して、Mapping
クラスのインスタンス生成処理より先に行うよう修正しています。
stop adding path_info to the conditions hash
ActionPackの修正です。
conditions hashから不要なpath_info
を削除しています。
extract ast finding to a method
actionpack/test/dispatch/mapper_test.rb
の修正です。
conditions hashから直接astを取得していたのを、ast取得用メソッドを切り出して、それを使用するよう修正しています。
implement the asts
method in terms of paths / patterns
actionpack/test/dispatch/mapper_test.rb
の修正です。
テスト用のFakeSet
クラスに、ast追加用のメソッド及び保持用の変数を追加しています。
implement requirements
in terms of routes
actionpack/test/dispatch/mapper_test.rb
の修正です。
pathのrequirementsを取得する為のメソッドをFakeSet
クラスに実装しています。
remove hard coded regular expression
actionpack/lib/action_dispatch/journey/route.rb
、
actionpack/lib/action_dispatch/journey/router.rb
の修正です。
Router#match_head_routes
メソッド内で直接行っていたverbのマッチング処理を、別メソッドに切り出しています。
use predicate methods instead of hard coding verb strings
ActionPackの修正です。
request methodがHEAD
がどうかをチェックするのに、"HEAD"文字列と値が一致するか比較処理を行っていたのを、head?
メソッドを使用するよう修正しています。
ActionPackの修正です。
Mapper
クラスのコンストラクタで行っていたastの取得処理を、クラスメソッドに切り出して、Mapping
クラスのインスタンス生成処理より先に行うよう修正しています。
actionpack/lib/action_dispatch/routing/mapper.rb
、
actionpack/lib/action_dispatch/routing/route_set.rb
の修正です。
add_route
メソッドにastを渡す際、conditions hashに内に設定していて渡していたのを、astを渡すようの引数をadd_route
メソッドに追加して、そちら経由で渡すよう修正しています。
現状、conditions hashにastを追加する必要は無く、後で削除する必要もある為、そもそも渡さないようにしたとの事です。
extract method on wildcard path parameter handling
actionpack/lib/action_dispatch/routing/mapper.rb
の修正です。
path parameterにワイルドーカードを指定された場合の処理を、add_wildcard_options
メソッドに切り出しています。
actionpack/lib/action_dispatch/routing/mapper.rb
の修正です。
不要なpath_params
メソッドを削除しています。
メソッドでは、astからJourney::Nodes::Symbol
を検索し一致した値をto_sym
に変換したmapしていただけで、わざわざメソッドに切り出す程ではない、という事で削除されたようです。
deprecate passing a string for both the beginning path and :path option
actionpack/lib/action_dispatch/routing/mapper.rb
の修正です。
routesにpath
オプションを指定するのがdeprecateになりました。
path
オプションってまだ使用出来たんですねえ。
extract method on determining :to from the path
actionpack/lib/action_dispatch/routing/mapper.rb
の修正です。
process_path
メソッド内で行っていた、pathからのtoの取得処理を、別メソッド(get_to_from_path
)に切り出しています。
explicitly return nil from get_to_from_path
actionpack/lib/action_dispatch/routing/mapper.rb
の修正です。
get_to_from_path
メソッドでto
が取得出来なかった場合に、明示的にnil
を返すよう修正しています。
初期化チェックを減らせるようにするために、明示的に返すようにしたようです。
actionpack/lib/action_dispatch/routing/mapper.rb
の修正です。
to
の初期化処理を外だしにした事により不要になったprocess_path
メソッドを削除しています。
rails guideのA Guide to Testing Rails Applications
の修正です。
タイポ、グラマーの修正等をまとめて行っています。
pass the mapping object to build_route
ActionPackの修正です。
RouteSet#add_route
メソッドに、conditions hashの代わりに、Mapping
クラスのオブジェクトを渡すよう修正しています。
add_route
の中でconditions hashを直接操作する事は無くなり、また、Mapping
クラスのオブジェクトの方が値が整形済みで、必要な値の取得がしやすいため、修正したようです。
pass pass the mapping object down the add_route stack https://github.com/rails/rails/commit/6c48d9392fe964640fe5721fcd27bb170613cc27
ActionPackの修正です。
Routes#add_route
メソッドでmappingオブジェクトを渡せるようにする為のリファクタリングを行っており、検証用に一時的にRoutes#add_route2
メソッドを追加しています。
ActionPackの修正です。
先のコミットで一時的に作成した、Routes#add_route2
メソッドを削除しています。
only keep one hash of named routes
ActionPackの修正です。
Routes
クラスから、named_routes
変数を削除しています。Routes
クラスでは、named_routes
とは別にouter router objectを保持しており、そちらからnamed routesも取得出来るので、別途変数を保持せず、outer router object経由で値を取得するよう修正しています。
Convert Releasing Rails guide to Markdown
Releasing Rails Guideのファイルのフォーマットを、rdocからMarkdownに変更しています。
use average_scheduled_poll_interval
option instead of deprecated poll_interval
activejob/test/support/integration/adapters/sidekiq.rb
の修正です。
sidekiq adapterのテストで、deprecatedになったpoll_interval
オプションの代わりに、average_scheduled_poll_interval
を使用するよう修正しています。