ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

エンティティのインスタンス生成について

Open nanasess opened this issue 9 years ago • 2 comments

Repository と Service で、エンティティのインスタンス生成をしている箇所があるが、エンティティのコンストラクタで実装すれば良いと思われる。

https://github.com/EC-CUBE/ec-cube/search?utf8=%E2%9C%93&q=%22public+function+new%22&type=Code

あえて、 Repository や Service でインスタンス生成をする理由があれば、その旨のコメントを追加する.

nanasess avatar Oct 02 '15 02:10 nanasess

@nanasess

以前いちど試みたのですが、

  • Doctrineのentity manager経由のデータ取得
  • FormTypeのview dataとmodel dataの変換

の挙動に影響ないかどうか検証しきれず、断念しました、、、 上記問題なければ、Entityで行うのが適切な気がします。

chihiro-adachi avatar Oct 05 '15 10:10 chihiro-adachi

手元の環境では問題なかったはずですので、テスト書いてみますね。

nanasess avatar Oct 05 '15 11:10 nanasess

現状は問題は発生してないので、クローズいたします。

xuelian311 avatar Jul 21 '23 02:07 xuelian311