Skip to content

Commit 38d3044

Browse files
dgdg
committed
cs
1 parent 54a435c commit 38d3044

17 files changed

+46
-33
lines changed

‎src/Utils/ArrayHash.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public static function from(array $array, bool $recursive = true): static
2929
$obj = new static;
3030
foreach ($array as $key => $value) {
3131
$obj->$key = $recursive && is_array($value)
32-
? static::from($value, true)
32+
? static::from($value)
3333
: $value;
3434
}
3535

‎src/Utils/Arrays.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ public static function mergeTree(array $array1, array $array2): array
9898
*/
9999
public static function getKeyOffset(array $array, string|int $key): ?int
100100
{
101-
return Helpers::falseToNull(array_search(self::toKey($key), array_keys($array), true));
101+
return Helpers::falseToNull(array_search(self::toKey($key), array_keys($array), strict: true));
102102
}
103103

104104

@@ -151,9 +151,9 @@ public static function last(array $array): mixed
151151
public static function insertBefore(array &$array, string|int|null $key, array $inserted): void
152152
{
153153
$offset = $key === null ? 0 : (int) self::getKeyOffset($array, $key);
154-
$array = array_slice($array, 0, $offset, true)
154+
$array = array_slice($array, 0, $offset, preserve_keys: true)
155155
+ $inserted
156-
+ array_slice($array, $offset, count($array), true);
156+
+ array_slice($array, $offset, count($array), preserve_keys: true);
157157
}
158158

159159

@@ -167,9 +167,9 @@ public static function insertAfter(array &$array, string|int|null $key, array $i
167167
$offset = count($array) - 1;
168168
}
169169

170-
$array = array_slice($array, 0, $offset + 1, true)
170+
$array = array_slice($array, 0, $offset + 1, preserve_keys: true)
171171
+ $inserted
172-
+ array_slice($array, $offset + 1, count($array), true);
172+
+ array_slice($array, $offset + 1, count($array), preserve_keys: true);
173173
}
174174

175175

‎src/Utils/FileSystem.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ final class FileSystem
2323
*/
2424
public static function createDir(string $dir, int $mode = 0777): void
2525
{
26-
if (!is_dir($dir) && !@mkdir($dir, $mode, true) && !is_dir($dir)) { // @ - dir may already exist
26+
if (!is_dir($dir) && !@mkdir($dir, $mode, recursive: true) && !is_dir($dir)) { // @ - dir may already exist
2727
throw new Nette\IOException(sprintf(
2828
"Unable to create directory '%s' with mode %s. %s",
2929
self::normalizePath($dir),

‎src/Utils/Html.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -628,7 +628,7 @@ public function insert(?int $index, HtmlStringable|string $child, bool $replace
628628
*/
629629
final public function offsetSet($index, $child): void
630630
{
631-
$this->insert($index, $child, true);
631+
$this->insert($index, $child, replace: true);
632632
}
633633

634634

‎src/Utils/ObjectHelpers.php

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,11 @@ public static function strictCall(string $class, string $method, array $addition
6262
? ($trace[2]['class'] ?? null)
6363
: null;
6464

65-
if ($context && is_a($class, $context, true) && method_exists($context, $method)) { // called parent::$method()
65+
if (
66+
$context
67+
&& is_a($class, $context, allow_string: true)
68+
&& method_exists($context, $method)
69+
) { // called parent::$method()
6670
$class = get_parent_class($context);
6771
}
6872

@@ -95,7 +99,11 @@ public static function strictStaticCall(string $class, string $method): void
9599
? ($trace[2]['class'] ?? null)
96100
: null;
97101

98-
if ($context && is_a($class, $context, true) && method_exists($context, $method)) { // called parent::$method()
102+
if (
103+
$context
104+
&& is_a($class, $context, allow_string: true)
105+
&& method_exists($context, $method)
106+
) { // called parent::$method()
99107
$class = get_parent_class($context);
100108
}
101109

‎src/Utils/Type.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public static function fromReflection(
3535
? $reflection->getReturnType() ?? (PHP_VERSION_ID >= 80100 && $reflection instanceof \ReflectionMethod ? $reflection->getTentativeReturnType() : null)
3636
: $reflection->getType();
3737

38-
return $type ? self::fromReflectionType($type, $reflection, true) : null;
38+
return $type ? self::fromReflectionType($type, $reflection, asObject: true) : null;
3939
}
4040

4141

@@ -49,7 +49,7 @@ private static function fromReflectionType(\ReflectionType $type, $of, bool $asO
4949

5050
} elseif ($type instanceof \ReflectionUnionType || $type instanceof \ReflectionIntersectionType) {
5151
return new self(
52-
array_map(fn($t) => self::fromReflectionType($t, $of, false), $type->getTypes()),
52+
array_map(fn($t) => self::fromReflectionType($t, $of, asObject: false), $type->getTypes()),
5353
$type instanceof \ReflectionUnionType ? '|' : '&',
5454
);
5555

@@ -109,7 +109,7 @@ public static function resolve(
109109

110110
private function __construct(array $types, string $kind = '|')
111111
{
112-
$o = array_search('null', $types, true);
112+
$o = array_search('null', $types, strict: true);
113113
if ($o !== false) { // null as last
114114
array_splice($types, $o, 1);
115115
$types[] = 'null';
@@ -260,7 +260,7 @@ private function allows3(array $types, array $subtypes): bool
260260
$subtypes,
261261
fn($subtype) => Validators::isBuiltinType($type)
262262
? strcasecmp($type, $subtype) === 0
263-
: is_a($subtype, $type, true)
263+
: is_a($subtype, $type, allow_string: true)
264264
)
265265
);
266266
}

‎src/Utils/Validators.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ public static function assert(mixed $value, string $expected, string $label = 'v
102102
$translate = ['boolean' => 'bool', 'integer' => 'int', 'double' => 'float', 'NULL' => 'null'];
103103
$type = $translate[gettype($value)] ?? gettype($value);
104104
if (is_int($value) || is_float($value) || (is_string($value) && strlen($value) < 40)) {
105-
$type .= ' ' . var_export($value, true);
105+
$type .= ' ' . var_export($value, return: true);
106106
} elseif (is_object($value)) {
107107
$type .= ' ' . $value::class;
108108
}
@@ -245,7 +245,7 @@ public static function isNumeric(mixed $value): bool
245245
*/
246246
public static function isCallable(mixed $value): bool
247247
{
248-
return $value && is_callable($value, true);
248+
return $value && is_callable($value, syntax_only: true);
249249
}
250250

251251

‎tests/Utils/ArrayHash.phpt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ test('', function () {
8787
'children' => [
8888
'c' => 'John',
8989
],
90-
], false);
90+
], recursive: false);
9191
Assert::type(Nette\Utils\ArrayHash::class, $list);
9292
Assert::type('array', $list['children']);
9393
});

‎tests/Utils/Arrays.flatten().phpt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ $res = Arrays::flatten([
3535
],
3636
'y' => 'd',
3737
'z' => 'e',
38-
], true);
38+
], preserveKeys: true);
3939

4040
Assert::same([
4141
5 => 'a',

‎tests/Utils/Arrays.normalize.phpt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,5 +37,5 @@ Assert::same(
3737
Arrays::normalize([
3838
1 => 'first',
3939
'' => 'second',
40-
], true),
40+
], filling: true),
4141
);

0 commit comments

Comments
 (0)