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

Opened 10 years ago

Last modified 10 years ago

SC_Helper_Purchase#registerOtherDeliv を削除

Reported by: Seasoft Owned by: Seasoft
Priority: Milestone: EC-CUBE2.11.0
Component: フロント Version: 2.5-dev
Keywords: Cc:
修正済み:

Description

どういったケースでこのロジックが必要となるのか理解できない。

このロジックに入るケースではDBエラーが発生し、分析が困難な状況。

▼▼▼ エラーメッセージ ▼▼▼
https://1.eccube.local/ec25d/shopping/confirm.php?

SERVER_ADDR: *
REMOTE_ADDR: *
USER_AGENT: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US) AppleWebKit/534.10 (KHTML, like Gecko) Chrome/8.0.552.215 Safari/534.10

SQL: INSERT INTO dtb_other_deliv(name01,name02,kana01,kana02,tel01,tel02,tel03,zip01,zip02,pref,addr01,addr02,customer_id,order_deliv_id) VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?)

MDB2 Error: no such field

prepare: [Error message: Unable to create prepared statement handle]
[Last executed query: SELECT NEXTVAL('dtb_other_deliv_other_deliv_id_seq')]
[Native message: ERROR:  column "order_deliv_id" of relation "dtb_other_deliv" does not exist
LINE 1: ...,tel03,zip01,zip02,pref,addr01,addr02,customer_id,order_deli...
                                                             ^]


/home/eccube/ec25d/html/shopping/confirm.php 34:LC_Page_Shopping_Confirm_Ex->process
/home/eccube/ec25d/data/class_extends/page_extends/shopping/LC_Page_Shopping_Confirm_Ex.php 56:LC_Page_Shopping_Confirm->process
/home/eccube/ec25d/data/class/pages/shopping/LC_Page_Shopping_Confirm.php 65:LC_Page_Shopping_Confirm->action
/home/eccube/ec25d/data/class/pages/shopping/LC_Page_Shopping_Confirm.php 168:SC_Helper_Purchase->completeOrder
/home/eccube/ec25d/data/class/helper/SC_Helper_Purchase.php 70:SC_Helper_Purchase->registerOtherDeliv
/home/eccube/ec25d/data/class/helper/SC_Helper_Purchase.php 235:SC_Query->insert
/home/eccube/ec25d/data/class/SC_Query.php 477:SC_Query->query
/home/eccube/ec25d/data/class/SC_Query.php 729:SC_Query->prepare
/home/eccube/ec25d/data/class/SC_Query.php 835:MDB2_Driver_pgsql->prepare
/home/eccube/ec25d/data/module/MDB2/Driver/pgsql.php 967:MDB2_Driver_Common->raiseError
/home/eccube/ec25d/data/module/MDB2.php 1497:PEAR->raiseError
/home/eccube/ec25d/data/module/PEAR.php 557:MDB2_Error->MDB2_Error
/home/eccube/ec25d/data/module/MDB2.php 1009:PEAR_Error->PEAR_Error
▲▲▲ エラーメッセージ ▲▲▲

一旦削除して、様子を見ようと思う。

Change History

comment:1 Changed 10 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • Status changed from new to assigned
  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.0

comment:2 Changed 10 years ago by nanasess

  • Milestone changed from EC-CUBE2.5.0 to EC-CUBE2.5.0beta

中途半端な状態で申し訳ございません.

下記ロジックの移植ですが, 不要でしょうか?

source:branches/comu-ver2/data/class/pages/shopping/LC_Page_Shopping_Complete.php@18071#L592

念のため, マイルストーンを差し戻します. 問題なければ, 2.5.0 へ戻してください.

comment:3 Changed 10 years ago by Seasoft

会員では送り先を都度入力する機能は無かったと記憶しています。(あったら、それはそれで便利そうな気はしますが。) そのため、このロジックを通ることはないと推測しました。

過去の遺産なのかなぁ、などと想像しています。

たとえば、過去のバージョンでは「会員でもお届け先を都度手入力できた」や「現在でも都度手入力できる」といった事情をご存知ないでしょうか?

comment:4 Changed 10 years ago by nanasess

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.0

会員では送り先を都度入力する機能は無かったと記憶しています。

確かに, よくよく考えてみれば, 住所を追加登録しても, 異なる住所が deliv_* に入ることはありえませんね.

たとえば、過去のバージョンでは「会員でもお届け先を都度手入力できた」や「現在でも都度手入力できる」といった事情をご存知ないでしょうか?

うろ覚えですが, 0.9.x の頃はあったような気がします.

comment:5 Changed 10 years ago by Seasoft

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.