Closed yupix closed 4 months ago
ClientActions系のクラスにあるメソッドのIDに関する引数を廃止する
ClientActions系のクラスはモデルのIDを参照して使うことを想定している。 過去はManagerのID引数にNoneが渡される可能性があったこと等の理由が重なり引数として残していたが今はほぼ全てNoneではなくあくまで上書き用として残してある状態になる。
この上書きというのが問題で、上書きできるからという理由で本来使うべきメソッドを使わなくなる可能性がある。 そういったことを行わせないために廃止したい
あとadmin系が残ってそう
多分消し終わった
What
ClientActions系のクラスにあるメソッドのIDに関する引数を廃止する
Why
ClientActions系のクラスはモデルのIDを参照して使うことを想定している。 過去はManagerのID引数にNoneが渡される可能性があったこと等の理由が重なり引数として残していたが今はほぼ全てNoneではなくあくまで上書き用として残してある状態になる。
この上書きというのが問題で、上書きできるからという理由で本来使うべきメソッドを使わなくなる可能性がある。 そういったことを行わせないために廃止したい