バグ報告 > フロント機能 > 受注メールのテンプレートにおける記述について |
フロント機能
スレッド表示 | 古いものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
seasoft |
投稿日時: 2009/6/4 19:00
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7367 |
Re: 受注メールのテンプレートにおける記述について 現状を擁護してしまいましたが、やはり問題がありますね。
正式版では、 ・ === true ・ === false の2者が混在しています。 本来は、 ・ !== false ・ === false とすべきだと思われます。 正式版をご利用の方は、パラメータ「USE_POINT」には、true/false 以外の値を誤ってセットすると、不可解な動作をすると思いますのでご注意ください。 EC-CUBEコミュニティ(eccube-comu) では、そのようなバグは無いと思われますのでご安心ください。
|
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/ |
aither |
投稿日時: 2009/5/28 16:57
対応状況: −−−
|
新米 登録日: 2009/5/28 居住地: 投稿: 3 |
Re: 受注メールのテンプレートにおける記述について すいません。お騒がせしました。
自分でもどうせtrueなということで気づいて <!--{if $arrOrder.customer_id && $smarty.const.USE_POINT}--> にしたんですが、今後のソース比較のために元にもどしました。 |
seasoft |
投稿日時: 2009/5/28 16:50
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7367 |
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}--> にした方が良いと思います。 まぁ、宗教・美術的な領域ですが。参考まで。
|
aither |
投稿日時: 2009/5/28 16:47
対応状況: −−−
|
新米 登録日: 2009/5/28 居住地: 投稿: 3 |
Re: 受注メールのテンプレートにおける記述について いえいえそういうことではなく、"="が3つってありですか?
|
seasoft |
投稿日時: 2009/5/28 16:16
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7367 |
Re: 受注メールのテンプレートにおける記述について それは、$smarty.const.USE_POINT にどのような値が入っていることを想定してですか?
|
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}--> にしたほうが。 |
スレッド表示 | 古いものから | 前のトピック | 次のトピック | トップ |