バグ報告 > フロント機能 > 2.11.1 受注編集画面でステータス変更するとエラーになる |
フロント機能
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
kwhr1972 |
投稿日時: 2011/7/27 15:59
対応状況: −−−
|
一人前 ![]() ![]() 登録日: 2011/4/1 居住地: 投稿: 74 |
2.11.1 受注編集画面でステータス変更するとエラーになる [EC-CUBE]
2.11.1 チェンジセットを随時適用中 [現象] チェンジセット(20981)を適用後からだと思うのですが、 いろいろカスタマイズしている為、申し訳ございませんが、 どなたか、再現性の確認をお願い致します。 ・購入時にポイントを使用する(分かりやすいため全部使う) ・受注管理で対応状況を「入金済」に変更して更新 →「最終保持ポイントがマイナス表示にならないように調整して下さい。」のエラーが出る /data/class/pages/admin/order/LC_Page_Admin_Order_Edit.phpの、lfCheckErrorで、 // 最終保持ポイント $arrValues['total_point'] = $objFormParam->getValue('custmer_point') - $arrValues['use_point']; になっていますが、注文時にポイントを減算しているはずなので、これだと2重に引き算しませんか? そのためにエラーになってしまうのかと思います。 お手数ですが、ご確認をお願い致します。 |
AMUAMU |
投稿日時: 2011/7/27 19:36
対応状況: −−−
|
神 ![]() ![]() 登録日: 2009/5/2 居住地: 東京都 投稿: 2712 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる なんか r20981 の修正後におかしいですね
チケット #1360 にコメントを残して詳しい調査に委ねたいと思います。 http://svn.ec-cube.net/open_trac/ticket/1360
|
habu |
投稿日時: 2011/7/28 10:16
対応状況: −−−
|
長老 ![]() ![]() 登録日: 2006/9/15 居住地: 投稿: 280 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる スペルミスが原因の様ですね。
まだ確認してはいませんが、 「custmer_point」→「customer_point」 に修正すると不具合解消されるかと思います。 r21019で修正してくださっている様です。 http://svn.ec-cube.net/open_trac/changeset/21019 追って、チケットにも追記しておきます。
|
kwhr1972 |
投稿日時: 2011/7/28 10:47
対応状況: −−−
|
一人前 ![]() ![]() 登録日: 2011/4/1 居住地: 投稿: 74 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる AMUAMUさん、habuさん、確認有難うございます。
報告ですが、 habuさんご指摘のスペルミスを修正しても、やはりおかしい様でした。 追伸: 昨日は慌てていて、管理機能のスレをフロント機能の方に書いてしまいました。 お詫びいたします。 |
habu |
投稿日時: 2011/7/28 12:20
対応状況: −−−
|
長老 ![]() ![]() 登録日: 2006/9/15 居住地: 投稿: 280 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる kwhr1972様
引用: 報告ですが、 こちらでも動作確認し、最新版でも不具合が再現する事を確認しました。 引き続き調査を行ない、コミットまで漕ぎつけようと思います。
|
habu |
投稿日時: 2011/7/28 13:45
対応状況: −−−
|
長老 ![]() ![]() 登録日: 2006/9/15 居住地: 投稿: 280 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる r21068 で修正してみました。
詳細はチケットに記しています。 http://svn.ec-cube.net/open_trac/ticket/1360#comment:6
|
kwhr1972 |
投稿日時: 2011/8/2 15:26
対応状況: −−−
|
一人前 ![]() ![]() 登録日: 2011/4/1 居住地: 投稿: 74 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる habuさん
連絡遅くなりましたが、修正有難う御座いました。 ですが、まだ問題が残っているようですので、 一旦、r20981をロールバックした状態で、ごまかしつつ運用します。 ぱっと見ですが、新規登録時と変更時で共通ロジックなのが難しさを増しているのかもと思ったり。。。 |
habu |
投稿日時: 2011/8/2 16:58
対応状況: −−−
|
長老 ![]() ![]() 登録日: 2006/9/15 居住地: 投稿: 280 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる kwhr1972さん
再度修正を行ないましたので、報告致します。 http://svn.ec-cube.net/open_trac/changeset/21101
|
kwhr1972 |
投稿日時: 2011/8/5 11:27
対応状況: −−−
|
一人前 ![]() ![]() 登録日: 2011/4/1 居住地: 投稿: 74 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる habuさん
ご対応&ご連絡有難うございます。 動作確認できました。助かりました! 結局、SC_Helper_DB_Ex::sfGetRollbackPointに渡しているadd_pointって不要だったんですかね? まあ、ポイントを与えた状態=発送済だと思うので、 そのステータスの注文情報を変更することはないだろうから問題ないのでしょうが、ちょっと疑問が残りました。 |
habu |
投稿日時: 2011/8/9 17:04
対応状況: −−−
|
長老 ![]() ![]() 登録日: 2006/9/15 居住地: 投稿: 280 |
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる kwhr1972さん
引用: 結局、SC_Helper_DB_Ex::sfGetRollbackPointに渡しているadd_pointって不要だったんですかね? ご確認ありがとうございます!仰る通りです。 「発送済み」の場合に、今回付与されたポイント分まで使用できてしまいますね。 修正版をコミットしました。 http://svn.ec-cube.net/open_trac/changeset/21165 http://svn.ec-cube.net/open_trac/changeset/21166 以下のチケットで管理されています。 http://svn.ec-cube.net/open_trac/ticket/1419
|
(1) 2 » |
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |