Changeset 6048

Timestamp:

Mar 25, 2008, 12:01:17 AM (14 years ago)

Author:

ebihara

Message:

#2044:存在しない画像を指定してもランクの追加ができていたのを修正

File:

• OpenPNE/branches/stable-2.10.x/webapp/modules/admin/do/insert_c_rank.php (modified) (2 diffs)

OpenPNE/branches/stable-2.10.x/webapp/modules/admin/do/insert_c_rank.php

@@ -15 +15 @@
     function execute($requests)
     {
+        // ランク機能が無効
         if (!OPENPNE_USE_POINT_RANK) {
             admin_client_redirect('top', '指定されたページにはアクセスできません');
@@ -23 +24 @@
         }
 
+        $upfile_obj = $_FILES['image_upfile'];
+
+        // ランク画像が指定されていない
+        if (empty($upfile_obj) || $upfile_obj['error'] === UPLOAD_ERR_NO_FILE) {
+            admin_client_redirect('list_c_rank', 'ランク画像を指定してください');
+        }
+
+        // ランク画像の形式チェック
+        if (!empty($upfile_obj) && $upfile_obj['error'] !== UPLOAD_ERR_NO_FILE) {
+            if (!($image = t_check_image($upfile_obj))) {
+                admin_client_redirect('list_c_rank', '画像は' . IMAGE_MAX_FILESIZE . 'KB以内のGIF・JPEG・PNGにしてください');
+            }
+        }
+
+        // ランクを追加
         $c_rank_id = db_admin_insert_c_rank($requests['name'], '', $requests['point']);
-        if ($_FILES['image_upfile']['name']) {
-            $ext = t_check_image_format($_FILES['image_upfile']);
-            $image_filename = "r_{$c_rank_id}_".time().".{$ext}";
-            admin_insert_c_image($_FILES['image_upfile'], $image_filename);
+        if (!$c_rank_id) {
+            admin_client_redirect('list_c_rank', 'ランクが追加できませんでした');
+        }
 
-            db_admin_update_c_rank($c_rank_id, $requests['name'], $image_filename, $requests['point']);
-        }
+        // ランク画像挿入
+        $ext = t_check_image_format($upfile_obj);
+        $image_filename = "r_{$c_rank_id}_".time().".{$ext}";
+        admin_insert_c_image($upfile_obj, $image_filename);
+
+        // ランクとランク画像の関連付け
+        db_admin_update_c_rank($c_rank_id, $requests['name'], $image_filename, $requests['point']);
 
         admin_client_redirect('list_c_rank', 'ランクを追加しました');