ExercisesのURLがChapter 9だ...

あれ本当だ...。修正しました!

ありがとう! LGTM 🍺

improved attr update method because 'update_attributes' is deprecated

deprecatedなんですか？

@hanazuki
http://apidock.com/rails/ActiveRecord/Base/update_attributes
にはdeprecatedもしくはmovedって書いてある。
どちらにせよaliasとして update があるから使わないほうがいいかなと思ってるのだけどどうだろうか。

@takuminnnn それはActiveRecord::Base#update_attributesが廃止されてActiveRecord::Persistence#update_attributesに移ったといういう意味の記述ですよね．

update_attributesを使うべきではないとはどこにも書かれていないと思いますが，仮にそうだったとしてupdateを使うべきであって，update_columnsにするのはおかしくないですか．

@hanazuki なるほど確かに自分の早とちりでした。ただ、今回のExerciseではTODOにも示した通り、 update_attribute を update_columns に変更する問題であるため update を使いませんでした。

@takuminnnn なるほどそういう設問だったんですね（いま問題よみました，ごめんなさい）．

ただ，設問の意図は，update_attributeを単に2回呼ぶと単一トランザクションにならない，という問題を解決することだと思うので，それは元々のコードのようにupdate_attributes（ないしupdate）を使うことで解決できていると思います．

今回はたまたま関係ないのですが，update_columnsがバリデーションを省略することなどを説明しないあたりあまり良くない課題と思いました．

@hanazuki

ただ，設問の意図は，update_attributeを単に2回呼ぶと単一トランザクションにならない，という問題を解決することだと思うので，それは元々のコードのようにupdate_attributes（ないしupdate）を使うことで解決できていると思います．

確かに設問の意図的にはそちらでも問題ないですね。

validation, callbackをスキップする update_columns の挙動を考えたら、元のコードの方が後々ハマらなくて済みそうなので戻しておきます。ツッコミありがとう!

よいとおもいます〜

@hanazuki ありがとう!