Skip to content

Commit d27e9a4

Browse files
committed
#18075: No param fix.
1 parent db60f71 commit d27e9a4

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

setup/src/Magento/Setup/Model/ConfigOptionsList/DriverOptions.php

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public function getDriverOptions(array $options): array
3737
}
3838
}
3939
foreach ($booleanDriverOptionKeys as $configKey => $driverOptionKey) {
40-
$driverOptions[$configKey] = $this->getBooleanValue($options, $driverOptionKey);
40+
$driverOptions[$configKey] = $this->booleanValue($options, $driverOptionKey);
4141
}
4242

4343
return $driverOptions;
@@ -52,7 +52,8 @@ public function getDriverOptions(array $options): array
5252
*/
5353
private function optionExists(array $options, string $driverOptionKey): bool
5454
{
55-
return $options[$driverOptionKey] === false || !empty($options[$driverOptionKey]);
55+
return isset($options[$driverOptionKey])
56+
&& ($options[$driverOptionKey] === false || !empty($options[$driverOptionKey]));
5657
}
5758

5859
/**
@@ -64,8 +65,8 @@ private function optionExists(array $options, string $driverOptionKey): bool
6465
* @param string $driverOptionKey
6566
* @return bool
6667
*/
67-
private function getBooleanValue(array $options, string $driverOptionKey): bool
68+
private function booleanValue(array $options, string $driverOptionKey): bool
6869
{
69-
return isset($options[$driverOptionKey]) ? (bool)$options[$driverOptionKey] : false;
70+
return isset($options[$driverOptionKey]) && (bool)$options[$driverOptionKey];
7071
}
7172
}

0 commit comments

Comments
 (0)