static code analysis fixes

This commit is contained in:
Dennis Eichhorn 2022-04-05 20:04:48 +02:00
parent 50cfd020b9
commit 719f75b434
2 changed files with 9 additions and 9 deletions

View File

@ -94,8 +94,8 @@ final class ApiController extends Controller
$task = $this->createTaskFromRequest($request); $task = $this->createTaskFromRequest($request);
$this->createModel($request->header->account, $task, TaskMapper::class, 'task', $request->getOrigin()); $this->createModel($request->header->account, $task, TaskMapper::class, 'task', $request->getOrigin());
if (!empty($request->getFiles() ?? []) if (!empty($request->getFiles())
|| !empty($request->getDataJson('media') ?? []) || !empty($request->getDataJson('media'))
) { ) {
$this->createTaskMedia($task, $request); $this->createTaskMedia($task, $request);
} }
@ -118,7 +118,7 @@ final class ApiController extends Controller
$path = $this->createTaskDir($task); $path = $this->createTaskDir($task);
$account = AccountMapper::get()->where('id', $request->header->account)->execute(); $account = AccountMapper::get()->where('id', $request->header->account)->execute();
if (!empty($uploadedFiles = $request->getFiles() ?? [])) { if (!empty($uploadedFiles = $request->getFiles())) {
$uploaded = $this->app->moduleManager->get('Media')->uploadFiles( $uploaded = $this->app->moduleManager->get('Media')->uploadFiles(
[], [],
[], [],
@ -159,7 +159,7 @@ final class ApiController extends Controller
} }
} }
if (!empty($mediaFiles = $request->getDataJson('media') ?? [])) { if (!empty($mediaFiles = $request->getDataJson('media'))) {
$collection = null; $collection = null;
foreach ($mediaFiles as $file) { foreach ($mediaFiles as $file) {
@ -393,8 +393,8 @@ final class ApiController extends Controller
$this->createModel($request->header->account, $element, TaskElementMapper::class, 'taskelement', $request->getOrigin()); $this->createModel($request->header->account, $element, TaskElementMapper::class, 'taskelement', $request->getOrigin());
if (!empty($request->getFiles() ?? []) if (!empty($request->getFiles())
|| !empty($request->getDataJson('media') ?? []) || !empty($request->getDataJson('media'))
) { ) {
$this->createTaskElementMedia($task, $element, $request); $this->createTaskElementMedia($task, $element, $request);
} }
@ -419,7 +419,7 @@ final class ApiController extends Controller
$path = $this->createTaskDir($task); $path = $this->createTaskDir($task);
$account = AccountMapper::get()->where('id', $request->header->account)->execute(); $account = AccountMapper::get()->where('id', $request->header->account)->execute();
if (!empty($uploadedFiles = $request->getFiles() ?? [])) { if (!empty($uploadedFiles = $request->getFiles())) {
$uploaded = $this->app->moduleManager->get('Media')->uploadFiles( $uploaded = $this->app->moduleManager->get('Media')->uploadFiles(
[], [],
[], [],
@ -457,7 +457,7 @@ final class ApiController extends Controller
} }
} }
if (!empty($mediaFiles = $request->getDataJson('media') ?? [])) { if (!empty($mediaFiles = $request->getDataJson('media'))) {
$collection = null; $collection = null;
foreach ($mediaFiles as $file) { foreach ($mediaFiles as $file) {

View File

@ -12,7 +12,7 @@
"friendsofphp/php-cs-fixer": ">=3.2", "friendsofphp/php-cs-fixer": ">=3.2",
"squizlabs/php_codesniffer": ">=3.6", "squizlabs/php_codesniffer": ">=3.6",
"phpmd/phpmd": ">=2.9", "phpmd/phpmd": ">=2.9",
"phpstan/phpstan": ">=0.12.58", "phpstan/phpstan": ">=1.5.4",
"phan/phan": ">=3.2.6" "phan/phan": ">=3.2.6"
}, },
"minimum-stability": "dev", "minimum-stability": "dev",