読者です 読者をやめる 読者になる 読者になる

なるようになるブログ

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

rails commit log流し読み(2017/03/14)

2017/03/14分のコミットです。

CHANGELOGへの追加はありませんでした。


Merge pull request #28385 from javan/actioncable/fix-npm-package

Gemfileactioncable/Rakefileの修正です。

action_cable.jsのUMD module supportの為に、Gemfileにsprockets-exportを追加しています。

元々はbladeの依存で入るようになっていた為問題なかったのですが、blade 0.7.0でsprockets-exportが依存から外れてしまっていた為、今回明示的にGemfileに追加しています。

合わせて、コンパイルされたAction Cable assetsが正常に動作する事を確認する為のrake taskも追加しています。


Merge pull request #28381 from shime/docs-action-controller-base

Action Packのdocの修正です。

ActionController module配下の各メソッド、クラス等のdocのフォーマット、グラマーの修正をまとめて行っています。


Fix HTTPS grammar

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

ActionController::ForceSSL moduleのdocのグラマーの修正を行っています。


Fix Typo [ci skip]

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

ActionDispatch::IntegrationTestクラスのdoc内のexampleコードで、クラス名をタイポしている箇所があったのを修正しています。


Small grammar change

actionpack/CHANGELOG.mdの修正です。

ActionController::Renderer#with_defaultsを修正した対応のエントリーのグラマーの修正を行っています。


Small grammar change + new line for “fixes”

actionview/CHANGELOG.mdの修正です。

encode_special_charsオプションについて対応したエントリーのグラマーの修正を行っています。


Lint activesupport/CHANGELOG.md

activesupport/CHANGELOG.mdの修正です。

各エントリーのフォーム、グラマーの修正をまとめて行っています。


Move “fixes” to a new line

railties/CHANGELOG.mdの修正です。

各エントリーのフォーム、グラマーの修正をまとめて行っています。


Merge pull request #28068 from kamipo/refactor_data_sources

Active Recordの修正です。

各メソッド各々で行っていたdata sourcesの取得処理を、メソッド(data_source_sql)に切り出して、そのメソッドを使用するよう修正しています。


Merge pull request #28356 from jasl/patch-1

activemodel/lib/active_model/type.rbの修正です。

ActiveModel::Type::Registryに存在しないメソッド(#add_modifier)の呼び出しがあったのを削除しています。


Merge pull request #28369 from mylake/reduce-postgresql-adapter-memory-bloat

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

PostgreSQLAdapter#load_additional_typesメソッドでtypesを取得する際に、DISTINCTで重複データ取得しないよう修正しています。

            query = <<-SQL
-              SELECT t.oid, t.typname, t.typelem, t.typdelim, t.typinput, r.rngsubtype, t.typtype, t.typbasetype
+              SELECT DISTINCT on (t.typname) t.oid, t.typname, t.typelem, t.typdelim, t.typinput, r.rngsubtype, t.typtype, t.typbasetype
               FROM pg_type as t
               LEFT JOIN pg_range as r ON oid = rngtypid
             SQL

schemaが大量にある場合に、不要なデータの取得を抑える為に、とのことです。DISTINCT ONという書き方出来るんですねえ。

が、そもそもtypenameでDISTINCTしてしまうのは良くない(typenameは同じ場合にそれらが関係あると断言してしまう)という事で、後ほどrevertされています。


Add CHANGELOG for #26226 [ci skip]

actionview/CHANGELOG.mdの修正です。

各view用のhelper method(image_tagjavascript_include_tag等)に、asset pipelineを使用しているかどうかを明示的に指定するためのオプション(skip_pipeline)を追加した対応(Make public asset use explicit)について、CHANGELOGにエントリーを追加しています。


Revert “Merge pull request #28369 from mylake/reduce-postgresql-adapter-memory-bloat”

先ほどコミットされたDISTINCTの対応をrevertしています。理由は先に書いた通り。


Remove obsolete warning about regular expression

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

numericalityの項にonly_integerを使用する場合の注意事項に、末尾に改行記号がある場合にマッチする旨説明記載されていたのですが、実際使用される正規表現/\A[+-]?\d+\z/であり、改行記号にはマッチしない為該当の説明を削除しています。