なるようになるブログ

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

rails commit log流し読み(2015/07/28)

2015/07/28分のコミットです。

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

railties/CHANGELOG.md


[ci skip] it should be protect_from_forgery

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

protect_from_forgeryprotect_form_forgeryにタイポしていたのを修正しています。


Merge pull request #21003 from robin850/api-plugins

railtiesの修正です。

api only pluginを作成出来るよう対応しています。

rails app同様、--apiオプションを指定すると api only pluginが作成出来ます。

./bin/rails plugin new blorgh --mountable --api

create
create  README.rdoc
create  Rakefile
create  blorgh.gemspec
create  MIT-LICENSE
create  .gitignore
create  Gemfile
create  app
create  app/controllers/blorgh/application_controller.rb
create  app/mailers
create  app/models
create  config/routes.rb
create  lib/blorgh.rb
create  lib/tasks/blorgh_tasks.rake
create  lib/blorgh/version.rb
create  lib/blorgh/engine.rb
create  bin
create  bin/rails
create  test/test_helper.rb
create  test/blorgh_test.rb
append  Rakefile
create  test/integration/navigation_test.rb

rails api同様、assets、helpers、viewsが作成されないようになっています。


Merge pull request #21031 from vngrs/fix_params_wrapper_doc

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

wrap_parametersメソッドformatオプションに指定出来る値のサンプルにurl_encoded_formmultipart_formを追加、及び、 wrap_parametersの説明でwrap_parametersはPOSTリクエストでのみ使用出来るような説明になっていたのですが、実際はPUT、PATCH、DELETEリクエストでも使用出来るので、 POSTリクエストについて言及していた箇所を削除しています。


Merge pull request #20759 from vngrs/fix_wrong_doc

railties/lib/rails/generators/actions.rbのdocの修正です。

environmentメソッドのexampleで、config.autoload_pathsを設定するexampleを記載していたのですが、autoload_pathsapplication.rbででしか指定出来ないので、 例として不適切だろう、という事でaction_controller.asset_hostを指定するexampleに修正しています。


Merge pull request #20849 from vngrs/misleading_nested_exceptions

ActiveRecordのassociation処理の修正です。

has_one though associations を使用し、associationに不正なインスタンスを指定した場合に、HasManyThroughCantAssociateThroughHasOneOrManyReflectionHasManyThroughNestedAssociationsAreReadonly exceptionがraiseする場合があったのですが、has_oneを使用しているのに、exceptionのクラス名がHasMany~となっているのは 紛らわしいだろう、という事で、HasOneThroughCantAssociateThroughHasOneOrManyReflectionHasOneThroughNestedAssociationsAreReadonlyクラスを新規に作成し、 そちらがraiseされるよう修正しています。


Merge pull request #20590 from vngrs/set_default_charset

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

ActionDispatch#charset=メソッドで、処理分岐if..elseを使用して処理を記載していたのを、 三項演算子を使用するようリファクタリングしています。


Add :nodoc: for internal testing methods [ci skip]

各テスト内の、テスト用メソッド:nodoc:を設定しています。


[ci skip] Now Action View is completely separated from Action Pack

rails guideのAction View Overviewの修正です。

What is Action View?の項で、Action View と Action ControllerはAction Packのコンポーネントである旨説明されているのですが、 現在Acion ViewはAction Packから分離しており、現状と説明が一致していなかったので、最新の内容に説明を更新しています。


[ci skip] Add a link to action_controller_overview.html

rails guideのAction View Overviewの修正です。

What is Action View?の項で、Action Controllerについて説明している箇所で、Action Controller Overview guideへのリンクを設定しています。


Tiny documentation edits [ci skip]

各docの修正です。

適切なスペースの追加、1行80文字に収まるよう適切な位置に改行を追加、等を行っています。


pass the correct argument to mock on a test of validates_length_of

activemodel/test/cases/validations/i18n_validation_test.rbの修正です。

validates_length_ofが生成するエラーメッセージについて確認するテストで、引数を正しく渡していなかったのを、正しく渡すよう修正しています。