Fixing bug created by doctrine upgrade.

krulis-martin · krulis-martin · commit 72727cc105e4 · 2025-11-04T21:07:31.000+01:00
diff --git a/app/commands/DoctrineFixtures.php b/app/commands/DoctrineFixtures.php
@@ -8,7 +8,6 @@
 use Doctrine\ORM\Id\AssignedGenerator;
 use Nette\Utils\Finder;
 use Nette\Utils\FileInfo;
-use SplFileInfo;
 use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
diff --git a/app/model/repository/ShadowAssignmentPointsRepository.php b/app/model/repository/ShadowAssignmentPointsRepository.php
@@ -26,6 +26,10 @@ public function __construct(EntityManagerInterface $em)
      */
     public function findPointsForAssignments(array $shadowAssignments, User $awardee): array
     {
+        if (!$shadowAssignments) {
+            return []; // we should not ask a query with empty IN ()
+        }
+
         $pointsList = $this->findBy(
             [
                 "awardee" => $awardee,
diff --git a/composer.json b/composer.json
@@ -58,7 +58,6 @@
         "nettrine/extensions-atlantic18": "^0.6.0",
         "nettrine/migrations": "^0.10",
         "nettrine/orm": "^0.9",
-        "mrdm-nl/ldap": "^1.1",
         "ramsey/uuid-doctrine": "^2.0",
         "symfony/process": "^7.3",
         "tracy/tracy": "^2.11",
diff --git a/composer.lock b/composer.lock

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,10 @@ public function __construct(EntityManagerInterface $em)`
`26`	`26`	`*/`
`27`	`27`	`public function findPointsForAssignments(array $shadowAssignments, User $awardee): array`
`28`	`28`	`{`
	`29`	`+ if (!$shadowAssignments) {`
	`30`	`+ return []; // we should not ask a query with empty IN ()`
	`31`	`+ }`
	`32`	`+`
`29`	`33`	`$pointsList = $this->findBy(`
`30`	`34`	`[`
`31`	`35`	`"awardee" => $awardee,`