なるようになるブログ

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

rails commit log流し読み(2015/12/22)

rails ruby

2015/12/22分のコミットです。

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

railties/CHANGELOG.md

Fixes using add_source with a block after using gem in a custom rails

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

#add_sourceメソッドでGemfileにsourceを追加する際、改行が追加されず、生成されたGemfileを使用するとbundle installがエラーになってしまう問題があったのを修正しています。

PRより。

gem "font-awesome-rails"

add_source "https://rails-assets.org" do
  gem 'rails-assets-bootstrap'
  gem 'rails-assets-jasny-bootstrap'
end

上記のようなcustom generatorがあった場合に、

gem 'font-awesome-rails'source 'https://rails-assets.org' do
  gem 'rails-assets-bootstrap'
  gem 'rails-assets-jasny-bootstrap'
end

生成されるGemfileが上記のようになっていたのを、

gem 'font-awesome-rails'
source 'https://rails-assets.org' do
  gem 'rails-assets-bootstrap'
  gem 'rails-assets-jasny-bootstrap'
end

となるよう修正しています。

Use an appropriate rebuild task on Travis

ci/travis.rbの修正です。

travisでtaskのrebuildを行う際、適切なtask(mysql2 adapterを使用している場合はdb:mysql:rebuild、postgres adapterを使用している場合はdb:postgresql:rebuild)が実行されるよう修正しています。

Ensure that assets are enabled back after the test that tests assets are disabled

railties/test/generators/channel_generator_test.rbの修正です。

Rails::Generators.options[:rails][:assets]がfalseの場合にchannel assetが生成されない事を確認するテストで、テストの後処理でRails::Generators.options[:rails][:assets]を元々設定されていた値に戻すよう修正しています。

Remember the parameter hash we return

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

ActionDispatch::Http::Parameters#path_parametersメソッドで、#get_headerで値を取得できなかった時に空のHashを返していたのを、#set_headerを呼び出して、その戻り値を返すよう修正しています。

      def path_parameters
-        get_header(PARAMETERS_KEY) || {}
+        get_header(PARAMETERS_KEY) || set_header(PARAMETERS_KEY, {})
       end

ActionDispatch::Http::Parameters#parametersメソッドも同じような挙動になっており、set処理までやる事を期待されていた挙動のようです。

Update "Support" in ActionCable README [ci skip]

actioncable/README.mdの修正です。

Supportの項で、Action Cableに関するissueの登録先がhttps://github.com/rails/actioncable/issuesになっていたのをhttps://github.com/rails/rails/issuesに修正、及びrails-coreメーリングリストへのリンクを追加しています。

