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

受注登録・編集 #3385

Merged
merged 27 commits into from
Jul 24, 2018
Merged

Conversation

chihiro-adachi
Copy link
Contributor

概要(Overview・Refs Issue)

  • 赤電を登録できるように対応
  • ステータス更新処理に対応
  • 明細変更時の在庫の確認・戻し処理に対応

方針(Policy)

  • 明細に赤電を登録できるように対応
    • 受注明細にマイナスの金額/マイナスの個数を登録できるように修正
  • ステータス更新時のステートマシンの組み込みに対応
  • 明細変更時の在庫の確認・戻し処理に対応

実装に関する補足(Appendix)

  • 発送済のステータス変更時の、「すべての出荷が出荷済」というガード条件は削除し、発送済ステータス変更時に出荷日が登録されるように修正しています。
  • 在庫のProcessorはStockDiffProcessorで実装

テスト(Test)

  • 既存のテストにパスすることを確認

@chihiro-adachi chihiro-adachi added this to the 3.n.0 milestone Jul 23, 2018
@ryo-endo ryo-endo merged commit 93516f7 into EC-CUBE:experimental/sf Jul 24, 2018
@itotoru
Copy link

itotoru commented Oct 21, 2018

突然のコメントで大変恐縮です。

赤伝票の際の在庫数量の計算につきまして
お伺いしたく、こちらにコメントをさせて頂きました。

当方ローカル環境のEC-CUBE4(正式版)で、例えば以下のように新規受注登録を行い、

例)
彩のジェラートCUBE cube-03 / ( フレーバー: バニラ / サイズ: 16mm × 16mm ) ×2個
チェリーアイスサンド sand-01 ×1個

を受注登録

その際の在庫数は

彩のジェラートCUBE 無制限
チェリーアイスサンド 100→99

となります。

その後、この受注について以下のように変更を行い更新します。

彩のジェラートCUBE cube-03 / ( フレーバー: バニラ / サイズ: 16mm × 16mm ) ×2個
チェリーアイスサンド sand-01 ×- 1個

そうすると在庫数が

彩のジェラートCUBE 無制限
チェリーアイスサンド 99→101

となってしまいます。

通常赤伝票で1 → -1 と変更した場合
その数だけ(1だけ在庫に戻る)ような気がしますが
計算方法が加算されていないでしょうか?

在庫算出のロジックまでは追えていないので何ともですが、
2系の際に挑戦したことがありまして、うろ覚えですが
赤伝票の場合の在庫数算出と、通常の場合の在庫数算出を分けないと
おかしいことになってしまったような記憶があります・・・。

お忙しいところ大変恐縮ですが、ご確認、コメント頂ければ幸いです。
どうぞよろしくお願いいたします。

@nanasess
Copy link
Contributor

@itotoru ご報告ありがとうございます。埋もれてしまうといけないので、 issue 作成しておきました
#3943

1 similar comment
@nanasess
Copy link
Contributor

@itotoru ご報告ありがとうございます。埋もれてしまうといけないので、 issue 作成しておきました
#3943

@chihiro-adachi chihiro-adachi deleted the improve/akaden branch October 23, 2018 03:46
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.

4 participants