Remove dump and debug statements

Michael Vasseur · Michael Vasseur · commit 031d24a67ef3 · 2024-10-18T21:03:27.000+02:00
diff --git a/webapp/src/Controller/Jury/UserController.php b/webapp/src/Controller/Jury/UserController.php
@@ -200,11 +200,10 @@ private function serializeJudgeTasks(array $judgeTasks, Judgehost $judgehost): a
                 }
             }
         }
-        dump($judgetaskids);
 
         $now = Utils::now();
         // We do need this, but for now it stops us from debugging.
-        /*$numUpdated = $this->em->getConnection()->executeStatement(
+        $numUpdated = $this->em->getConnection()->executeStatement(
             'UPDATE judgetask SET judgehostid = :judgehostid, starttime = :starttime WHERE starttime IS NULL AND valid = 1 AND judgetaskid IN (:ids)',
             [
                 'judgehostid' => $judgehost->getJudgehostid(),
@@ -216,11 +215,10 @@ private function serializeJudgeTasks(array $judgeTasks, Judgehost $judgehost): a
             ]
         );
 
-        dump($numUpdated);
         if ($numUpdated == 0) {
             // Bad luck, some other judgehost beat us to it.
             return [];
-        }*/
+        }
 
         // We got at least one, let's update the starttime of the corresponding judging if haven't done so in the past.
         $starttime_set = $this->em->getConnection()->executeStatement(

Original file line number	Diff line number	Diff line change
`@@ -200,11 +200,10 @@ private function serializeJudgeTasks(array $judgeTasks, Judgehost $judgehost): a`
`200`	`200`	`}`
`201`	`201`	`}`
`202`	`202`	`}`
`203`		`- dump($judgetaskids);`
`204`	`203`
`205`	`204`	`$now = Utils::now();`
`206`	`205`	`// We do need this, but for now it stops us from debugging.`
`207`		`- /*$numUpdated = $this->em->getConnection()->executeStatement(`
	`206`	`+ $numUpdated = $this->em->getConnection()->executeStatement(`
`208`	`207`	`'UPDATE judgetask SET judgehostid = :judgehostid, starttime = :starttime WHERE starttime IS NULL AND valid = 1 AND judgetaskid IN (:ids)',`
`209`	`208`	`[`
`210`	`209`	`'judgehostid' => $judgehost->getJudgehostid(),`
`@@ -216,11 +215,10 @@ private function serializeJudgeTasks(array $judgeTasks, Judgehost $judgehost): a`
`216`	`215`	`]`
`217`	`216`	`);`
`218`	`217`
`219`		`- dump($numUpdated);`
`220`	`218`	`if ($numUpdated == 0) {`
`221`	`219`	`// Bad luck, some other judgehost beat us to it.`
`222`	`220`	`return [];`
`223`		`- }*/`
	`221`	`+ }`
`224`	`222`
`225`	`223`	`// We got at least one, let's update the starttime of the corresponding judging if haven't done so in the past.`
`226`	`224`	`$starttime_set = $this->em->getConnection()->executeStatement(`