受注編集時にすでに受注にセットされている顧客情報を書き換えない

[yshirai-joolen]

概要(Overview・Refs Issue)

管理画面で受注を編集する時に、受注にセットされている顧客情報を、その時点の顧客情報で上書きしないようにします。
#4059

方針(Policy)

Issueでの議論では初回受注作成時の判定は $Order->getId() のチェックで行うということでしたが、すぐ後のコードでnull === $Order->getOrderStatus()を用いて初回の作成を判定していたため、その中に処理をまとめました。

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

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

レビュワー確認項目

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

[codecov-io]

Codecov Report

Merging #4997 (49ca093) into 4.0 (23da54c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##                4.0    #4997   +/-   ##
=========================================
  Coverage     76.19%   76.20%           
  Complexity     6124     6124           
=========================================
  Files           437      437           
  Lines         20744    20744           
=========================================
+ Hits          15806    15807    +1     
+ Misses         4938     4937    -1     

Flag	Coverage Δ	Complexity Δ
tests	76.20% <100.00%> (+<0.01%)	0.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ	Complexity Δ
src/Eccube/Form/Type/Admin/OrderType.php	95.45% <100.00%> (ø)	41.00 <0.00> (ø)
.../Eccube/Doctrine/Common/CsvDataFixtures/Loader.php	82.05% <0.00%> (+2.56%)	16.00% <0.00%> (ø%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23da54c...49ca093. Read the comment docs.

[chihiro-adachi]

@yshirai-joolen
ありがとうございます。
動作確認しました。問題ありません。

[chihiro-adachi]

@matsuoshi
テストコードを追加してます。
テスト通りましたらマージお願いします。

[codecov-commenter]

Codecov Report

Merging #4997 (035b001) into 4.1 (e955010) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                4.1    #4997      +/-   ##
============================================
- Coverage     68.35%   68.35%   -0.01%     
  Complexity     6036     6036              
============================================
  Files           456      456              
  Lines         24945    24945              
============================================
- Hits          17052    17050       -2     
- Misses         7893     7895       +2     

Flag	Coverage Δ
E2E	57.17% <25.00%> (-0.06%)	⬇️
Unit	75.87% <100.00%> (+0.04%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/Eccube/Form/Type/Admin/OrderType.php	62.40% <100.00%> (ø)
...orage/Handler/SameSiteNoneCompatSessionHandler.php	88.05% <0.00%> (-2.99%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e955010...035b001. Read the comment docs.