受注編集画面で商品変更時に使用ポイントエラー

[no61]

受注編集画面で商品を減らした場合に使用ポイントでマイナスチェックのエラーがでる。

関連： #1419

[no61]

r20981 で修正しました。

[AMUAMU]

r20981 適用後に問題が報告されています。

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=8514&forum=8&post_id=41313

詳しい調査が必要かと・・・いったん修正済みのチェックを外しておきます。

[habu]

r21068 で、r20981 をロールバックし、再修正してみました。

---

■不具合原因と修正方法：
商品数を減らした場合に、現在ポイントがマイナスになってしまうのは、
先にSC_Helper_Purchase::sfUpdateOrderStatus関数で
今回加算分のポイントを引いているにも関わらず、
後から呼び出されたSC_Helper_DB_Ex::sfGetRollbackPoint関数にて、
購入前のポイントの計算時に、今回加算分のポイントを引いてしまっている事が原因。

それで、SC_Helper_DB_Ex::sfGetRollbackPoint関数にて、
今回加算分のポイントを引かない様に修正した。


■呼び出し階層：
・LC_Page_Admin_Order_Edit::doRegister()

→ SC_Helper_Purchase::registerOrder
→ SC_Helper_Purchase::sfUpdateOrderStatus()

・LC_Page_Admin_Order_Edit::setOrderToFormParam()

→ SC_Helper_DB_Ex::sfGetRollbackPoint()

■SC_Helper_Purchase::sfUpdateOrderStatus関数の挙動

• 「発送済み」ステータスに変更された場合、今回購入分のポイントを加算。
• 「発送済み」ステータスから、別のステータスに変更した時は、今回購入分のポイントを減算。
  
• 「発送済み」ステータス以外から、「発送済み」ステータス以外への変更時や、「発送済み」ステータス以外で、ステータスを変更していない場合は、もともと、今回購入分のポイントを加算していない。
• 「発送済み」ステータスで、ステータスを変更していない場合は、ポイント増減無し。（今回購入分のポイントを一度減算して、再度加算するのでプラスマイナスゼロ）

■影響範囲の確認：
SC_Helper_DB_Ex::sfGetRollbackPoint関数を利用しているのは、
以下の２ファイル(各ファイル１箇所)のみ。

• data/class/pages/admin/order/LC_Page_Admin_Order_Disp.php
• data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php

---

修正していく中で、別の不具合も発見しましたが、別チケットを切ろうと思います。

[habu]

r21068 での修正ですが、 現在保持しているポイント以上のポイントを利用できる不具合が発生する事が分かりました。。 （今回加算ポイント分まで使用できてしまいます）

再度、修正済みのチェックを外しておきます。

[habu]

r21101 で修正しました。

[kajiwara]

no61さん、habuさん、AMUAMUさん

本件、ご対応ありがとうございました。
（habuさん、詳細な調査をいただき、誠にありがとうございます。）

本チケットに関しては、修正が確認できましたので、クローズします。

[habu]

その後、対応状況が「発送済み」や「キャンセル」だった場合に、
使用ポイント欄に、保持ポイントを超えるポイントを設定できてしまう不具合が見つかりましたので
r21165 にて対応しました。
（#1419 に紐付けています）