なるようになるブログ

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

rails commit log流し読み(2017/01/09)

2017/01/09分のコミットです。

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

railties/CHANGELOG.md


Merge pull request #27604 from prathamesh-sonpatki/dont-generate-erb-views-for-api-scaffold-controller

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

scaffold controller generatorに--apiフラグを指定した場合にviewファイルを生成しないよう修正しています。


Merge pull request #25427 from eugeneius/update_increment_documentation

activerecord/lib/active_record/persistence.rbのdoc及びrails guideのActive Record Callbacksの修正です。

increment!及びdecrement!メソッドのdocに、これらのメソッドではcallbacksは実行されない、指定されたattributesだけ更新されてrecord自体は保存されない、等の説明を追加しています。


Merge pull request #27416 from voray/remove-throw-from-action-cable

actioncable/app/assets/javascripts/action_cable/connection.coffeeの修正です。

connectionをopenしようとした際に、既にconnectionがopen済みだった場合に、Errorをthrowしていたのを、falseを返すよう修正しています。

  open: =>
     if @isActive()
       ActionCable.log("Attempted to open WebSocket, but existing socket is #{@getState()}")
-      throw new Error("Existing connection must be closed before opening")
+      false

ブラウザがスリープ状態になるデバイスでは頻繁に発生してしまう、かつ、logを見れば状態はわかる為、Errorはthrowしないようにした、との事です。