バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

 > フロント機能 > 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました

フロント機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
shutta
投稿日時: 2012/2/3 14:28
対応状況: −−−
仙人
登録日: 2010/2/4
居住地: 関西
投稿: 835
複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました
みなさん、こんにちは

2.11から追加された機能に複数配送先指定機能がありますが、
無効にしたいという要望がありそうに思ったので、
下記のチケットにて使用する・しないをパラメータ設定から選択できるようにコミットしてみました。

チケット #1627
複数配送先指定機能の使用可否をパラメータ設定化
http://svn.ec-cube.net/open_trac/ticket/1627

チェンジセット 21435
http://svn.ec-cube.net/open_trac/changeset/21435


複数配送先指定は運用上のいろいろな絡みから無効にしたいという事もあるでしょうし、
支払い方法が代引の場合に、複数の配送先が指定されている場合、どう請求すれば良いのか困るんじゃないかと思います。

2.11で運用されているみなさんは、複数配送先指定に関してどうされていますか?
常々気になっていたので、この際、みなさんのコメントを聞いてみたいなぁ、と思う次第です。


----------------

リゾート会員権(エクシブ等)の売買・仲介

seasoft
投稿日時: 2012/2/3 14:36
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7333
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました
2.11 系で配送周りが絡むカスタマイズを行う時には、複数指定を外す事が多々ありました。

なので、機能的には賛成です。


ただ、フロント機能の標準実装としては、テンプレートを切るだけで良いのかな? という疑問はあります。ロジックで不正入力チェックとかした方が良いような。

また、細かい点ですが、テンプレートの IF 分岐が適切にインデントされていないのは気になりました。


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

shutta
投稿日時: 2012/2/3 14:48
対応状況: −−−
仙人
登録日: 2010/2/4
居住地: 関西
投稿: 835
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました
引用:

seasoftさんは書きました:
2.11 系で配送周りが絡むカスタマイズを行う時には、複数指定を外す事が多々ありました。

なので、機能的には賛成です。


早速のコメントありがとうございます!

やはり無効にしたい需要は多そうですね。

引用:

ただ、フロント機能の標準実装としては、テンプレートを切るだけで良いのかな? という疑問はあります。ロジックで不正入力チェックとかした方が良いような。


はい、完全に対応するには、ロジック側の変更も必要だと思います。
まずは簡単に実現できる部分をコミットして、反応を見てみたかったので、本格的にロジック側の変更も検討してみます。

引用:

また、細かい点ですが、テンプレートの IF 分岐が適切にインデントされていないのは気になりました。


sphone/shopping/nonmember_input.tpl

に関しては、全体的にインデントが崩れていたので、考慮せずに横着してしまいました。すみません。

その他のテンプレートは、インデントは気を付けてみたつもりですが、
気になる部分はありますでしょうか。


----------------

リゾート会員権(エクシブ等)の売買・仲介

shutta
投稿日時: 2012/2/3 15:24
対応状況: −−−
仙人
登録日: 2010/2/4
居住地: 関西
投稿: 835
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました
引用:


引用:

ただ、フロント機能の標準実装としては、テンプレートを切るだけで良いのかな? という疑問はあります。ロジックで不正入力チェックとかした方が良いような。


はい、完全に対応するには、ロジック側の変更も必要だと思います。
まずは簡単に実現できる部分をコミットして、反応を見てみたかったので、本格的にロジック側の変更も検討してみます。



ロジック側も実装してみました。

チェンジセット 21437
http://svn.ec-cube.net/open_trac/changeset/21437


----------------

リゾート会員権(エクシブ等)の売買・仲介

seasoft
投稿日時: 2012/2/8 20:15
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7333
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました
引用:

shuttaさんは書きました:
引用:

また、細かい点ですが、テンプレートの IF 分岐が適切にインデントされていないのは気になりました。


sphone/shopping/nonmember_input.tpl

に関しては、全体的にインデントが崩れていたので、考慮せずに横着してしまいました。すみません。

その他のテンプレートは、インデントは気を付けてみたつもりですが、
気になる部分はありますでしょうか。

とりあえず、気になった点を修正してみました。
http://svn.ec-cube.net/open_trac/changeset/21459


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

shutta
投稿日時: 2012/2/8 20:25
対応状況: −−−
仙人
登録日: 2010/2/4
居住地: 関西
投稿: 835
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました
引用:

とりあえず、気になった点を修正してみました。
http://svn.ec-cube.net/open_trac/changeset/21459


修正有難うございます。

テンプレートに関してのインデントの取り方の解釈を誤っていたようですね。
今後は、前例に倣って、スタイルに沿うように気を付けます。


----------------

リゾート会員権(エクシブ等)の売買・仲介

スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン



統計情報

総メンバー数は75,050名です
総投稿数は104,352件です

投稿数ランキング

1
seasoft
7333
2
468
3217
3
AMUAMU
2712
4
nanasess
2202
5
umebius
2085
6
yuh
1664
7
red
1535
8
h_tanaka
1189
9
tsuji
942
10
fukap
907
11
shutta
835
12
tao_s
794
13 ramrun 789
14 karin 689
15 sumida 641
16
homan
633
17 DELIGHT 572
18
patapata
502
19
flealog
485
20 tonton 437
Copyright© EC-CUBE CO.,LTD. All Rights Reserved.