console e:f:generate が動作しないのを修正

[ryosukemori]

概要(Overview・Refs Issue)

dockerコンテナ内での

bin/console e:f:generate

が、以下の複数のエラーで動作しない

• bheller/images-generator 内のGD2関数が定義されていない
• GenerateDummyDataCommandのautowiringで、$entityManagerがnull
• オプションなし実行で、PHPのメモリー上限に達している

方針(Policy)

dockerを利用したダミーデータを正常に作成できること。
bin/console e:f:generate　を、オプションなしで正常に動作させられること。

実装に関する補足(Appendix)

以下の件については、他にも利用されている箇所がないか要確認

// 動作しない
EntityManager $entityManager
//　動作する
EntityManagerInterface $entityManager

テスト（Test)

ピュアなインストール状態でのコマンド動作を確認した。

docker-compose exec ec-cube bin/console e:f:generate

相談（Discussion）

オプションなしで実行できる方針の場合に、
phpのメモリーリミットを変更は、この方針で問題ないか。

修正パターン候補は以下かと思います。

1. php.iniでメモリーリミットを変更する (今回の修正パターン)
2. オプションなし実行の場合の生成ダミーデータ量を調整する

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[nanasess]

#4955 にも PR しましたが、 プラグインインストール時に 700MB 近くはメモリを消費するため、 memory_limit=784M くらいにしても良いかなと思います。
(Composer API が memory_limit=1.5G まで自動的に伸張するので、ここでの上限設定はそれほど意味が無さそうです)

[okazy]

ありがとうございます。
修正内容、動作確認してエラーなくデータが作成できることを確認しました。

プルリク先のブランチを 4.1-core に変更しました。
落ちているテストは本件と関係ありません。

[okazy]

ありがとうございます。マージしました。