なるようになるブログ

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

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

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

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

actionpack/CHANGELOG.md

activerecord/CHANGELOG.md

actionview/CHANGELOG.md


Section explaining how rails runner accepts code in a ruby file.

rails guideのThe Rails Command Lineの修正です。

rails runnerにRubyのコードが記載されたファイルを渡せる旨説明を追加しています。


[ci skip] Add changelog entry for #18948.

actionview/CHANGELOG.mdの修正です。

multi_fetch_fragments gemをrailsにマージしたコミットについて、CHANGELOGに追記しています。


Merge pull request #18434 from brainopia/change_filter_on_rails_info_routes

ActionPackの修正です。

/rails/info/routesで、routesのあいまい検索を行う際、クライアント側で行っていたのをサーバ側で行うよう修正しています。JSのRuby(Oniguruma)とで書ける正規表現に差が有り、Ruby側行った方が正しい結果が得られる為、サーバ側で行うよう修正したとの事です。


Properly create through records when called with where

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

has many through association + whereメソッドを使用した場合に、associationが正しく生成されないバグがあったのを修正しています。

issueより。

class User < ActiveRecord::Base
  has_many :likes
  has_many :collections, through: :likes
end

class Like < ActiveRecord::Base
  belongs_to :collection
end

class Collection < ActiveRecord::Base
end

user = User.create!
user.collections.where(true).create!
user.likes.count #=> 0. 正しくは"1"になるべき

AssociationRelationクラスに、buildcreatecreate!メソッドを追加して対応しています。


Fix regression when passing a value different of String.

actionview/lib/action_view/helpers/translation_helper.rbの修正です。

Rails 4.2まではTranslationHelper#translateメソッドの引数にString以外のクラスを渡す事が出来たのが、4.2.1では出来なくなっていました。で、再度String以外のクラスも渡せるよう修正しています。

translate('foo', default: 1)

fix NameError in skip_filter. callback doesn't exist.

actionpack/lib/abstract_controller/callbacks.rbの修正です。

skip_filterメソッドで表示しているdeprecateメッセージで、存在しない変数を参照して、NameErrorが発生するバグがあったのを修正しています。


Revert "Merge pull request #18764 from tsun1215/master"

HEADリクエストのroutesを検索する際、routesのverbが//になっているものを削除するよう修正したコミットをrevertしています。

この対応のせいでActionPackのテストがコケていると思われるので、一旦revertしたとの事です。


Revert integration test refactoring that caused app test regressions

integration testに関する、三つのコミット(there is always an integration session, so remove the checklazily create the integration sessionuse before_setup to set up test instance variables)をまとめてrevertしています。

ActionPackのテストがまだコケて続けている為、関係ありそうなコミットをまとめてrevertしたとの事です。


Fix wording in template inheritance section of layout guide [ci skip]

rails guideのLayouts and Rendering in Railsの修正です。

Similarly -> Similarに言い回しを修正しています。


[ci-skip] correct doc about custom validators

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

Custom Validatorsの項のグラマーの修正を行っています。


Merge pull request #19110 from aditya-kapoor/fix-val-doc