バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

質問 > フロント機能 > 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・

フロント機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
ゲスト
投稿日時: 2012/7/9 22:48
対応状況: −−−
会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
会員登録をしないで購入 を選択
 ↓
(step1)お届け先と注文者を入力
 ↓
(step2)お届け日やお届け時間を選ぶ欄が2つ表示される(1つでいいのでは?)
 ↓
(step3)入力情報確認画面でお届け先が2つ表示され、1つは空白になっている。

進めば進むほど疑問や問題ばかりでてきますが、
2.12をお使いのみなさまは、同じようになりますか?
変なのは私だけでしょうか?
そもそも、お届け先が2つ表示され1つが空白になっていることが、正しいのか正しくないのすらわかりません。

どのなのでしょうか?

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

tokuhiro
投稿日時: 2012/7/10 10:25
対応状況: −−−
常連
登録日: 2012/4/18
居住地: 大阪
投稿: 53
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
おはようございます。
tokuhiroと申します。

上記内容ですが、最新の2.12.1、2.12.0両方共に実証できました。

2.11では、お届け日やお届け時間を選ぶ欄は1つ表示でしたので、バグだと思います。

チケットに登録いたしました。
http://svn.ec-cube.net/open_trac/ticket/1893
tokuhiro
投稿日時: 2012/7/10 12:23
対応状況: −−−
常連
登録日: 2012/4/18
居住地: 大阪
投稿: 53
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
以下、修正して開発サイトに取り込みいたしました。

http://svn.ec-cube.net/open_trac/changeset/21965
ゲスト
投稿日時: 2012/7/10 13:16
対応状況: −−−
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
すごいですね。
こんなに簡単に直るんですから。

ちなみに、こんなことになると思っていなかったので、言わなかったのですが、
管理画面の受注管理>受注登録の画面にも、お届け先1とお届け先2が表示されるようです。

いいのかわるいのかわかりませんが、報告まで。
sea_gull
投稿日時: 2012/8/6 21:15
対応状況: −−−
常連
登録日: 2011/3/28
居住地:
投稿: 40
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
下記の修正の件についてですが、
http://svn.ec-cube.net/open_trac/ticket/1893

フロント購入画面の表示は改善されましたが、「受注管理>詳細」と「受注メール」も影響しているようです。

▼下記と同じ状況となっています。
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=11026&forum=11&post_id=52024#forumpost52024

データベースの「dtb_shipping」に2つ登録されてしまっているのが原因でしょうか?

---------------------------------
[EC-CUBE]2.12.0
[OS]Linux
[PHP] 5.3.13
[データベース]MySQL 5.0.67
---------------------------------
134
投稿日時: 2012/8/6 22:51
対応状況: −−−
一人前
登録日: 2009/10/20
居住地: 京都市
投稿: 91
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
弊社でも現象を確認しました。

data/class/pages/shopping/LC_Page_Shopping.php の186行目
$objPurchase->setShipmentItemTempForSole($objCartSess);


$arrParams = $objFormParam->getHashArray();
$shipping_id = (isset($arrParams['deliv_check']) && $arrParams['deliv_check'] == '1') ? 1 : 0;
$objPurchase->setShipmentItemTempForSole($objCartSess, $shipping_id);

にすれば解決するのではないかと思います。

テストを行った後、問題がなさそうなら
明日にでも修正版をコミットしたいと思います。
ゲスト
投稿日時: 2012/8/7 0:21
対応状況: −−−
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
報告

ローカル環境ですが、LC_Page_Shopping.php の186行目に変更を加えたところ、問題は解消されました。
sea_gull
投稿日時: 2012/8/7 1:01
対応状況: 解決済
常連
登録日: 2011/3/28
居住地:
投稿: 40
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
134様

早速の修正ありがとうございます。

こちらも記載の修正で正常に動作し、問題が解消されました。
seasoft
投稿日時: 2012/8/7 9:16
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7365
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
134様

lfRegistData でも同様の判定を行なっている箇所がありますので、この辺りと統合できるとスマートかなとは思いますが、値の受け渡しとか結構面倒になりますかね。

少なくとも、lfRegistData の判定条件と、isset の使用の辺りは揃えておくのが良いと感じました。

所感まで。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

134
投稿日時: 2012/8/7 12:05
対応状況: −−−
一人前
登録日: 2009/10/20
居住地: 京都市
投稿: 91
Re: 会員登録をしないで購入し、お届け先を注文者と別にする場合の・・・・
>seasoft様

コメントありがとうございます。
確かに似た処理が分散していますね。。。
とりあえず、私のほうでは最低限の修正にとどめさせて頂きました。

isset の使用は lfCheckError を参考に入れていましたが
セットされていないケースは期待されていないようなので
isset は削除してコミットいたしました。
http://svn.ec-cube.net/open_trac/changeset/21993

>goodspeed様 tokuhiro様 sea_gull様

ご報告、ご対応ありがとうございました。
最終的に、昨晩の本スレッドへの書き込みと若干異なるコードでコミットしましたが
そのままで問題無いと思います。
(1) 2 »
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


EC-CUBE公式 Amazon Payプラグイン

統計情報

総メンバー数は88,286名です
総投稿数は109,688件です

投稿数ランキング

1
seasoft
7365
2
468
3217
3
AMUAMU
2712
4
nanasess
2303
5
umebius
2085
6
yuh
1818
7
h_tanaka
1610
8
red
1567
9
mcontact
1240
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
796
14 ramrun 789
15 karin 689
16 sumida 641
17
homan
633
18 DELIGHT 572
19
patapata
502
20
flealog
485


ネットショップの壺

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

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