fix: added missing error case

thorsten · Jul 21, 2024 · 786ffa4 · 786ffa4
1 parent 1fde281
commit 786ffa4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 16 deletions.
diff --git a/phpmyfaq/admin/assets/src/configuration/upgrade.js b/phpmyfaq/admin/assets/src/configuration/upgrade.js
@@ -41,6 +41,10 @@ export const handleCheckForUpdates = () => {
           card.classList.add('text-bg-warning');
           result.replaceWith(addElement('p', { innerText: responseData.warning }));
         }
+        if (responseData.error) {
+          card.classList.add('text-bg-danger');
+          result.replaceWith(addElement('p', { innerText: responseData.error }));
+        }
       } catch (error) {
         if (error.cause && error.cause.response) {
           const errorMessage = await error.cause.response.json();

diff --git a/phpmyfaq/src/phpMyFAQ/Setup/Upgrade.php b/phpmyfaq/src/phpMyFAQ/Setup/Upgrade.php
@@ -82,22 +82,6 @@ public function checkFilesystem(): bool
             throw new Exception('The folder /phpmyfaq/assets/themes is missing.');
         }
 
-        if (!is_dir(PMF_CONTENT_DIR . '/user/attachments')) {
-            return false;
-        }
-
-        if (!is_dir(PMF_CONTENT_DIR . '/user/images')) {
-            return false;
-        }
-
-        if (!is_dir(PMF_CONTENT_DIR . '/core/data')) {
-            return false;
-        }
-
-        if (!is_dir(PMF_ROOT_DIR . '/assets/themes')) {
-            return false;
-        }
-
         if (
             is_file(PMF_CONTENT_DIR . '/core/config/constants.php') &&
             is_file(PMF_CONTENT_DIR . '/core/config/database.php')
@@ -124,6 +108,7 @@ public function checkFilesystem(): bool
 
             return true;
         }
+
         throw new Exception(
             'The files /content/core/config/constant.php and /content/core/config/database.php are missing.'
         );