プラグインでのDBALベストプラクティスについて

[ttsuru]

現状、プラグインのマイグレーション時に、 executeUpdate() を利用していますが、変数をそのままSQL文に入れるなど、あまりよいとは言えない例が多くあります。

・冪等性を考慮したアップデートについて
http://qiita.com/iakio/items/adf1f923d1cfccbba765
こちらにあるような形が良いかと思います。

・INSERT または UPDATE について
現状、なぜか executeUpdate を利用していますが、こちらは insert もしくは update のほうが安全にSQLを実行できます。

http://doctrine-orm.readthedocs.org/projects/doctrine-dbal/en/latest/reference/data-retrieval-and-manipulation.html

[ttsuru]

他にもありますが、ぜひ、migration自体のベストプラクティスを定義していった方が良いかと思います。

[nanasess]

本当は、やっちゃいけないのかもしれませんが、こんなことを試みたことがあります。
nanasess@e42ee29

migration のベストプラクティスを定義しない理由は無いと思いますが、リソースがそこまで追いついていないのが実情かと...

[ttsuru]

@nanasess
気づいたタイミングでは何らかのアクションを起こしていますが、現状、プラグイン作成者によっては危険な表記が多い点が非常に気になります。

[Yangsin]

何がベストプラティクスとして定義できるかも、用意でききれていないのですが、2系でプラグイン開発時のセキュリティガイド作ったように、そうしていきたいのは山々です。
https://github.com/EC-CUBE/ec-cube.github.io
へも、お気づきの点などご協力いただけると助かります

[ttsuru]

@Yangsin
どこかのタイミングで整理します。

[chihiro-adachi]

ec-cube.github.ioにマイグレーションの作成についてissue起案しました
EC-CUBE/ec-cube.github.io#71

[ryo-endo]

上記のとおり、github.io側にIssue作成させていただきましたので、こちらはCLOSEさせていただきます。
EC-CUBE/ec-cube.github.io#71