EC-CUBE 開発コミュニティ

投稿者	スレッド
AMUAMU	投稿日時: 2009/7/27 9:30 対応状況: −−−
神登録日: 2009/5/2 居住地: 東京都投稿: 2712	Re: 商品一覧気になったので自分でさらに調べてみました `!$objDb->sfIsRecord('dtb_category', 'category_id', (array)$category_id, 'del_flg = 0')` という、存在問い合わせを足したって感じのようですが、その前の `if (!defined('MOBILE_SITE') &&　!isset($_REQUEST['category_id'])) SC_Utils_Ex::sfDispSiteError(CATEGORY_NOT_FOUND);` が余計なのかも。 sfGetCategoryIdの関数仕様としては、有効でなければ空を返す仕様で、LC_Page_Products_List的には空であれば全商品として扱うルーチンが入っていたのですが、それを無視しちゃうチェック処理を足してしまった感じですね。このlfCheckCategoryIdで、LC_Page_Products_Listで個別にチェックすべき内容なのか微妙。 sfGetCategoryId関数に追加チェック処理をマージするのが美しい感じもしますが、SC_Helper_DBの挙動を変えるのは影響範囲が大きいので確認すること多いので怖いな・・・ ---------------- EC-CUBE公式エヴァンジェリスト EC-CUBEインテグレートパートナー (株)スピリット・オブ移転・拡張・高速化・問題解決各種カスタマイズ・支援依頼承ります。 [url=h