バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

バグ報告 > フロント機能 > 受注メールのテンプレートにおける記述について

フロント機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
aither
投稿日時: 2009/5/28 16:13
対応状況: −−−
新米
登録日: 2009/5/28
居住地:
投稿: 3
受注メールのテンプレートにおける記述について
EC-CUBE 2.4.0の正式リリース環境下で、変更箇所を確認していたのですが、気づいた部分があり、フォーラムで探しきれなかったので、投稿します。

ソースコード
/data/Smarty/templates/default/mail_templates/order_mail.tpl

のLine72に

<!--{if $arrOrder.customer_id && $smarty.const.USE_POINT === true}-->

の記述があります。
正しい動作を期待するためには、今のうち

<!--{if $arrOrder.customer_id && $smarty.const.USE_POINT == true}-->

にしたほうが。
seasoft
投稿日時: 2009/5/28 16:16
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7333
Re: 受注メールのテンプレートにおける記述について
それは、$smarty.const.USE_POINT にどのような値が入っていることを想定してですか?


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

aither
投稿日時: 2009/5/28 16:47
対応状況: −−−
新米
登録日: 2009/5/28
居住地:
投稿: 3
Re: 受注メールのテンプレートにおける記述について
いえいえそういうことではなく、"="が3つってありですか?
seasoft
投稿日時: 2009/5/28 16:50
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7333
Re: 受注メールのテンプレートにおける記述について
ありです。

マニュアルに書かれてますから。
http://www.smarty.net/manual/ja/language.function.if.php


ちなみに
<!--{if $arrOrder.customer_id && $smarty.const.USE_POINT == true}-->
にするなら
<!--{if $arrOrder.customer_id && $smarty.const.USE_POINT}-->
にした方が良いと思います。
まぁ、宗教・美術的な領域ですが。参考まで。


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

aither
投稿日時: 2009/5/28 16:57
対応状況: −−−
新米
登録日: 2009/5/28
居住地:
投稿: 3
Re: 受注メールのテンプレートにおける記述について
すいません。お騒がせしました。
自分でもどうせtrueなということで気づいて
<!--{if $arrOrder.customer_id && $smarty.const.USE_POINT}-->
にしたんですが、今後のソース比較のために元にもどしました。
kishik
投稿日時: 2009/5/28 17:15
対応状況: −−−
仙人
登録日: 2009/4/3
居住地: 東京
投稿: 382
Re: 受注メールのテンプレートにおける記述について
== は緩い比較
=== 厳密な比較
違いがあります。
一応、念のため。
PHPは文字と数字が比較できたりして
(しかも結果がtrueなことも!)

ちなみに後者の方が高速です。


-----
カスタマイズ承ります
http://ec-cube-mall.jp/
http://ec-cube.ec-orange2.jp/
http://wiki.ec-orange2.jp/
seasoft
投稿日時: 2009/6/4 19:00
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7333
Re: 受注メールのテンプレートにおける記述について
現状を擁護してしまいましたが、やはり問題がありますね。

正式版では、
・ === true
・ === false
の2者が混在しています。

本来は、
・ !== false
・ === false
とすべきだと思われます。

正式版をご利用の方は、パラメータ「USE_POINT」には、true/false 以外の値を誤ってセットすると、不可解な動作をすると思いますのでご注意ください。

EC-CUBEコミュニティ(eccube-comu) では、そのようなバグは無いと思われますのでご安心ください。


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

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


 



ログイン



統計情報

総メンバー数は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.