なるようになるブログ

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

rails commit log流し読み(2014/05/22)

2014/05/22分のコミットです。

CHANGELOGにのったコミットは以下の通りです(詳細は本文参照)。

activerecord/CHANGELOG.md

actionpack/CHANGELOG.md


Add missing nodocs to MySQL adapter

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

:nodoc:を追加しています。


Revert "Merge pull request #14544 from jefflai2/named_scope_sti"

昨日マージされたnamed scopeに関するバグ修正のPRをrevertしています。

このPRにより、activerecord-deprecated_findersのgemが壊れてしまったとの事です。


Merge pull request #14613 from Sirupsen/fix-serialize-update-column

serialized fieldsを使用している時に、update_columnメソッドで値を更新した際に、serialize済みのデータを戻すよう修正しています。


Rename oid_type to cast_type to make PG columns consistent

activerecord/lib/active_record/connection_adapters/postgresql/column.rbの修正です。

oid_typecast_typeに変数名を変更しています。


fixes in active record query for reorder

rails guideのActive Record Query Interfaceの修正です。

発行されるSQLが間違えてたのを修正しています。


push the formatter up to the Route object

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

Visitors::FormatBuilderをActionDispatch::Journey::Routeクラスで保持するよう修正しています。


do not mutate parameters, let the caller do mutations

ActionDispatch::Journey::Route#formatで行っていたパラメータのチェック処理を、ActionDispatch::Journey::Formattr##generateメソッドで行うよう修正しています。


make variable name more clear

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

変数名を、formatter->path_formatterに修正しています。


reduce object allocations

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

オブジェクトの生成処理を減らすよう修正しています。

コミットログにオブジェクト生成を記録した際のサンプルソースが乗っています。ご参考に。


reuse path formatter from the non-optimized path.

actionpack/lib/action_dispatch/journeyの修正です。

OptimizedPathクラスに関する処理を消しています。まだ作業中のコミットのような。


Move extract_scale to decimal type

extract_scaleをdecimal typeに移動しています。

処理をtype objectに集約している一環だと思うのですが、内容いまいちわかりませんでした…。


we can just use Ruby here

actionpack/lib/action_controller/metal/url_for.rbの修正です。

reverse_mergeを使用している箇所を、mergeを使用するように修正しています。

reverse_mergeActiveSupportの拡張だからですね。

2014-05-22 07:51:28 +0900


remove dead code. @klass isn't used anymore

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

使用していない変数の削除。


use the existing constant rather than the hardcoded string

actionpack/lib/action_dispatch/http/parameters.rbの修正です。

Stringがハードコーディングされているのを、定数を使うように修正しています。

コミットログがあらぶっているのが面白いです。


middle variable is never used, so rm

@router.recognizeを呼び出している箇所から、使用してない変数を削除しています。


fewer object allocations and method calls during route match

ActionDispatch::Journey::Router#find_routesメソッドの修正です。

オブジェクト生成を減らすようリファクタリングしています。


drop hash allocations during match

ActionDispatch::Journey::Router#find_routesメソッドの修正です。

Hash#merge呼び出しを減らしています。


Merge pull request #15218 from sgrif/sg-move-oid-types

activerecord/lib/active_record/connection_adapters/postgresql/oid.rbの修正です。

一ファイルにまとめられていたActiveRecord::ConnectionAdapters::PostgreSQL::OID配下のクラスを、それぞれ別ファイルに切り出しています。


Rename Posts to Articles in Guides, continuation of 2d446e77 / #13774 [ci skip]

rails guide全般の修正です。サンプルのモデルをPostからArticleに修正しています。

HTTPの方のpostと混乱してややこしいから、という理由のようです。


Update CHANGELOG.md

guides/CHANGELOG.mdの修正です。

順番を入れ替えています。


Merge pull request #14803 from kuldeepaggarwal/null_relation_sum_fix

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

NullRelationgroup を設定している時に、, sum, size, average, minimummaximumHashを戻すよう修正しています。


Form full URI as string to be parsed in Rack::Test.

actionpack/lib/action_dispatch/testing/integration.rbの修正です。

URIを生成するのに、元々はURIクラスを使用していたのを、Stringで文字連結するよう修正しています。

性能改善のためとの事。数字等はPR参照。


pg test, remove unused code.

activerecord/test/cases/adapters/postgresql/datatype_test.rbの修正です。

使用していないコードの削除。


Do not gsub non ASCII characters in header anchor.

guides/rails_guides/markdown.rbの修正です。

ASCII以外の文字をアンカーに指定出来なかったのを、ASCII以外の文字も使用出来るよう修正しています。


pg test, extract network address type tests into separate file.

PostgreSQL DBAdapterのテストの修正です。

ネットワーク関係のテストを、activerecord/test/cases/adapters/postgresql/network_test.rbにまとめています。


pg test, move timestamp tests over to postgresql/timestamp_test.rb.

PostgreSQL DBAdapterのテストの修正です。

timestampのテストを、activerecord/test/cases/adapters/postgresql/timestamp_test.rbにまとめています。


pg test, extract tsvector tests into postgresql/full_text_test.rb PostgreSQL DBAdapterのテストの修正です。

full textのテストを、activerecord/test/cases/adapters/postgresql/full_text_test.rbにまとめています。


Use break instead of next in AD::Journey::Formatter#match_route

ActionDispatch::Journey::Formatter#match_routeメソッドの修正です。

hash.keys.sort.reverse_eachした値でループを回しているので、一度0以下の値が出たら、以降全て0以下になるはずなので、nextではなく、breakを使うよう修正しています。


Fix the hash key being delete from env

actionpack/lib/action_dispatch/http/parameters.rbの修正です。

文字列がハードコードされている箇所を定数に置き換える修正で、誤った修正が行われてしまったので、文字列に戻しています。