Ticket #843 (closed 新規開発: 修正済)

Opened 13 years ago

Last modified 12 years ago

複数配送先の指定

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

Description

配送先を複数指定可能とする.

Attachments

受注関連情報.pdf Download (141.5 KB) - added by nanasess 12 years ago.

Change History

Changed 12 years ago by nanasess

comment:1 follow-up: ↓ 2 Changed 12 years ago by nanasess

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

r19759 受注関連を下記に修正しました.

comment:2 in reply to: ↑ 1 Changed 12 years ago by eccuore

モバイルサイトの再注文で、下記エラーが発生しています。

FATAL Error(256) D:\pleiades-all-in-one-php-jre_20090426\xampp\htdocs\version-2_5-dev\data\class\SC_Query.php:838 https://localhost/version-2_5-dev/html/shopping/payment.php?PHPSESSID=b6309966098da14feecdbade730e4bbb&transactionid=6789335cf27da1f6d6b68b40e895cea072b36b84
SERVER_ADDR: 127.0.0.1
REMOTE_ADDR: 127.0.0.1
USER_AGENT: DoCoMo/2.0 P903i(c100;TB;W24H12)

SQL: SELECT payment_id, payment_method, rule, upper_rule, note, payment_image FROM dtb_payment WHERE del_flg = 0 AND payment_id IN ()  ORDER BY rank DESC 

MDB2 Error: syntax error

_doQuery: [Error message: Could not execute statement]
[Last executed query: PREPARE mdb2_statement_mysql_13cbe0b883e9d9689a2318a8c2a5ae791bfee889a4 FROM 'SELECT payment_id, payment_method, rule, upper_rule, note, payment_image FROM dtb_payment WHERE del_flg = 0 AND payment_id IN ()  ORDER BY rank DESC ']
[Native code: 1064]
[Native message: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')  ORDER BY rank DESC' at line 1]


\version-2_5-dev\html\shopping\payment.php 34:LC_Page_Shopping_Payment_Ex->process
\version-2_5-dev\data\class_extends\page_extends\shopping\LC_Page_Shopping_Payment_Ex.php 56:LC_Page_Shopping_Payment->process
\version-2_5-dev\data\class\pages\shopping\LC_Page_Shopping_Payment.php 67:LC_Page_Shopping_Payment->action
\version-2_5-dev\data\class\pages\shopping\LC_Page_Shopping_Payment.php 140:SC_Helper_Purchase->getPayment
\version-2_5-dev\data\class\helper\SC_Helper_Purchase.php 356:SC_Query->select
\version-2_5-dev\data\class\SC_Query.php 152:SC_Query->getAll
\version-2_5-dev\data\class\SC_Query.php 264:SC_Query->prepare
\version-2_5-dev\data\class\SC_Query.php 835:MDB2_Driver_mysql->prepare
\version-2_5-dev\data\module\MDB2\Driver\mysql.php 1123:MDB2_Driver_mysql->_doQuery
\version-2_5-dev\data\module\MDB2\Driver\mysql.php 798:MDB2_Driver_Common->raiseError
\version-2_5-dev\data\module\MDB2.php 1497:PEAR->raiseError
\version-2_5-dev\data\module\PEAR.php 557:MDB2_Error->MDB2_Error
\version-2_5-dev\data\module\MDB2.php 1009:PEAR_Error->PEAR_Error

comment:3 Changed 12 years ago by nanasess

r19963 時点

  • 管理画面 -> 受注編集
    • 規格付きの商品を追加した場合に, 規格が正常に反映されない
    • 前項の商品を複数配送割り当てしようとした際に, 商品名等が正常に反映されない
    • 入力チェックが不十分. 特に必須チェック

SC_FormParam の入力チェックは配列に使えないので, 1次元のフォームパラメータにしましたが, そのために弊害が発生している模様です.

また, 再現ケース不明ですが, 複数配送割り当て小画面のプルダウンが正常に反映されない場合があります. 小画面表示よりも, レイヤー表示の方が安全かつシンプルに実装可能かもしれません.

また, ハードコーディング多数なので, 要リファクタリングです

comment:4 Changed 12 years ago by AMUAMU

このチケットに関連しての修正において、#766 (二重に注文が入る不具合) の再発危険性が上がっている可能性があります(※ソース査読のみによる想定)。

現状では複数配送先の指定に関する修正が頻繁なため、そちらを優先し #766 の検討及び修正は安定後としますが、関連項目として、こちらにもコメントしておきます。

comment:5 follow-up: ↓ 6 Changed 12 years ago by Seasoft

管理機能「受注管理 > 新規受注入力」のポップアップ画面(/admin/order/multiple.php)にて、「お届け先」プルダウンの選択肢が表示されない模様。「都道府県を選択」のみ出る。

comment:6 in reply to: ↑ 5 Changed 12 years ago by nanasess

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

Seasoft への返信

管理機能「受注管理 > 新規受注入力」のポップアップ画面(/admin/order/multiple.php)にて、「お届け先」プルダウンの選択肢が表示されない模様。「都道府県を選択」のみ出る。

r20468 で修正しました.

複数配送機能については, おおむね動作しているようですので, 本チケットは close します. その他複数配送機能に起因する不具合に関しては, 別チケット発行でお願いします.

Note: See TracTickets for help on using tickets.