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

Opened 9 years ago

Last modified 8 years ago

「お届け先の指定」に戻った際に、選択した状態を引き継いでいない

Reported by: shutta Owned by: pingpong
Priority: Milestone: EC-CUBE2.12.0
Component: フロント Version: 2.11.2
Keywords: Cc: Seasoft
修正済み: yes

Description

購入時に「お届け先の指定」に戻った際に、お届け先情報が引き継がれておらず、選択した追加登録住所がチェックされずに、最初の項目の「会員登録住所」にチェックが入った状態になっている。

Change History

comment:1 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.3 to EC-CUBE2.11.4(仮)

comment:2 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.5 to EC-CUBE2.12.0alpha

comment:3 Changed 9 years ago by pingpong

  • Owner changed from somebody to pingpong
  • Status changed from new to assigned

comment:4 follow-up: ↓ 5 Changed 9 years ago by pingpong

  • 修正済み set

購入フローはPOST値での保持でなく、一時中データやSESSIONへの書込みで保持していたので、 SESSIONshipping?で保存するようにして保持するようにしました。 ※別のキーのほうが良いのかもしれません。 ※GET値で保持するようにする仕組みが見られましたが、ほとんど実装されていないので、セッションでの維持に変更しました

comment:5 in reply to: ↑ 4 Changed 9 years ago by Seasoft

  • 修正済み unset

PHP 5.2 / PostgreSQL 9.0 でエラーが発生しておりますので、大変に恐縮ですが、一旦差し戻しをさせて頂きます。
r21570

SQL: SELECT * , ? as deliv_check FROM dtb_other_deliv WHERE customer_id = ? AND other_deliv_id = ?   

MDB2 Error: unknown error

prepare: [Error message: Unable to create prepared statement handle]
[Last executed query: DEALLOCATE PREPARE mdb2_statement_pgsql_65c5f7a69ef56d705a0f52f12d5cf87aae0922548]
[Native message: ERROR:  パラメータ$1のデータ型が決定できません]

おそらく、値を DB に渡す必要はなく、PHP ロジックで解決するのが適切ではないかと思います。

また、ソースのインデントは、タブではなく、半角スペース(4文字単位)の使用をお願いいたします。

comment:6 Changed 9 years ago by Seasoft

  • Cc Seasoft added
  • 修正済み set

dtb_order_temp.deliv_check 自体が、結構トリッキーに見受けられるので、セッションで済ます方向で実装してみました。

r21629

comment:7 Changed 8 years ago by kajiwara

  • Status changed from assigned to closed
  • Resolution set to 修正済

shuttaさん、Seasoftさん、pingpongさん

本件、ご報告ならびにご対応いただきありがとうございました。

現状、こちらにて不具合もでておりませんので、こちらにてクローズとさせていただきます。

Note: See TracTickets for help on using tickets.