thorsten / phpMyFAQ

phpmyfaq/admin/ajax.trans.php

Switch Indentation +162 added lines, -162 removed lines

@@ -40,167 +40,167 @@ 
 
 switch ($ajax_action) {
 
-    case 'save_page_buffer':
-        /*
+        case 'save_page_buffer':
+            /*
          * Build language variable definitions
          * @todo Change input handling using PMF_Filter
          */
-        foreach ((array) @$_POST['PMF_LANG'] as $key => $val) {
-            if (is_string($val)) {
-                $val = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $val);
-                $val = str_replace("'", "\\'", $val);
-                $_SESSION['trans']['rightVarsOnly']["PMF_LANG[$key]"] = $val;
-            } elseif (is_array($val)) {
-                /*
+            foreach ((array) @$_POST['PMF_LANG'] as $key => $val) {
+                if (is_string($val)) {
+                    $val = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $val);
+                    $val = str_replace("'", "\\'", $val);
+                    $_SESSION['trans']['rightVarsOnly']["PMF_LANG[$key]"] = $val;
+                } elseif (is_array($val)) {
+                    /*
                  * Here we deal with a two dimensional array
                  */
-                foreach ($val as $key2 => $val2) {
-                    $val2 = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $val2);
-                    $val2 = str_replace("'", "\\'", $val2);
-                    $_SESSION['trans']['rightVarsOnly']["PMF_LANG[$key][$key2]"] = $val2;
+                    foreach ($val as $key2 => $val2) {
+                        $val2 = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $val2);
+                        $val2 = str_replace("'", "\\'", $val2);
+                        $_SESSION['trans']['rightVarsOnly']["PMF_LANG[$key][$key2]"] = $val2;
+                    }
                 }
             }
