Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

出荷CSVでの受注ステータス変更を実装 #3374

Merged
merged 10 commits into from
Jul 24, 2018

Conversation

nanasess
Copy link
Contributor

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

OrderStateMachine::apply() をコールすると、 Order::OrderStatus ではなく、 OrderStatus オブジェクト自身の ID が変更されてしまう。
このため、後続の処理で EntityManager::flush() をコールすると、Doctrine は OrderStatus::id を更新しようとする。
これを防ぐために、 OrderStatus を detach して更新対象から除外している

テスト(Test)

  • Codeception が通るのを確認

相談(Discussion)

- ステータス一括更新メソッドを使いまわすよう修正
- 使用しなくなった JavaScript を削除
以下は使用しなくなったため、新しいテストケースに更新
- admin_order_bulk_order_status
- admin_shipping_mark_as_shipped
- なぜか先行する行で更新されている場合があるため refresh する
@ryo-endo ryo-endo added this to the 3.n.0 milestone Jul 21, 2018
@chihiro-adachi chihiro-adachi merged commit 26236da into EC-CUBE:experimental/sf Jul 24, 2018
@nanasess nanasess deleted the shipped-csv branch December 3, 2018 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants