Skip to content

Commit 681498a

Browse files
committed
fix(core) null safe on metadata
Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
1 parent 9b6424e commit 681498a

2 files changed

Lines changed: 10 additions & 7 deletions

File tree

build/psalm-baseline.xml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2139,6 +2139,14 @@
21392139
<code>$file</code>
21402140
</MoreSpecificImplementedParamType>
21412141
</file>
2142+
<file src="lib/private/Files/Cache/Scanner.php">
2143+
<RedundantCondition>
2144+
<code>$metadataQuery?->extractMetadata($data)->asArray() ?? []</code>
2145+
</RedundantCondition>
2146+
<TypeDoesNotContainNull>
2147+
<code>$metadataQuery?->extractMetadata($data)->asArray() ?? []</code>
2148+
</TypeDoesNotContainNull>
2149+
</file>
21422150
<file src="lib/private/Files/Cache/Scanner.php">
21432151
<InvalidArgument>
21442152
<code>self::SCAN_RECURSIVE_INCOMPLETE</code>

lib/private/Files/Cache/QuerySearchHelper.php

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -194,13 +194,8 @@ public function searchInCaches(ISearchQuery $searchQuery, array $caches): array
194194
$result = $query->execute();
195195
$files = $result->fetchAll();
196196

197-
$rawEntries = array_map(function (array $data) use ($metadataQuery) {
198-
// migrate to null safe ...
199-
if ($metadataQuery === null) {
200-
$data['metadata'] = [];
201-
} else {
202-
$data['metadata'] = $metadataQuery->extractMetadata($data)->asArray();
203-
}
197+
$rawEntries = array_map(function (array $data) use ($metadataQuery): CacheEntry {
198+
$data['metadata'] = $metadataQuery?->extractMetadata($data)->asArray() ?? [];
204199
return Cache::cacheEntryFromData($data, $this->mimetypeLoader);
205200
}, $files);
206201

0 commit comments

Comments
 (0)