Skip to content

Commit

Permalink
Merge pull request #1230 from nextcloud/backport/1229/stable8
Browse files Browse the repository at this point in the history
[stable8] (U) Fix to ClusterFacesJob.php in Instances with Numeric User IDs
  • Loading branch information
marcelklehr authored Jan 9, 2025
2 parents c7d19b9 + 5fce506 commit f5b3718
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/BackgroundJobs/ClusterFacesJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ public function __construct(ITimeFactory $time, Logger $logger, IJobList $jobLis
*/
protected function run($argument) {
/** @var string $userId */

Check failure on line 40 in lib/BackgroundJobs/ClusterFacesJob.php

View workflow job for this annotation

GitHub Actions / Nextcloud (8.1, dev-stable30)

UnnecessaryVarAnnotation

lib/BackgroundJobs/ClusterFacesJob.php:40:12: UnnecessaryVarAnnotation: The @var string annotation for $userId is unnecessary (see https://psalm.dev/212)

Check failure on line 40 in lib/BackgroundJobs/ClusterFacesJob.php

View workflow job for this annotation

GitHub Actions / Nextcloud (8.2, dev-stable30)

UnnecessaryVarAnnotation

lib/BackgroundJobs/ClusterFacesJob.php:40:12: UnnecessaryVarAnnotation: The @var string annotation for $userId is unnecessary (see https://psalm.dev/212)

Check failure on line 40 in lib/BackgroundJobs/ClusterFacesJob.php

View workflow job for this annotation

GitHub Actions / Nextcloud (8.3, dev-stable30)

UnnecessaryVarAnnotation

lib/BackgroundJobs/ClusterFacesJob.php:40:12: UnnecessaryVarAnnotation: The @var string annotation for $userId is unnecessary (see https://psalm.dev/212)
$userId = $argument['userId'];
$userId = (string) $argument['userId'];
try {
$this->clusterAnalyzer->calculateClusters($userId, self::BATCH_SIZE);
} catch (\JsonException|Exception $e) {
Expand Down

0 comments on commit f5b3718

Please sign in to comment.