Skip to content

Commit

Permalink
(U) JAM: Fix face clustering on instances that use numeric user IDs
Browse files Browse the repository at this point in the history
  • Loading branch information
joeyame authored and backportbot[bot] committed Jan 9, 2025
1 parent c7d19b9 commit 5fce506
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 5fce506

Please sign in to comment.