Ticket #463 (closed バグ指摘: 修正済)

Opened 12 years ago

Last modified 11 years ago

受注管理 納品書の「ポイント値引」額の誤り

Reported by: kishik Owned by: somebody
Priority: Milestone: EC-CUBE2.4.0
Component: 管理画面 Version: 2.4.0RC-2
Keywords: Cc:
修正済み:

Description

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=3935&post_id=16717&order=0&viewmode=flat&pid=16700&forum=9#forumpost16717

yuchizo様より以下の通の指摘がありました。 2.4.0rc2でも該当箇所に間違いがあることを確認しています。

受注管理で 納品書を作成すると、1ポイントx円(変換レート=x) という設定にも関わらず、ポイント値引きは -ポイント数 となります。

受注時の金額計算などでは すでにポイント数×x という計算になっている為、気づきにくいのですが、納品書をよく見てみるとおかしな計算になっていることがわかります。

原因は、\data\class\SC_Fpdf.php の中でポイント値引額を計算している箇所で、ポイント数に 定数「POINT_VALUE」を乗じておらず、そのままポイント数を表示しているためでした。

以下の一文

$arrOrder[$i][3] = "- ".number_format($this->arrDisp['use_point'] + $this->arrDisp['discount']).$monetary_unit;

こちらを

$arrOrder[$i][3] = "- ".number_format(($this->arrDisp['use_point'] * POINT_VALUE) + $this->arrDisp['discount']).$monetary_unit;

と変更することで解決しました。

Change History

comment:1 Changed 12 years ago by kajiwara

  • Status changed from new to closed
  • Resolution set to 修正済
  • Milestone changed from EC-CUBE2.4.1 to EC-CUBE2.4.0

ご指摘ありがとうございます。>kishik様

r18011 にて対応させていただきました。

本件に関しては、改修点が明確な点、他に影響を及ぼさない点、及び多くのユーザ様にご迷惑をおかけする可能性がある点などを考慮し、2.4.0に含めさせていただきたいと思います。

よろしくお願いいたします。

comment:2 Changed 11 years ago by Seasoft

コミュニティ関連チェンジセット

Note: See TracTickets for help on using tickets.