[Svn-src-all:2798] [version-2_12-dev 21697] #1466 LC_Page_Admin_Products_Maker バリデーションに共通処理を利用していない の対応

pingpong admin @ mail.ec-cube.net
2012年 3月 30日 (金) 15:47:20 JST


Subversion committed to /home/svn/open 21697
http://svn.ec-cube.net/open_trac/changeset/21697
┌────────────────────────────┐
│更新者 :  pingpong                                     │
│更新日時:  2012-03-30 15:47:19 +0900 (金, 30  3月 2012)│
└────────────────────────────┘

Log:
--------------------------------------------------------
#1466 LC_Page_Admin_Products_Maker バリデーションに共通処理を利用していない の対応


Changed:                      [U:修正,A:追加,D:削除]
--------------------------------------------------------
U   branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Maker.php

変更: branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Maker.php
===================================================================
--- branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Maker.php	2012-03-29 18:54:06 UTC (rev 21696)
+++ branches/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Maker.php	2012-03-30 06:47:19 UTC (rev 21697)
@@ -95,7 +95,7 @@
             // 入力文字の変換
 
                 // エラーチェック
-                $this->arrErr = $this->lfErrorCheck($this->arrForm);
+                $this->arrErr = $this->lfErrorCheck($this->arrForm, $objFormParam);
                 if (count($this->arrErr) <= 0) {
                     if ($this->arrForm['maker_id'] == '') {
                         // メーカー情報新規登録
@@ -294,10 +294,13 @@
      * @param  array $arrForm メーカー情報
      * @return array $objErr->arrErr エラー内容
      */
-    function lfErrorCheck(&$arrForm) {
-        $objErr = new SC_CheckError_Ex($arrForm);
-        $objErr->doFunc(array('メーカー名', 'name', SMTEXT_LEN), array('EXIST_CHECK','SPTAB_CHECK','MAX_LENGTH_CHECK'));
+    function lfErrorCheck(&$arrForm, &$objFormParam) {
 
+        $arrErr = $objFormParam->checkError();
+        if (!empty($arrErr)) {
+            return $arrErr;
+        }
+
         // maker_id の正当性チェック
         if (!empty($arrForm['maker_id'])) {
             $objDb = new SC_Helper_DB_Ex();
@@ -306,20 +309,20 @@
                 || !$objDb->sfIsRecord('dtb_maker', 'maker_id', array($arrForm['maker_id']))
             ) {
                 // maker_idが指定されていて、且つその値が不正と思われる場合はエラー
-                $objErr->arrErr['maker_id'] = '※ メーカーIDが不正です<br />';
+                $arrErr['maker_id'] = '※ メーカーIDが不正です<br />';
             }
         }
-        if (!isset($objErr->arrErr['name'])) {
+        if (!isset($arrErr['name'])) {
             $objQuery =& SC_Query_Ex::getSingletonInstance();
             $arrMaker = array();
             $arrMaker = $objQuery->select('maker_id, name', 'dtb_maker', 'del_flg = 0 AND name = ?', array($arrForm['name']));
 
             // 編集中のレコード以外に同じ名称が存在する場合
             if ($arrMaker[0]['maker_id'] != $arrForm['maker_id'] && $arrMaker[0]['name'] == $arrForm['name']) {
-                $objErr->arrErr['name'] = '※ 既に同じ内容の登録が存在します。<br />';
+                $arrErr['name'] = '※ 既に同じ内容の登録が存在します。<br />';
             }
         }
 
-        return $objErr->arrErr;
+        return $arrErr;
     }
 }




Svn-src-all メーリングリストの案内