Detect changed testcases based on hash

The page shows a warning
```The problem's testcases have changed since this judging has been performed. We recommend rejudging the whole problem.```
but this warning does not cover all possible changes in the testcases (e.g. changing input / output files does not trigger this warning).

Make this test more rigor by comparing the hashes.

Author: Kevinjil

webapp/src/Controller/Jury/SubmissionController.php

@@ -377,7 +377,7 @@ public function viewAction(
         $runsOutstanding = false;
         $runs       = [];
         $runsOutput = [];
-        $sameTestcaseIds = true;
+        $sameTestcaseHashes = true;
         if ($selectedJudging || $externalJudgement) {
             $queryBuilder = $this->em->createQueryBuilder()
                 ->from(Testcase::class, 't')
@@ -414,25 +414,25 @@ public function viewAction(
                 ->getQuery()
                 ->getResult();
 
-            $judgingRunTestcaseIdsInOrder = $this->em->createQueryBuilder()
+            $judgingRunTestcaseHashesInOrder = $this->em->createQueryBuilder()
                 ->from(JudgeTask::class, 'jt')
-                ->select('jt.testcase_id')
+                ->select('jt.testcase_hash')
                 ->andWhere('jt.jobid = :judging')
                 ->setParameter('judging', $selectedJudging)
                 ->orderBy('jt.judgetaskid')
                 ->getQuery()
                 ->getScalarResult();
 
             $cnt = 0;
-            if (count($judgingRunTestcaseIdsInOrder) !== count($runResults)) {
-                $sameTestcaseIds = false;
+            if (count($judgingRunTestcaseHashesInOrder) !== count($runResults)) {
+                $sameTestcaseHashes = false;
             }
             foreach ($runResults as $runResult) {
                 /** @var Testcase $testcase */
                 $testcase = $runResult[0];
-                if (isset($judgingRunTestcaseIdsInOrder[$cnt])) {
-                    if ($testcase->getTestcaseid() != $judgingRunTestcaseIdsInOrder[$cnt]['testcase_id']) {
-                        $sameTestcaseIds = false;
+                if (isset($judgingRunTestcaseHashesInOrder[$cnt])) {
+                    if ($testcase->getTestcasehash() != $judgingRunTestcaseHashesInOrder[$cnt]['testcase_hash']) {
+                        $sameTestcaseHashes = false;
                     }
                 }
                 $cnt++;
@@ -571,7 +571,7 @@ public function viewAction(
             'runs' => $runs,
             'runsOutstanding' => $runsOutstanding,
             'judgehosts' => $judgehosts,
-            'sameTestcaseIds' => $sameTestcaseIds,
+            'sameTestcaseHashes' => $sameTestcaseHashes,
             'externalRuns' => $externalRuns,
             'runsOutput' => $runsOutput,
             'lastRuns' => $lastRuns,
@@ -710,7 +710,7 @@ public function requestOutput(Request $request, Judging $jid, JudgingRun $jrid):
             ->setJobId($jid->getJudgingid())
             ->setUuid($jid->getUuid())
             ->setTestcaseId($testcase->getTestcaseid())
-            ->setTestcaseHash($testcase->getMd5sumInput() . '_' . $testcase->getMd5sumOutput());
+            ->setTestcaseHash($testcase->getTestcaseHash());
         $this->em->persist($judgeTask);
         $this->em->flush();
         return $this->redirectToLocalReferrer($this->router, $request, $this->generateUrl('jury_submission', [

webapp/templates/jury/submission.html.twig

@@ -279,7 +279,7 @@
         </div>
     {% endif %}
 
-    {% if not sameTestcaseIds and selectedJudging is not null and selectedJudging.result is not empty %}
+    {% if not sameTestcaseHashes and selectedJudging is not null and selectedJudging.result is not empty %}
         <div class="alert alert-danger">The problem's testcases have changed since this judging has been performed. We recommend rejudging the whole problem.</div>
     {% endif %}