Pārlūkot izejas kodu

fix ot_admin commands to match new user_upload structure

Olivier Massot 4 gadi atpakaļ
vecāks
revīzija
86fcc1be50

+ 7 - 19
ot_admin/Classes/Controller/SiteController.php

@@ -5,8 +5,8 @@ namespace Opentalent\OtAdmin\Controller;
 use http\Exception\RuntimeException;
 use Opentalent\OtAdmin\Domain\Entity\SiteInfos;
 use Opentalent\OtAdmin\Domain\Entity\SiteStatus;
-use Opentalent\OtAdmin\Exception\NoSuchRecordException;
-use Opentalent\OtAdmin\Exception\NoSuchWebsiteException;
+use Opentalent\OtCore\Exception\NoSuchRecordException;
+use Opentalent\OtCore\Exception\NoSuchWebsiteException;
 use Opentalent\OtCore\Cache\OtCacheManager;
 use Opentalent\OtCore\Controller\ActionController;
 use Opentalent\OtCore\Domain\Model\Organization;
@@ -525,8 +525,6 @@ class SiteController extends ActionController
             if (file_exists($uploadDir)) {
                 throw new \RuntimeException("A directory or file " . $uploadDir . " already exists. Abort.");
             }
-            $defaultUploadRelPath = $uploadRelPath . '/images';
-            $defaultUploadDir = $uploadDir . '/images';
 
             $formsRelPath = '/form_definitions/' . $organizationId;
             $formsDir = $fileadminDir . $formsRelPath;
@@ -535,7 +533,6 @@ class SiteController extends ActionController
             }
 
             $this->mkDir($uploadDir);
-            $this->mkDir($defaultUploadDir);
             $this->mkDir($formsDir);
 
             // Insert the filemounts points (sys_filemounts)
@@ -543,7 +540,7 @@ class SiteController extends ActionController
             $queryBuilder->insert('sys_filemounts')
                 ->values([
                     'title' => 'Documents',
-                    'path' => rtrim($defaultUploadRelPath, '/') . '/',
+                    'path' => rtrim($uploadRelPath, '/') . '/',
                     'base' => 1
                 ])
                 ->execute();
@@ -576,7 +573,7 @@ class SiteController extends ActionController
 
             // Update the user TsConfig
             $tsconfig = "options.uploadFieldsInTopOfEB = 1\n" .
-                        "options.defaultUploadFolder=1:" . $defaultUploadDir . "\n";
+                        "options.defaultUploadFolder=1:" . rtrim($uploadRelPath, '/') . "/\n";
             $queryBuilder = $this->connectionPool->getQueryBuilderForTable('be_users');
             $queryBuilder
                 ->update('be_users')
@@ -590,7 +587,7 @@ class SiteController extends ActionController
             // Try to commit the result
             $commitSuccess = $this->connectionPool->getConnectionByName('Default')->commit();
             if (!$commitSuccess) {
-                throw new \RuntimeException('Something went wrong while commiting the result');
+                throw new \RuntimeException('Something went wrong while committing the result');
             }
 
         } catch(\Throwable $e) {
@@ -832,9 +829,6 @@ class SiteController extends ActionController
                 if (is_dir(dirname($configYamlFile))) {
                     FileUtility::rmdir(dirname($configYamlFile), $force);
                 }
-                if (is_dir($uploadDir . 'images')) {
-                    FileUtility::rmdir($uploadDir . 'images', $force);
-                }
                 if (is_dir($uploadDir)) {
                     FileUtility::rmdir($uploadDir, $force);
                 }
@@ -965,13 +959,7 @@ class SiteController extends ActionController
             $queryBuilder
                 ->update('sys_filemounts')
                 ->set('deleted', 0)
-                ->where($queryBuilder->expr()->eq('path', "'/user_upload/" . $organizationId . "/images'"))
-                ->execute();
-
-            $queryBuilder
-                ->update('sys_filemounts')
-                ->set('deleted', 0)
-                ->where($queryBuilder->expr()->eq('path', "'/user_upload/" . $organizationId . "/Forms'"))
+                ->where($queryBuilder->expr()->eq('path', "'/user_upload/" . $organizationId . "/'"))
                 ->execute();
 
 
@@ -2224,7 +2212,7 @@ class SiteController extends ActionController
         $queryBuilder
             ->select('uid')
             ->from('sys_filemounts')
-            ->where("path LIKE '%user_upload/" . $organizationId . "/%'")
+            ->where("path LIKE '%user_upload/" . $organizationId . "/'")
             ->orWhere("path LIKE '%form_definitions/" . $organizationId . "/'");
         $statement = $queryBuilder->execute();
         $rows = $statement->fetchAll(3) ?: [];

+ 0 - 0
ot_admin/Classes/Exception/NoSuchRecordException.php → ot_core/Classes/Exception/NoSuchRecordException.php


+ 0 - 0
ot_admin/Classes/Exception/NoSuchWebsiteException.php → ot_core/Classes/Exception/NoSuchWebsiteException.php