This commit is contained in:
Dennis Eichhorn 2024-02-04 20:34:12 +00:00
parent 792ee4018c
commit 831af335af
2 changed files with 55 additions and 58 deletions

View File

@ -112,8 +112,10 @@ final class ApiEquipmentController extends Controller
{ {
$path = $this->createEquipmentDir($equipment); $path = $this->createEquipmentDir($equipment);
$collection = null;
if (!empty($uploadedFiles = $request->files)) { if (!empty($uploadedFiles = $request->files)) {
$uploaded = $this->app->moduleManager->get('Media')->uploadFiles( $uploaded = $this->app->moduleManager->get('Media', 'Api')->uploadFiles(
names: [], names: [],
fileNames: [], fileNames: [],
files: $uploadedFiles, files: $uploadedFiles,
@ -123,7 +125,6 @@ final class ApiEquipmentController extends Controller
pathSettings: PathSettings::FILE_PATH pathSettings: PathSettings::FILE_PATH
); );
$collection = null;
foreach ($uploaded as $media) { foreach ($uploaded as $media) {
$this->createModelRelation( $this->createModelRelation(
$request->header->account, $request->header->account,
@ -160,9 +161,7 @@ final class ApiEquipmentController extends Controller
} }
} }
if (!empty($mediaFiles = $request->getDataJson('media'))) { $mediaFiles = $request->getDataJson('media');
$collection = null;
foreach ($mediaFiles as $file) { foreach ($mediaFiles as $file) {
/** @var \Modules\Media\Models\Media $media */ /** @var \Modules\Media\Models\Media $media */
$media = MediaMapper::get()->where('id', (int) $file)->limit(1)->execute(); $media = MediaMapper::get()->where('id', (int) $file)->limit(1)->execute();
@ -209,7 +208,6 @@ final class ApiEquipmentController extends Controller
); );
} }
} }
}
/** /**
* Validate equipment create request * Validate equipment create request
@ -260,7 +258,7 @@ final class ApiEquipmentController extends Controller
$uploaded = []; $uploaded = [];
if (!empty($uploadedFiles = $request->files)) { if (!empty($uploadedFiles = $request->files)) {
$uploaded = $this->app->moduleManager->get('Media')->uploadFiles( $uploaded = $this->app->moduleManager->get('Media', 'Api')->uploadFiles(
names: [], names: [],
fileNames: [], fileNames: [],
files: $uploadedFiles, files: $uploadedFiles,
@ -321,7 +319,7 @@ final class ApiEquipmentController extends Controller
} }
} }
if (!empty($mediaFiles = $request->getDataJson('media'))) { $mediaFiles = $request->getDataJson('media');
foreach ($mediaFiles as $media) { foreach ($mediaFiles as $media) {
$this->createModelRelation( $this->createModelRelation(
$request->header->account, $request->header->account,
@ -333,7 +331,6 @@ final class ApiEquipmentController extends Controller
$request->getOrigin() $request->getOrigin()
); );
} }
}
$this->fillJsonResponse($request, $response, NotificationLevel::OK, 'Media', 'Media added to equipment.', [ $this->fillJsonResponse($request, $response, NotificationLevel::OK, 'Media', 'Media added to equipment.', [
'upload' => $uploaded, 'upload' => $uploaded,
@ -562,8 +559,6 @@ final class ApiEquipmentController extends Controller
* *
* @return array<string, bool> * @return array<string, bool>
* *
* @todo Implement API validation function
*
* @since 1.0.0 * @since 1.0.0
*/ */
private function validateEquipmentDelete(RequestAbstract $request) : array private function validateEquipmentDelete(RequestAbstract $request) : array

View File

@ -212,6 +212,8 @@ final class BackendController extends Controller
$view->data['inspections'] = $inspections; $view->data['inspections'] = $inspections;
// @feature Create a new read mapper function that returns relation models instead of its own model
// https://github.com/Karaka-Management/phpOMS/issues/320
$query = new Builder($this->app->dbPool->get()); $query = new Builder($this->app->dbPool->get());
$results = $query->selectAs(EquipmentMapper::HAS_MANY['files']['external'], 'file') $results = $query->selectAs(EquipmentMapper::HAS_MANY['files']['external'], 'file')
->from(EquipmentMapper::TABLE) ->from(EquipmentMapper::TABLE)