どうか教えて下さい！
| > フロント機能 > 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました |
フロント機能
 |
| スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
| 投稿者 | スレッド |
|---|---|
| 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 居住地: 投稿: 7369 |
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました 2.11 系で配送周りが絡むカスタマイズを行う時には、複数指定を外す事が多々ありました。
なので、機能的には賛成です。 ただ、フロント機能の標準実装としては、テンプレートを切るだけで良いのかな? という疑問はあります。ロジックで不正入力チェックとかした方が良いような。 また、細かい点ですが、テンプレートの IF 分岐が適切にインデントされていないのは気になりました。
|
| shutta |
投稿日時: 2012/2/3 14:48
対応状況: −−−
|
仙人 登録日: 2010/2/4 居住地: 関西 投稿: 835 |
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました 引用：
早速のコメントありがとうございます！ やはり無効にしたい需要は多そうですね。 引用：
はい、完全に対応するには、ロジック側の変更も必要だと思います。 まずは簡単に実現できる部分をコミットして、反応を見てみたかったので、本格的にロジック側の変更も検討してみます。 引用：
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 居住地: 投稿: 7369 |
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました 引用：
とりあえず、気になった点を修正してみました。 http://svn.ec-cube.net/open_trac/changeset/21459
|
| shutta |
投稿日時: 2012/2/8 20:25
対応状況: −−−
|
仙人 登録日: 2010/2/4 居住地: 関西 投稿: 835 |
Re: 複数配送先指定機能の使用をパラメーター設定から変更できるようしてみました 引用：
修正有難うございます。 テンプレートに関してのインデントの取り方の解釈を誤っていたようですね。 今後は、前例に倣って、スタイルに沿うように気を付けます。
|
| スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |
