diff --git a/api/components/com_media/src/Controller/MediaController.php b/api/components/com_media/src/Controller/MediaController.php index ed21098dc32bf..328e56819a283 100644 --- a/api/components/com_media/src/Controller/MediaController.php +++ b/api/components/com_media/src/Controller/MediaController.php @@ -212,7 +212,7 @@ public function add(): void } // Content is only required when it is a file - if (empty($content) && strpos($path, '.') !== false) { + if (empty($content) && str_contains($path, '.')) { $missingParameters[] = 'content'; } diff --git a/api/components/com_media/src/Model/AdapterModel.php b/api/components/com_media/src/Model/AdapterModel.php index 34e731578acfd..70b6d2b2837d0 100644 --- a/api/components/com_media/src/Model/AdapterModel.php +++ b/api/components/com_media/src/Model/AdapterModel.php @@ -35,7 +35,7 @@ class AdapterModel extends BaseModel */ public function getItem(): \stdClass { - list($provider, $account) = array_pad(explode('-', $this->getState('id'), 2), 2, null); + [$provider, $account] = array_pad(explode('-', $this->getState('id'), 2), 2, null); if ($account === null) { throw new \Exception('Account was not set'); diff --git a/api/components/com_media/src/Model/MediaModel.php b/api/components/com_media/src/Model/MediaModel.php index 6b9e00afca441..445478803d6b1 100644 --- a/api/components/com_media/src/Model/MediaModel.php +++ b/api/components/com_media/src/Model/MediaModel.php @@ -77,7 +77,7 @@ public function getItems(): array ['adapter' => $adapterName, 'path' => $path] = $this->resolveAdapterAndPath($this->getState('path', '')); try { $files = $this->mediaApiModel->getFiles($adapterName, $path, $options); - } catch (FileNotFoundException $e) { + } catch (FileNotFoundException) { throw new ResourceNotFound( Text::sprintf('WEBSERVICE_COM_MEDIA_FILE_NOT_FOUND', $path), 404 diff --git a/api/components/com_media/src/Model/MediumModel.php b/api/components/com_media/src/Model/MediumModel.php index dcf2df8fe96d6..0b0025ec8d2c5 100644 --- a/api/components/com_media/src/Model/MediumModel.php +++ b/api/components/com_media/src/Model/MediumModel.php @@ -69,7 +69,7 @@ public function getItem() try { return $this->mediaApiModel->getFile($adapterName, $path, $options); - } catch (FileNotFoundException $e) { + } catch (FileNotFoundException) { throw new ResourceNotFound( Text::sprintf('WEBSERVICE_COM_MEDIA_FILE_NOT_FOUND', $path), 404 @@ -121,7 +121,7 @@ public function save($path = null): string $this->mediaApiModel->move($adapterName, $oldPath, $path, $override), '/' ); - } catch (FileNotFoundException $e) { + } catch (FileNotFoundException) { throw new Save( Text::sprintf( 'WEBSERVICE_COM_MEDIA_FILE_NOT_FOUND', @@ -159,7 +159,7 @@ public function save($path = null): string ); $resultPath = $dirname . '/' . $name; - } catch (FileNotFoundException $e) { + } catch (FileNotFoundException) { throw new Save( Text::sprintf( 'WEBSERVICE_COM_MEDIA_FILE_NOT_FOUND', @@ -167,7 +167,7 @@ public function save($path = null): string ), 404 ); - } catch (FileExistsException $e) { + } catch (FileExistsException) { throw new Save( Text::sprintf( 'WEBSERVICE_COM_MEDIA_FILE_EXISTS', @@ -175,7 +175,7 @@ public function save($path = null): string ), 400 ); - } catch (InvalidPathException $e) { + } catch (InvalidPathException) { throw new Save( Text::sprintf( 'WEBSERVICE_COM_MEDIA_BAD_FILE_TYPE', @@ -201,7 +201,7 @@ public function save($path = null): string $dirname, $content ); - } catch (FileNotFoundException $e) { + } catch (FileNotFoundException) { throw new Save( Text::sprintf( 'WEBSERVICE_COM_MEDIA_FILE_NOT_FOUND', @@ -209,7 +209,7 @@ public function save($path = null): string ), 404 ); - } catch (InvalidPathException $e) { + } catch (InvalidPathException) { throw new Save( Text::sprintf( 'WEBSERVICE_COM_MEDIA_BAD_FILE_TYPE', @@ -250,7 +250,7 @@ public function delete(): void try { $this->mediaApiModel->delete($adapterName, $path); - } catch (FileNotFoundException $e) { + } catch (FileNotFoundException) { throw new Save( Text::sprintf('WEBSERVICE_COM_MEDIA_FILE_NOT_FOUND', $path), 404