Changeset 23015 for branches/version-2_13-dev/data/class/pages/products/LC_Page_Products_CategoryList.php

Timestamp:

2013/07/31 00:17:49 (11 years ago)

Author:

shutta

Message:

#2316 (LC_Page_Products_CategoryList::lfCheckCategoryIdの引数不足によるWarning)
lfCheckCategoryIdメソッドを引数が不足した状態で呼び出していた部分を修正。
また、lfCheckCategoryIdが冗長に実行されていたのも修正。

File:

• branches/version-2_13-dev/data/class/pages/products/LC_Page_Products_CategoryList.php (modified) (2 diffs)

branches/version-2_13-dev/data/class/pages/products/LC_Page_Products_CategoryList.php

@@ -61 +61 @@
     function action()
     {
+        $objFormParam = $this->lfInitParam($_REQUEST);
+
         // カテゴリIDの正当性チェック
-        $this->lfCheckCategoryId();
+        $category_id = $this->lfCheckCategoryId($objFormParam->getValue('category_id'));
+        if ($category_id == 0) {
+            SC_Utils_Ex::sfDispSiteError(CATEGORY_NOT_FOUND);
+        }
 
         // カテゴリ情報を取得する。
-        $objFormParam = $this->lfInitParam($_REQUEST);
-        $arrCategoryData = $this->lfGetCategories($objFormParam->getValue('category_id'), true);
+        $arrCategoryData = $this->lfGetCategories($category_id, true);
         $this->arrCategory = $arrCategoryData['arrCategory'];
         $this->arrChildren = $arrCategoryData['arrChildren'];
@@ -92 +96 @@
     function lfGetCategories($category_id, $count_check = false)
     {
-        // カテゴリの正しいIDを取得する。
-        $category_id = $this->lfCheckCategoryId($category_id);
-        if ($category_id == 0) {
-            SC_Utils_Ex::sfDispSiteError(CATEGORY_NOT_FOUND);
-        }
-
         $arrCategory = null;    // 選択されたカテゴリ
         $arrChildren = array(); // 子カテゴリ