-        }
-
-        foreach ((array) @$_POST['LANG_CONF'] as $key => $val) {
-            // if string like array(blah-blah-blah), extract the contents inside the brackets
-            if (preg_match('/^\s*array\s*\(\s*(\d+.+)\s*\).*$/', $val, $matches1)) {
-                // split the resulting string of delimiters such as "number =>"
-                $valArr = preg_split(
-                    '/\s*(\d+)\s*\=\>\s*/',
-                    $matches1[1],
-                    null,
-                    PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY
-                );
-                $numVal = count($valArr);
-                if ($numVal > 1) {
-                    $newValArr = [];
-                    for ($i = 0; $i < $numVal; $i += 2) {
-                        if (is_numeric($valArr[$i])) {
-                            // clearing quotes
-                            if (preg_match('/^\s*\\\\*[\"|\'](.+)\\\\*[\"|\'][\s\,]*$/', $valArr[$i + 1], $matches2)) {
-                                $subVal = $matches2[1];
-                                // normalize quotes
-                                $subVal = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $subVal);
-                                $subVal = str_replace("'", "\\'", $subVal);
-                                // assembly of the original substring back
-                                $newValArr[] = $valArr[$i].' => \''.$subVal.'\'';
+
+            foreach ((array) @$_POST['LANG_CONF'] as $key => $val) {
+                // if string like array(blah-blah-blah), extract the contents inside the brackets
+                if (preg_match('/^\s*array\s*\(\s*(\d+.+)\s*\).*$/', $val, $matches1)) {
+                    // split the resulting string of delimiters such as "number =>"
+                    $valArr = preg_split(
+                        '/\s*(\d+)\s*\=\>\s*/',
+                        $matches1[1],
+                        null,
+                        PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY
+                    );
+                    $numVal = count($valArr);
+                    if ($numVal > 1) {
+                        $newValArr = [];
+                        for ($i = 0; $i < $numVal; $i += 2) {
+                            if (is_numeric($valArr[$i])) {
+                                // clearing quotes
+                                if (preg_match('/^\s*\\\\*[\"|\'](.+)\\\\*[\"|\'][\s\,]*$/', $valArr[$i + 1], $matches2)) {
+                                    $subVal = $matches2[1];
+                                    // normalize quotes
+                                    $subVal = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $subVal);
+                                    $subVal = str_replace("'", "\\'", $subVal);
+                                    // assembly of the original substring back
+                                    $newValArr[] = $valArr[$i].' => \''.$subVal.'\'';
+                                }
                             }
                         }
+                        $_SESSION['trans']['rightVarsOnly']["LANG_CONF[$key]"] = 'array('.implode(', ', $newValArr).')';
                     }
-                    $_SESSION['trans']['rightVarsOnly']["LANG_CONF[$key]"] = 'array('.implode(', ', $newValArr).')';
+                } else {  // compatibility for old behavior
+                    $val = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $val);
+                    $val = str_replace("'", "\\'", $val);
+                    $_SESSION['trans']['rightVarsOnly']["LANG_CONF[$key]"] = $val;
                 }
-            } else {  // compatibility for old behavior
-                $val = str_replace(array('\\\\', '\"', '\\\''), array('\\', '"', "'"), $val);
-                $val = str_replace("'", "\\'", $val);
-                $_SESSION['trans']['rightVarsOnly']["LANG_CONF[$key]"] = $val;
             }
-        }
 
-        echo 1;
-    break;
+            echo 1;
+        break;
     
-    case 'save_translated_lang':
-
-        if (!$user->perm->checkRight($user->getUserId(), 'edittranslation')) {
-            echo $PMF_LANG['err_NotAuth'];
-            exit;
-        }
-
-        $lang = strtolower($_SESSION['trans']['rightVarsOnly']['PMF_LANG[metaLanguage]']);
-        $filename = PMF_ROOT_DIR.'/lang/language_'.$lang.'.php';
-
-        if (!is_writable(PMF_ROOT_DIR.'/lang')) {
-            echo 0;
-            exit;
-        }
-
-        if (!copy($filename, PMF_ROOT_DIR.'/lang/language_'.$lang.'.bak.php')) {
-            echo 0;
-            exit;
-        }
-
-        $newFileContents = '';
-        $tmpLines = [];
-
-        // Read in the head of the file we're writing to
-        $fh = fopen($filename, 'r');
-        do {
-            $line = fgets($fh);
-            array_push($tmpLines, rtrim($line));
-        } while ('*/' != substr(trim($line), -2));
-        fclose($fh);
-
-        // Construct lines with variable definitions
-        foreach ($_SESSION['trans']['rightVarsOnly'] as $key => $val) {
-            if (0 === strpos($key, 'PMF_LANG')) {
-                $val = "'$val'";
+        case 'save_translated_lang':
+
+            if (!$user->perm->checkRight($user->getUserId(), 'edittranslation')) {
+                echo $PMF_LANG['err_NotAuth'];
+                exit;
             }
-            array_push($tmpLines, '$'.str_replace(array('[', ']'), array("['", "']"), $key)." = $val;");
-        }
 
-        $newFileContents .= implode("\n", $tmpLines);
+            $lang = strtolower($_SESSION['trans']['rightVarsOnly']['PMF_LANG[metaLanguage]']);
+            $filename = PMF_ROOT_DIR.'/lang/language_'.$lang.'.php';
 
-        unset($_SESSION['trans']);
+            if (!is_writable(PMF_ROOT_DIR.'/lang')) {
+                echo 0;
+                exit;
+            }
 
-        $retval = file_put_contents($filename, $newFileContents);
-        echo intval($retval);
-        break;
+            if (!copy($filename, PMF_ROOT_DIR.'/lang/language_'.$lang.'.bak.php')) {
+                echo 0;
+                exit;
+            }
 
-    case 'remove_lang_file':
+            $newFileContents = '';
+            $tmpLines = [];
+
+            // Read in the head of the file we're writing to
+            $fh = fopen($filename, 'r');
+            do {
+                $line = fgets($fh);
+                array_push($tmpLines, rtrim($line));
+            } while ('*/' != substr(trim($line), -2));
+            fclose($fh);
+
+            // Construct lines with variable definitions
+            foreach ($_SESSION['trans']['rightVarsOnly'] as $key => $val) {
+                if (0 === strpos($key, 'PMF_LANG')) {
+                    $val = "'$val'";
+                }
+                array_push($tmpLines, '$'.str_replace(array('[', ']'), array("['", "']"), $key)." = $val;");
+            }
+
+            $newFileContents .= implode("\n", $tmpLines);
 
-        if (!$user->perm->checkRight($user->getUserId(), 'deltranslation')) {
-            echo $PMF_LANG['err_NotAuth'];
-            exit;
-        }
+            unset($_SESSION['trans']);
 
-        $lang = PMF_Filter::filterInput(INPUT_GET, 'translang', FILTER_SANITIZE_STRING);
+            $retval = file_put_contents($filename, $newFileContents);
+            echo intval($retval);
+            break;
 
-        if (!is_writable(PMF_ROOT_DIR.'/lang')) {
-            echo 0;
-            exit;
-        }
+        case 'remove_lang_file':
 
-        if (!copy(PMF_ROOT_DIR."/lang/language_$lang.php", PMF_ROOT_DIR."/lang/language_$lang.bak.php")) {
-            echo 0;
-            exit;
-        }
+            if (!$user->perm->checkRight($user->getUserId(), 'deltranslation')) {
+                echo $PMF_LANG['err_NotAuth'];
+                exit;
+            }
 
-        if (!unlink(PMF_ROOT_DIR."/lang/language_$lang.php")) {
-            echo 0;
-            exit;
-        }
+            $lang = PMF_Filter::filterInput(INPUT_GET, 'translang', FILTER_SANITIZE_STRING);
 
-        echo 1;
-        break;
+            if (!is_writable(PMF_ROOT_DIR.'/lang')) {
+                echo 0;
+                exit;
+            }
+
+            if (!copy(PMF_ROOT_DIR."/lang/language_$lang.php", PMF_ROOT_DIR."/lang/language_$lang.bak.php")) {
+                echo 0;
+                exit;
+            }
 
-    case 'save_added_trans':
-
-        if (!$user->perm->checkRight($user->getUserId(), 'addtranslation')) {
-            echo $PMF_LANG['err_NotAuth'];
-            exit;
-        }
-
-        if (!is_writable(PMF_ROOT_DIR.'/lang')) {
-            echo 0;
-            exit;
-        }
-
-        $langCode = PMF_Filter::filterInput(INPUT_POST, 'translang', FILTER_SANITIZE_STRING);
-        $langName = @$languageCodes[$langCode];
-        $langCharset = 'UTF-8';
-        $langDir = PMF_Filter::filterInput(INPUT_POST, 'langdir', FILTER_SANITIZE_STRING);
-        $langNPlurals = strval(PMF_Filter::filterVar(@$_POST['langnplurals'], FILTER_VALIDATE_INT, -1));
-        $langDesc = PMF_Filter::filterInput(INPUT_POST, 'langdesc', FILTER_SANITIZE_STRING);
-        $author = (array) @$_POST['author'];
-
-        if (empty($langCode) || empty($langName) || empty($langCharset) ||
-           empty($langDir) || empty($langDesc) || empty($author)) {
-            echo 0;
-            exit;
-        }
-
-        $fileTpl = <<<FILE
+            if (!unlink(PMF_ROOT_DIR."/lang/language_$lang.php")) {
+                echo 0;
+                exit;
+            }
+
+            echo 1;
+            break;
+
+        case 'save_added_trans':
+
+            if (!$user->perm->checkRight($user->getUserId(), 'addtranslation')) {
+                echo $PMF_LANG['err_NotAuth'];
+                exit;
+            }
+
+            if (!is_writable(PMF_ROOT_DIR.'/lang')) {
+                echo 0;
+                exit;
+            }
+
+            $langCode = PMF_Filter::filterInput(INPUT_POST, 'translang', FILTER_SANITIZE_STRING);
+            $langName = @$languageCodes[$langCode];
+            $langCharset = 'UTF-8';
+            $langDir = PMF_Filter::filterInput(INPUT_POST, 'langdir', FILTER_SANITIZE_STRING);
+            $langNPlurals = strval(PMF_Filter::filterVar(@$_POST['langnplurals'], FILTER_VALIDATE_INT, -1));
+            $langDesc = PMF_Filter::filterInput(INPUT_POST, 'langdesc', FILTER_SANITIZE_STRING);
+            $author = (array) @$_POST['author'];
+
+            if (empty($langCode) || empty($langName) || empty($langCharset) ||
+               empty($langDir) || empty($langDesc) || empty($author)) {
+                echo 0;
+                exit;
+            }
+
+            $fileTpl = <<<FILE
 <?php
 /**
  * %s
@@ -226,40 +226,40 @@ 
 \$PMF_LANG['nplurals'] = '%s';
 FILE;
 
-        $authorTpl = '';
-        foreach ($author as $authorData) {
-            $authorTpl .= " * @author    $authorData\n";
-        }
+            $authorTpl = '';
+            foreach ($author as $authorData) {
+                $authorTpl .= " * @author    $authorData\n";
+            }
 
-        $fileTpl = sprintf($fileTpl, $langDesc, $authorTpl, date('Y-m-d'), $langCode, date('Y'),
-                                     $langCharset, strtolower($langCode), $langName, $langDir, $langNPlurals);
+            $fileTpl = sprintf($fileTpl, $langDesc, $authorTpl, date('Y-m-d'), $langCode, date('Y'),
+                                         $langCharset, strtolower($langCode), $langName, $langDir, $langNPlurals);
 
-        $retval = @file_put_contents(PMF_ROOT_DIR.'/lang/language_'.strtolower($langCode).'.php', $fileTpl);
-        echo intval($retval);
-    break;
+            $retval = @file_put_contents(PMF_ROOT_DIR.'/lang/language_'.strtolower($langCode).'.php', $fileTpl);
+            echo intval($retval);
+        break;
 
-    case 'send_translated_file':
+        case 'send_translated_file':
 
-        $lang = PMF_Filter::filterInput(INPUT_GET, 'translang', FILTER_SANITIZE_STRING);
-        $filename = PMF_ROOT_DIR.'/lang/language_'.$lang.'.php';
+            $lang = PMF_Filter::filterInput(INPUT_GET, 'translang', FILTER_SANITIZE_STRING);
+            $filename = PMF_ROOT_DIR.'/lang/language_'.$lang.'.php';
 
-        if (!file_exists($filename)) {
-            echo 0;
-            exit;
-        }
+            if (!file_exists($filename)) {
+                echo 0;
+                exit;
+            }
 
-        $letterTpl = '';
+            $letterTpl = '';
 
-        $mail = new PMF_Mail($faqConfig);
-        $mail->subject = 'New phpMyFAQ language file submitted';
-        $mail->message = sprintf('The file below was sent by %s, which is using phpMyFAQ %s on %s',
-            $user->userdata->get('email'),
-            $faqConfig->get('main.currentVersion'),
-            $_SERVER['HTTP_HOST']);
+            $mail = new PMF_Mail($faqConfig);
+            $mail->subject = 'New phpMyFAQ language file submitted';
+            $mail->message = sprintf('The file below was sent by %s, which is using phpMyFAQ %s on %s',
+                $user->userdata->get('email'),
+                $faqConfig->get('main.currentVersion'),
+                $_SERVER['HTTP_HOST']);
 
-        $mail->addTo('thorsten@phpmyfaq.de');
-        $mail->addAttachment($filename, null, 'text/plain');
+            $mail->addTo('thorsten@phpmyfaq.de');
+            $mail->addAttachment($filename, null, 'text/plain');
 
-        echo (int) $mail->send();
-    break;
+            echo (int) $mail->send();
+        break;
 }

phpmyfaq/admin/ajax.records.php

Switch Indentation +79 added lines, -79 removed lines

@@ -48,98 +48,98 @@
 switch ($ajax_action) {
 
     // save active FAQs
-    case 'save_active_records':
-        if ($user->perm->checkRight($user->getUserId(), 'approverec')) {
-            if (!empty($items)) {
-                $faq = new PMF_Faq($faqConfig);
-
-                foreach ($items as $item) {
-                    if (is_array($item) && count($item) == 3 && PMF_Language::isASupportedLanguage($item[1])) {
-                        echo $faq->updateRecordFlag((int) $item[0], addslashes($item[1]), $item[2], 'active');
+        case 'save_active_records':
+            if ($user->perm->checkRight($user->getUserId(), 'approverec')) {
+                if (!empty($items)) {
+                    $faq = new PMF_Faq($faqConfig);
+
+                    foreach ($items as $item) {
+                        if (is_array($item) && count($item) == 3 && PMF_Language::isASupportedLanguage($item[1])) {
+                            echo $faq->updateRecordFlag((int) $item[0], addslashes($item[1]), $item[2], 'active');
+                        }
                     }
                 }
+            } else {
+                echo $PMF_LANG['err_NotAuth'];
             }
-        } else {
-            echo $PMF_LANG['err_NotAuth'];
-        }
-        break;
-
-    // save sticky FAQs
-    case 'save_sticky_records':
-        if ($user->perm->checkRight($user->getUserId(), 'editbt')) {
-            if (!empty($items)) {
-                $faq = new PMF_Faq($faqConfig);
-
-                foreach ($items as $item) {
-                    if (is_array($item) && count($item) == 3 && PMF_Language::isASupportedLanguage($item[1])) {
-                        echo $faq->updateRecordFlag((int) $item[0], addslashes($item[1]), $item[2], 'sticky');
+            break;
+
+        // save sticky FAQs
+        case 'save_sticky_records':
+            if ($user->perm->checkRight($user->getUserId(), 'editbt')) {
+                if (!empty($items)) {
+                    $faq = new PMF_Faq($faqConfig);
+
+                    foreach ($items as $item) {
+                        if (is_array($item) && count($item) == 3 && PMF_Language::isASupportedLanguage($item[1])) {
+                            echo $faq->updateRecordFlag((int) $item[0], addslashes($item[1]), $item[2], 'sticky');
+                        }
                     }
                 }
+            } else {
+                echo $PMF_LANG['err_NotAuth'];
             }
-        } else {
-            echo $PMF_LANG['err_NotAuth'];
-        }
-        break;
+            break;
 
-    // search FAQs for suggestions
-    case 'search_records':
-        if ($user->perm->checkRight($user->getUserId(), 'editbt')) {
-            $faq = new PMF_Faq($faqConfig);
-            $faqSearch = new PMF_Search($faqConfig);
-            $faqSearch->setCategory(new PMF_Category($faqConfig));
-            $faqSearchResult = new PMF_Search_Resultset($user, $faq, $faqConfig);
-            $searchResult = '';
-            $searchString = PMF_Filter::filterInput(INPUT_POST, 'search', FILTER_SANITIZE_STRIPPED);
+        // search FAQs for suggestions
+        case 'search_records':
+            if ($user->perm->checkRight($user->getUserId(), 'editbt')) {
+                $faq = new PMF_Faq($faqConfig);
+                $faqSearch = new PMF_Search($faqConfig);
+                $faqSearch->setCategory(new PMF_Category($faqConfig));
+                $faqSearchResult = new PMF_Search_Resultset($user, $faq, $faqConfig);
+                $searchResult = '';
+                $searchString = PMF_Filter::filterInput(INPUT_POST, 'search', FILTER_SANITIZE_STRIPPED);
 
-            if (!is_null($searchString)) {
+                if (!is_null($searchString)) {
 
-                $searchResult = $faqSearch->search($searchString, false);
+                    $searchResult = $faqSearch->search($searchString, false);
 
-                $faqSearchResult->reviewResultset($searchResult);
+                    $faqSearchResult->reviewResultset($searchResult);
 
-                $searchHelper = new PMF_Helper_Search($faqConfig);
-                $searchHelper->setSearchterm($searchString);
+                    $searchHelper = new PMF_Helper_Search($faqConfig);
+                    $searchHelper->setSearchterm($searchString);
 
-                echo $searchHelper->renderAdminSuggestionResult($faqSearchResult);
+                    echo $searchHelper->renderAdminSuggestionResult($faqSearchResult);
+                }
+            } else {
+                echo $PMF_LANG['err_NotAuth'];
             }
-        } else {
-            echo $PMF_LANG['err_NotAuth'];
-        }
-        break;
-
-    // delete FAQs
-    case 'delete_record':
-        if ($user->perm->checkRight($user->getUserId(), 'delbt')) {
-            $recordId = PMF_Filter::filterInput(INPUT_POST, 'record_id', FILTER_VALIDATE_INT);
-            $recordLang = PMF_Filter::filterInput(INPUT_POST, 'record_lang', FILTER_SANITIZE_STRING);
-
-            $logging = new PMF_Logging($faqConfig);
-            $logging->logAdmin($user, 'Deleted FAQ ID '.$recordId);
-
-            $faq->deleteRecord($recordId, $recordLang);
-            echo $PMF_LANG['ad_entry_delsuc'];
-        } else {
-            echo $PMF_LANG['err_NotAuth'];
-        }
-        break;
-
-    // delete open questions
-    case 'delete_question':
-        if ($user->perm->checkRight($user->getUserId(), 'delquestion')) {
-            $checks = array(
-                'filter' => FILTER_VALIDATE_INT,
-                'flags' => FILTER_REQUIRE_ARRAY,
-            );
-            $questionIds = PMF_Filter::filterInputArray(INPUT_POST, array('questions' => $checks));
-
-            if (!is_null($questionIds['questions'])) {
-                foreach ($questionIds['questions'] as $questionId) {
-                    $faq->deleteQuestion((int) $questionId);
+            break;
+
+        // delete FAQs
+        case 'delete_record':
+            if ($user->perm->checkRight($user->getUserId(), 'delbt')) {
+                $recordId = PMF_Filter::filterInput(INPUT_POST, 'record_id', FILTER_VALIDATE_INT);
+                $recordLang = PMF_Filter::filterInput(INPUT_POST, 'record_lang', FILTER_SANITIZE_STRING);
+
+                $logging = new PMF_Logging($faqConfig);
+                $logging->logAdmin($user, 'Deleted FAQ ID '.$recordId);
+
+                $faq->deleteRecord($recordId, $recordLang);
+                echo $PMF_LANG['ad_entry_delsuc'];
+            } else {
+                echo $PMF_LANG['err_NotAuth'];
+            }
+            break;
+
+        // delete open questions
+        case 'delete_question':
+            if ($user->perm->checkRight($user->getUserId(), 'delquestion')) {
+                $checks = array(
+                    'filter' => FILTER_VALIDATE_INT,
+                    'flags' => FILTER_REQUIRE_ARRAY,
+                );
+                $questionIds = PMF_Filter::filterInputArray(INPUT_POST, array('questions' => $checks));
+
+                if (!is_null($questionIds['questions'])) {
+                    foreach ($questionIds['questions'] as $questionId) {
+                        $faq->deleteQuestion((int) $questionId);
+                    }
                 }
+                echo $PMF_LANG['ad_entry_delsuc'];
+            } else {
+                echo $PMF_LANG['err_NotAuth'];
             }
-            echo $PMF_LANG['ad_entry_delsuc'];
-        } else {
-            echo $PMF_LANG['err_NotAuth'];
-        }
-        break;
+            break;
 }

phpmyfaq/admin/ajax.image.php

Switch Indentation +40 added lines, -40 removed lines

@@ -36,47 +36,47 @@ 
 }
 switch ($ajaxAction) {
 
-    case 'upload':
+        case 'upload':
 
-        $uploadDir = PMF_ROOT_DIR . '/images/';
-        $uploadFile = basename($_FILES['upload']['name']);
-        $isUploaded = false;
-        $height = $width = 0;
-        $validFileExtensions = [ 'gif', 'jpg', 'jpeg', 'png' ];
+            $uploadDir = PMF_ROOT_DIR . '/images/';
+            $uploadFile = basename($_FILES['upload']['name']);
+            $isUploaded = false;
+            $height = $width = 0;
+            $validFileExtensions = [ 'gif', 'jpg', 'jpeg', 'png' ];
 
-        if ($csrfOkay) {
-            if (is_uploaded_file($uploadedFile['tmp_name']) &&
-                $uploadedFile['size'] < $faqConfig->get('records.maxAttachmentSize')
-            ) {
+            if ($csrfOkay) {
+                if (is_uploaded_file($uploadedFile['tmp_name']) &&
+                    $uploadedFile['size'] < $faqConfig->get('records.maxAttachmentSize')
+                ) {
 
-                $fileInfo = getimagesize($uploadedFile['tmp_name']);
-                $fileExtension = pathinfo($uploadFile, PATHINFO_EXTENSION);;
+                    $fileInfo = getimagesize($uploadedFile['tmp_name']);
+                    $fileExtension = pathinfo($uploadFile, PATHINFO_EXTENSION);;
 
-                if (false === $fileInfo) {
-                    $isUploaded = false;
-                }
-
-                if (($fileInfo[2] !== IMAGETYPE_GIF) &&
-                    ($fileInfo[2] !== IMAGETYPE_JPEG) &&
-                    ($fileInfo[2] !== IMAGETYPE_PNG)) {
-                    $isUploaded = false;
-                } else {
-                    $isUploaded = true;
-                }
-
-                if (!in_array($fileExtension, $validFileExtensions)) {
-                    $isUploaded = false;
-                }
+                    if (false === $fileInfo) {
+                        $isUploaded = false;
+                    }
 
-                if ($fileInfo && $isUploaded) {
-                    list($width, $height) = $fileInfo;
-                    if (move_uploaded_file($uploadedFile['tmp_name'], $uploadDir . $uploadFile)) {
-                        $isUploaded = true;
+                    if (($fileInfo[2] !== IMAGETYPE_GIF) &&
+                        ($fileInfo[2] !== IMAGETYPE_JPEG) &&
+                        ($fileInfo[2] !== IMAGETYPE_PNG)) {
+                        $isUploaded = false;
                     } else {
+                        $isUploaded = true;
+                    }
+
+                    if (!in_array($fileExtension, $validFileExtensions)) {
                         $isUploaded = false;
                     }
-                }
-                ?>
+
+                    if ($fileInfo && $isUploaded) {
+                        list($width, $height) = $fileInfo;
+                        if (move_uploaded_file($uploadedFile['tmp_name'], $uploadDir . $uploadFile)) {
+                            $isUploaded = true;
+                        } else {
+                            $isUploaded = false;
+                        }
+                    }
+                    ?>
                 <script>
                     window.parent.window.pmfImageUpload.uploadFinished({
                         filename: '<?php echo $faqConfig->getDefaultUrl() . 'images/' . $uploadFile ?>',
@@ -87,8 +87,8 @@ 
                     });
                 </script>
                 <?php
-            } else {
-                ?>
+                } else {
+                    ?>
                 <script>
                     window.parent.window.pmfImageUpload.uploadFinished({
                         filename: '',
@@ -99,9 +99,9 @@ 
                     });
                 </script>
                 <?php
-            }
-        } else {
-            ?>
+                }
+            } else {
+                ?>
             <script>
                 window.parent.window.pmfImageUpload.uploadFinished({
                     filename: '',
@@ -112,7 +112,7 @@ 
                 });
             </script>
             <?php
-        }
+            }
 
-        break;
+            break;
 }
\ No newline at end of file