Changeset 23485

Timestamp:

2014/06/02 01:35:08 (9 years ago)

Author:

shutta

Message:

#2545 ファイルアップロード時のエラーメッセージが不適切
ファイルのアップロードのエラーチェック処理を分離。

File:

• branches/version-2_13-dev/data/class/SC_UploadFile.php (modified) (5 diffs)

branches/version-2_13-dev/data/class/SC_UploadFile.php

@@ -98 +98 @@
         $objErr = new SC_CheckError_Ex();
         $cnt = 0;
-        $arrKeyname = array_flip($this->keyname);
-
-        if ($_FILES[$keyname]['error'] === UPLOAD_ERR_OK) {
+        $check = $this->checkUploadError($keyname, $objErr);
+        if ($check) {
             foreach ($this->keyname as $val) {
                 // 一致したキーのファイルに情報を保存する。
@@ -135 +134 @@
                 $cnt++;
             }
-        } elseif ($_FILES[$keyname]['error'] === UPLOAD_ERR_NO_FILE) {
-            $objErr->arrErr[$keyname] = '※ '
-                . $this->disp_name[$arrKeyname[$keyname]]
-                . 'が選択されていません。'
-                . '<br />';
-        } elseif ($_FILES[$keyname]['error'] === UPLOAD_ERR_INI_SIZE) {
-            $objErr->arrErr[$keyname] = '※ '
-                . $this->disp_name[$arrKeyname[$keyname]]
-                . 'のアップロードに失敗しました。'
-                . '(.htaccessファイルのphp_value upload_max_filesizeを調整してください)'
-                . '<br />';
-        } else {
-            $objErr->arrErr[$keyname] = '※ '
-                . $this->disp_name[$arrKeyname[$keyname]]
-                . 'のアップロードに失敗しました。'
-                . 'エラーコードは[' . $_FILES[$keyname]['error'] . ']です。'
-                . '<br />';
         }
 
@@ -162 +144 @@
         $objErr = new SC_CheckError_Ex();
         $cnt = 0;
-        $arrKeyname = array_flip($this->keyname);
-
-        if ($_FILES[$keyname]['error'] === UPLOAD_ERR_OK) {
+        $check = $this->checkUploadError($keyname, $objErr);
+        if ($check) {
             foreach ($this->keyname as $val) {
                 // 一致したキーのファイルに情報を保存する。
@@ -184 +165 @@
                 $cnt++;
             }
-        } elseif ($_FILES[$keyname]['error'] === UPLOAD_ERR_NO_FILE) {
-            $objErr->arrErr[$keyname] = '※ '
-                . $this->disp_name[$arrKeyname[$keyname]]
-                . 'が選択されていません。'
-                . '<br />';
-        } elseif ($_FILES[$keyname]['error'] === UPLOAD_ERR_INI_SIZE) {
-            $objErr->arrErr[$keyname] = '※ '
-                . $this->disp_name[$arrKeyname[$keyname]]
-                . 'のアップロードに失敗しました。'
-                . '(.htaccessファイルのphp_value upload_max_filesizeを調整してください)'
-                . '<br />';
-        } else {
-            $objErr->arrErr[$keyname] = '※ '
-                . $this->disp_name[$arrKeyname[$keyname]]
-                . 'のアップロードに失敗しました。'
-                . 'エラーコードは[' . $_FILES[$keyname]['error'] . ']です。'
-                . '<br />';
         }
 
@@ -566 +530 @@
         return $dst_file;
     }
+
+    /**
+     * ファイルのアップロードのエラーを確認
+     *
+     * @param string $keyname ファイルinputタグのname
+     * @param object $objErr SC_CheckErrorインスタンス
+     * @return boolean
+     */
+    public function checkUploadError($keyname, SC_CheckError &$objErr)
+    {
+        $index = array_search($keyname, $this->keyname);
+
+        switch ($_FILES[$keyname]['error']) {
+            case UPLOAD_ERR_OK:
+                return true;
+                break;
+            case UPLOAD_ERR_NO_FILE:
+                $objErr->arrErr[$keyname] = '※ '
+                    . $this->disp_name[$index]
+                    . 'が選択されていません。'
+                    . '<br />';
+                break;
+            case UPLOAD_ERR_INI_SIZE:
+                $objErr->arrErr[$keyname] = '※ '
+                    . $this->disp_name[$index]
+                    . 'のアップロードに失敗しました。'
+                    . '(.htaccessファイルのphp_value upload_max_filesizeを調整してください)'
+                    . '<br />';
+                break;
+            default:
+                $objErr->arrErr[$keyname] = '※ '
+                    . $this->disp_name[$index]
+                    . 'のアップロードに失敗しました。'
+                    . 'エラーコードは[' . $_FILES[$keyname]['error'] . ']です。'
+                    . '<br />';
+                break;
+        }
+
+        return false;
+    }
 }