Changeset 22064


Ignore:
Timestamp:
2012/11/02 15:48:19 (9 years ago)
Author:
pineray
Message:

#1903 (無駄な処理を改善する) 冗長な処理を短縮。

Location:
branches/version-2_12-dev/data/class/pages/admin/products
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Product.php

    r21996 r22064  
    326326        $objFormParam->addParam('商品ステータス', 'product_status', INT_LEN, 'n', array('NUM_CHECK', 'MAX_LENGTH_CHECK')); 
    327327 
    328         if ($this->lfGetProductClassFlag($arrPost['has_product_class']) == false) { 
     328        if (!$arrPost['has_product_class']) { 
    329329            // 新規登録, 規格なし商品の編集の場合 
    330330            $objFormParam->addParam('商品種別', 'product_type_id', INT_LEN, 'n', array('EXIST_CHECK', 'NUM_CHECK', 'MAX_LENGTH_CHECK')); 
     
    949949 
    950950    /** 
    951      * 入力フォームパラメーターの規格ありなしフラグを判定 
    952      * 
    953      * @param string $has_product_class 入力フォームパラメーターの規格ありなしフラグ 
    954      * @return boolean true: 規格あり, false: 規格なし 
    955      */ 
    956     function lfGetProductClassFlag($has_product_class) { 
    957         if ($has_product_class == '1') { 
    958             return true; 
    959         } else { 
    960             return false; 
    961         } 
    962     } 
    963  
    964     /** 
    965951     * ページonload用JavaScriptを取得する 
    966952     * - 入力画面 
     
    10351021            // 複製商品の場合には規格も複製する 
    10361022            if ($arrList['copy_product_id'] != '' && SC_Utils_Ex::sfIsInt($arrList['copy_product_id'])) { 
    1037                 if ($this->lfGetProductClassFlag($arrList['has_product_class']) == false) { 
     1023                if (!$arrList['has_product_class']) { 
    10381024                    //規格なしの場合、複製は価格等の入力が発生しているため、その内容で追加登録を行う 
    10391025                    $this->lfCopyProductClass($arrList, $objQuery); 
  • branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_ReviewEdit.php

    r21867 r22064  
    8686            // 登録 
    8787            case 'complete': 
    88                 $this->arrErr = $this->lfCheckError($objFormParam); 
     88                $this->arrErr = $objFormParam->checkError(); 
    8989                // エラー無し 
    90                 if (!$this->arrErr) { 
     90                if (!SC_Utils_Ex::isBlank($this->arrErr)) { 
    9191                    // レビュー情報の更新 
    9292                    $this->lfRegistReviewData($this->arrForm['review_id'], $objFormParam); 
     
    137137 
    138138    /** 
    139      * フォーム入力パラメーターエラーチェック 
    140      * 
    141      * @param array $objFormParam フォームパラメータークラス 
    142      * @return array エラー配列 
    143      */ 
    144     function lfCheckError(&$objFormParam) { 
    145         $arrErr = $objFormParam->checkError(); 
    146         if (!SC_Utils_Ex::isBlank($arrErr)) { 
    147             return $arrErr; 
    148         } 
    149     } 
    150  
    151     /** 
    152139     * レビュー情報のDB取得 
    153140     * 
Note: See TracChangeset for help on using the changeset viewer.