Code quality improvement
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
This commit is contained in:
parent
b812277fca
commit
a59809ddc8
@ -5982,7 +5982,7 @@ parameters:
|
||||
|
||||
-
|
||||
message: "#^Construct empty\\(\\) is not allowed\\. Use more strict comparison\\.$#"
|
||||
count: 2
|
||||
count: 1
|
||||
path: src/Controllers/Table/DropColumnController.php
|
||||
|
||||
-
|
||||
@ -8222,7 +8222,7 @@ parameters:
|
||||
|
||||
-
|
||||
message: "#^Construct empty\\(\\) is not allowed\\. Use more strict comparison\\.$#"
|
||||
count: 3
|
||||
count: 2
|
||||
path: src/Controllers/VersionCheckController.php
|
||||
|
||||
-
|
||||
|
||||
@ -62,7 +62,7 @@ final class DropColumnController extends AbstractController
|
||||
$message = Message::success(__('No change'));
|
||||
}
|
||||
|
||||
if (empty($message)) {
|
||||
if (! isset($message)) {
|
||||
$message = Message::success(
|
||||
_ngettext(
|
||||
'%1$d column has been dropped successfully.',
|
||||
|
||||
@ -41,7 +41,7 @@ class VersionCheckController extends AbstractController
|
||||
|
||||
$versionDetails = $this->versionInformation->getLatestVersion();
|
||||
|
||||
if (empty($versionDetails)) {
|
||||
if ($versionDetails === null) {
|
||||
echo json_encode([]);
|
||||
|
||||
return;
|
||||
|
||||
@ -316,7 +316,7 @@ abstract class GisGeometry
|
||||
{
|
||||
$parts = explode('),(', $wktCoords);
|
||||
|
||||
return array_map(fn (string $coord) => $this->extractPoints1d($coord, $scaleData), $parts);
|
||||
return array_map(fn (string $coord): array => $this->extractPoints1d($coord, $scaleData), $parts);
|
||||
}
|
||||
|
||||
/**
|
||||
@ -329,7 +329,7 @@ abstract class GisGeometry
|
||||
{
|
||||
$parts = explode(')),((', $wktCoords);
|
||||
|
||||
return array_map(fn (string $coord) => $this->extractPoints2d($coord, $scaleData), $parts);
|
||||
return array_map(fn (string $coord): array => $this->extractPoints2d($coord, $scaleData), $parts);
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Loading…
Reference in New Issue
Block a user