受注編集時に不必要なデータが更新される

[nobuhiko]

概要(Overview)

受注編集時に不必要なデータが更新される

期待する内容(Expect) or 要望 (Requirement)

再現手順(Procedure)

環境 (environment)

• EC-CUBE: 4.x.x

関連情報 (Ref)

src/Eccube/Form/Type/Admin/OrderType.php

    // 会員受注の場合、会員の性別/職業/誕生日をエンティティにコピーする
    if ($Customer = $Order->getCustomer()) {
        $Order->setSex($Customer->getSex());
        $Order->setJob($Customer->getJob());
        $Order->setBirth($Customer->getBirth());
    }

[nobuhiko]

$Customer->getSex() など値がない時に、システムエラーが発生する

[kazumiiiiiiiiiii]

@nobuhiko さん、いつもありがとうございます。
こちら、以下の再現方法で検証してみましたが、システムエラーは確認できませんでした。

1. 会員登録時に「生年月日」「性別」「職業」を未設定のまま登録を行う。
   　（dtb_customerを確認。sex_id, job_id, birthがそれぞれNULLの状態）
2. 商品を登録した会員で購入。
3. 「管理画面/受注管理」から、2で登録された注文情報を編集し、保存。

こちらのシステムエラーの問題は、すでに解決済みでしょうか？
@nobuhiko さんが確認されている、発生する環境や再現手順などあれば、教えていただけないでしょうか？

または、「受注編集時に不必要なデータが更新される」という仕様の変更してほしいということでしょうか？

[nobuhiko]

@kazumiiiiiiiiiii debugモードだとエラーが出ないでしょうか？なにぶんこれを書いたのが2018年だったので、記憶がありません。

[kazumiiiiiiiiiii]

@nobuhiko ご確認ありがとうございます。

なにぶんこれを書いたのが2018年だったので、記憶がありません。

回答が遅くなり、申し訳ありません。

.envでAPP_ENV=devで実行してみましたが、システムエラーは発生しませんでした。

「受注編集時に不必要なデータが更新される」状態に関しては、仕様として問題があると思うので、4.0.x以降となりますが、修正を行なっていきたいと考えております。

[okazy]

新規受注の場合には必要な処理なので、「受注の性別/職業/誕生日が設定されていなければ情報を更新」にするのがいいかと思います。
認識が間違っていました。一つ下のコメントを参照 (2021/03.24 追記)

ec-cube/src/Eccube/Form/Type/Admin/OrderType.php

Lines 381 to 386 in f5dd1c8

	// 会員受注の場合、会員の性別/職業/誕生日をエンティティにコピーする
	if ($Customer = $Order->getCustomer()) {
	$Order->setSex($Customer->getSex());
	$Order->setJob($Customer->getJob());
	$Order->setBirth($Customer->getBirth());
	}

[okazy]

Issue 検討会より

$Order->getCustomer() が null の場合は非会員購入を表す。
初回受注作成時の判定は $Order->getId() のチェックで行える。

ec-cube/src/Eccube/Form/Type/Admin/OrderType.php

Line 382 in f5dd1c8

if ($Customer = $Order->getCustomer()) {

[chihiro-adachi]

Sex/Job/Birthは、受注の新規登録時かつ会員購入時のみ更新する

[chihiro-adachi]

#4997 で修正しました。