受注登録画面で配送先追加時にプラグインフィールドを考慮していない

[izayoi256]

概要(Overview)

複数配送先機能がONの状態で、受注登録画面で「お届け先を新規追加」ボタンを押下すると、値を引き継げないフィールドが存在する。

詳細に調査した訳ではありませんが、下記の箇所でPOST値ではなく$TargetOrderをセットしているため、'mapped' => falseなフィールドの値がセットされないものと推測します。

https://github.com/EC-CUBE/ec-cube/blob/3.0.14/src/Eccube/Controller/Admin/Order/EditController.php#L265-L274

環境 (environment)

• EC-CUBE: 3.0.14

[yosshieee]

@izayoi256 さん
お手数ですが、具体的に項目を教えて頂けますと幸いです。

補足

OrderType, OrderTypeが持っているentityの各Typeの、
mapped設定をみましたが、falseのものはありませんでした。

[izayoi256]

@t-nagahashi
本体側ではなくプラグインで追加したフィールドです。
手元で試したところ、OrderType::buildForm()に下記のコードを追記することで現象が確認できました。

$builder->add('plg_hoge', 'text', array('mapped' => false));

[yosshieee]

@izayoi256 さん
本体で対応するのは、難しいのではと考えております。
（プラグイン側にてpost時の値を取得、自前で設定する？）

もし何か、案がありましたら[WIP]でPRいただけますと幸いです。

[izayoi256]

@t-nagahashi
解決方法の検討はしますが、仕様とするにはちょっと乱暴というか、「受注登録編集画面をカスタマイズするプラグインと複数配送機能は共存できない」のも同然だと思います。

[yosshieee]

@izayoi256 さん

解決方法の検討はしますが、仕様とするにはちょっと乱暴というか、

誤解を招きすみません。
仕様なので対応しないという意味ではなく、
EC-CUBE本体側で対応可能かどうか難しいのでは？という表現でした。

プルリクありがとうございざいます。
確認させていただきます。

[Yangsin]

https://github.com/EC-CUBE/ec-cube/pull/2307
でプルリクエストをいただいておりましたが、取り込む事ができないままリポジトリが分割されたため、いちどクローズさせていただきました。
@izayoi256 面倒かけますが必要に応じて再度 Pull Request いただけますと幸いです。