バージョン選択

フォーラム

メニュー

オンライン状況

27 人のユーザが現在オンラインです。 (17 人のユーザが フォーラム を参照しています。)
登録ユーザ: 0
ゲスト: 27
もっと...

サイト内検索

バグ報告 > フロント機能 > 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


----------------
EC-CUBE公式エヴァンジェリスト
EC-CUBEインテグレートパートナー (株)スピリット・オブ
移転・拡張・高速化・問題解決
各種カスタマイズ・支援依頼承ります。

[url=h

habu
投稿日時: 2011/7/28 10:16
対応状況: −−−
長老
登録日: 2006/9/15
居住地:
投稿: 277
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる
スペルミスが原因の様ですね。

まだ確認してはいませんが、
「custmer_point」→「customer_point」
に修正すると不具合解消されるかと思います。

r21019で修正してくださっている様です。
http://svn.ec-cube.net/open_trac/changeset/21019

追って、チケットにも追記しておきます。


----------------
株式会社システムフレンド
羽生 賢太郎(Kentaro Habu)
改造専門店・EC-CUBE工房

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
居住地:
投稿: 277
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる
kwhr1972様

引用:
報告ですが、
habuさんご指摘のスペルミスを修正しても、やはりおかしい様でした。
ご報告ありがとうございます。
こちらでも動作確認し、最新版でも不具合が再現する事を確認しました。
引き続き調査を行ない、コミットまで漕ぎつけようと思います。


----------------
株式会社システムフレンド
羽生 賢太郎(Kentaro Habu)
改造専門店・EC-CUBE工房

habu
投稿日時: 2011/7/28 13:45
対応状況: −−−
長老
登録日: 2006/9/15
居住地:
投稿: 277
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる
r21068 で修正してみました。

詳細はチケットに記しています。
http://svn.ec-cube.net/open_trac/ticket/1360#comment:6


----------------
株式会社システムフレンド
羽生 賢太郎(Kentaro Habu)
改造専門店・EC-CUBE工房

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
居住地:
投稿: 277
Re: 2.11.1 受注編集画面でステータス変更するとエラーになる
kwhr1972さん

再度修正を行ないましたので、報告致します。

http://svn.ec-cube.net/open_trac/changeset/21101


----------------
株式会社システムフレンド
羽生 賢太郎(Kentaro Habu)
改造専門店・EC-CUBE工房

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
居住地:
投稿: 277
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


----------------
株式会社システムフレンド
羽生 賢太郎(Kentaro Habu)
改造専門店・EC-CUBE工房

(1) 2 »
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


EC-CUBEペイメント

公式ストアEC-CUBE4系デザインテンプレート続々リリース中

統計情報

総メンバー数は70,505名です
総投稿数は100,697件です

投稿数ランキング

1
seasoft
7333
2
468
2922
3
AMUAMU
2712
4
nanasess
2127
5
umebius
1931
6
yuh
1612
7
red
1437
8
h_tanaka
1076
9
tsuji
926
10
fukap
907
11
shutta
835
12
tao_s
793
13 ramrun 789
14 karin 689
15 sumida 641
16
homan
633
17 DELIGHT 572
18
patapata
502
19
flealog
485
20 tonton 437


ネットショップの壺

EC-CUBEインテグレートパートナー

Copyright© EC-CUBE CO.,LTD. All Rights Reserved.