[ci skip] No more no changes entries in the CHANGELOGs (cases left in #22718)

guides/CHANGELOG.md、railties/CHANGELOG.mdの修正です。

それぞれのCHANGELOGの先頭にあった不要なNo changes.字を削除しています。

No more no changes entries in the CHANGELOGsの対応漏れ対応です。

Ensure Action Cable files are removed when skip_action_cable is set.

railties/lib/rails/generators/rails/app/app_generator.rbの修正です。

rails newにskip_action_cableオプションが指定された場合に、Action Cableに関するファイル・ディレクトリを削除するよう対応しています。

Use the generators options to not generate channel assets

actioncable/lib/rails/generators/channel/channel_generator.rbの修正です。

assetsの生成を行うかどうかのチェックを、他のgenerator同様、class_optionを使用するよう修正しています。

+      class_option :assets, type: :boolean
+
       check_class_collision suffix: "Channel"

       def create_channel_file
         template "channel.rb", File.join('app/channels', class_path, "#{file_name}_channel.rb")
-        if Rails::Generators.options[:rails][:assets]
+
+        if options[:assets]
           template "assets/channel.coffee", File.join('app/assets/javascripts/channels', class_path, "#{file_name}.coffee")
         end

[ci skip] Fix typo

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

RailsGuides::Markdown::Renderer#headerメソッドのdocにタイポ(記載されるべき値が記載されていなかった)があったのを修正しています。

Fix Rails tutorial link

railties/RDOC_MAIN.rdocの修正です。

Rails Tutorialへのリンクが404になってしまっていたのを修正しています。

Merge pull request #22709 from jonatack/clarify-explanation-for-new-config-halt-callback-chains

rails guideのConfiguring Rails Applicationsの修正です。

ActiveSupport.halt_callback_chains_on_return_falseオプションについて、詳細な説明を追加しています。

corrects a typo

rails guideのRails Internationalization (I18n) APIの修正です。

Adding Date/Time Formatsの項のexampleで、pタグの閉じ括弧が不足していた箇所があったのを修正しています。

do not use div_for in example [ci skip]

actionview/lib/action_view/record_identifier.rbのdocの修正です。

ActionView::RecordIdentifier moduleのexampleで、rails 本体から削除されたdiv_forメソッドを使用している箇所があったので、使用しないよう修正しています。

Fix .keep file issue with ActionCable

railties/lib/rails/generators/rails/app/app_generator.rbの修正です。

rails newに--skip-keepsを指定された場合でも、app/assets/javascripts/channelsディレクトリは生成するよう修正しています。

app/assets/javascripts/cable.coffeeでrequire_tree ./channelsをしており、app/assets/javascripts/channelsディレクトリが無いとJSのロードでエラーになってしまう為、との事です。

remove content_tag_for example

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

Passing Local Variablesの項にあるexampleコードで、rails 本体から削除されたcontent_tag_forメソッドを使用している箇所があったので、使用しないよう修正しています。

Merge pull request #22483 from y-yagi/use_bin_test_in_plugins_guide

rails guideのThe Basics of Creating Rails Pluginsの修正です。

テストの実行にrakeコマンドではなく、bin/testコマンドを使用するよう修正しています。

Merge pull request #22701 from maclover7/rails-5-release-notes

rails guideにRuby on Rails 5.0 Release Notesを追加しています。

あくまで雛形としての状態で、後ほど諸々修正されています。

guides, restructure Rails 5.0 release notes. [ci skip]

rails guideのRuby on Rails 5.0 Release Notesの修正です。

Major Features に、Active Rcord Attributes API、Test Runnerを追加、及び各コンポーネント毎にNotable changes、Removals等の情報を追記しています。

remove cors initializer from rails app

railties/lib/rails/generators/rails/app/app_generator.rbの修正です。

rails apiでのみ必要なinitializer(rack-cors用initializer)をrails appの場合は削除するよう修正しています。

release notes, extract notable changes from Railties CHANGELOG.

rails guideのRuby on Rails 5.0 Release Notesの修正、及びrailties/CHANGELOG.mdの修正です。

railtiesのCHANGELOGのグラマー、フォーマットの修正、及び、Ruby on Rails 5.0 Release Notesにrailtiesの変更内容(Removals、Notable changes、Deprecationsのそれぞれの項)を追記しています。

[ci skip] Steal ActionCable.server mounting from routes template.

actioncable/README.mdの修正です。

Action Cable用URLをroutes.rbに定義する場合のexampleを、config/routes.rbの雛形のコメントに書いてある内容と合わせて、mountメソッドを使用する例に修正しています。

  # config/routes.rb
  Example::Application.routes.draw do
 -  match "/cable", :to => ActionCable.server, via: [:get, :post]
 +  mount ActionCable.server => '/cable'
  end

release notes, extract notable changes from Action Pack CHANGELOG.

rails guideのRuby on Rails 5.0 Release Notesの修正、及びactionpack/CHANGELOG.mdの修正です。

Action PackのCHANGELOGのグラマー、フォーマットの修正、及び、Ruby on Rails 5.0 Release NotesにAction Packの変更内容(Removals、Notable changes、Deprecationsのそれぞれの項)を追記しています。

release notes, use past tense. [ci skip]

rails guideのRuby on Rails 5.0 Release Notesの修正です。

動詞に過去形を使用するようまとめて修正しています。

例。

-*   Remove deprecated `test:all` and `test:all:db` tasks.
+*   Removed deprecated `test:all` and `test:all:db` tasks.

release notes, extract notable changes from Action View CHANGELOG.

rails guideのRuby on Rails 5.0 Release Notesの修正、及びactionview/CHANGELOG.mdの修正です。

Action ViewのCHANGELOGのグラマー、フォーマットの修正、及び、Ruby on Rails 5.0 Release NotesにAction Viewの変更内容(Notable changes、Deprecationsのそれぞれの項)を追記しています。

release notes, extract notable changes from Action Mailer CHANGELOG.

rails guideのRuby on Rails 5.0 Release Notesの修正、及びactionmailer/CHANGELOG.mdの修正です。

Action MailerのCHANGELOGのグラマー、フォーマットの修正、及び、Ruby on Rails 5.0 Release NotesにAction Mailerの変更内容(Notable changes、Deprecationsのそれぞれの項)を追記しています。