[Tep-j-general] Re: ナビゲーションバーの変更

Back to archive index

Takuya Ando ando****@ironh*****
2004年 8月 13日 (金) 18:33:31 JST


こんにちは。安藤@IRONHEARTSです。

On Fri, 13 Aug 2004 18:05:30 +0900
Katsunori IMAI <imai****@glatt*****> wrote:
> 今井です。こんにちは。

フォローありがとうございますm(__)m

> > $model_query = tep_db_query("select products_name from " . TABLE_PRODUCTS_DESCRIPTION . " where products_id = '" . $_REQUEST['products_id'] . "'");
> 
> products_descriptionは、一プロダクトに対して複数行(複数言語分)ある
> ので、whereにlanguage_idの値も含め、最後の"'"は、
> 
> "' and language_id='" . $languages_id . "'"
> 
> としないとだめだと思います。application_top.phpの、もう少し上のほう
> に、categories_descriptionでの同様の例がありますね。

あー。なるほど。そうですね。僕は基本的に複数言語については
実質まったく使ってないので、気が回りませんでした。

> もう一点、私が見ている2.2ms1j-R5では、$_REQUEST は、$HTTP_GET_VARS 
> になってます。これは、使用する改造もとのコードがどちらを使っているか
> に合わせておけば良いと思います。

あー。そうか。それに、$_REUESTはphp 4.1.0〜でしたよね。それ以前のサーバ
では動かないです。$HTTP_GET_VARSのほうがいいですね。
$HTTP_GET_VARSが$_REQUESTになっていたのは、私が別途修正したものかも
しれません(汗

念のため、まとめたコードを載せます。

$model_query = tep_db_query("select products_name from " . TABLE_PRODUCTS_DESCRIPTION . " where products_id = '" . $HTTP_GET_VARS['products_id'] . "' and language_id='" . $languages_id . "'");
$model = tep_db_fetch_array($model_query);
$breadcrumb->add($model['products_name'], tep_href_link(FILENAME_PRODUCT_INFO, 'cPath=' . $cPath . '&products_id=' . $HTTP_GET_VARS['products_id']));

いかがでしょうか。(これもテストしてないので、自己責任でどうぞ)

-- 
有限会社アイアンハーツ / IRONHEARTS Inc.
Takuya Ando <ando****@ironh*****>




Tep-j-general メーリングリストの案内
Back to archive index