public/static/js/game.js

@@ -21,6 +21,7 @@ const GameType = Object.freeze({ 'SINGLE': 0, 'MULTI': 1, 'CHALLENGE': 2 });
         googleLink: null,
         history: [],
         restrictions: null,
+        finishers: null,
 
         readyToContinue: true,
         timeoutEnd: null,
@@ -487,6 +488,10 @@ const GameType = Object.freeze({ 'SINGLE': 0, 'MULTI': 1, 'CHALLENGE': 2 });
                     }
                 }
             }
+
+            if (response.finishers) {
+                Game.finishers = new Set(response.finishers);
+            }
         },
 
         reset: function () {
@@ -539,6 +544,7 @@ const GameType = Object.freeze({ 'SINGLE': 0, 'MULTI': 1, 'CHALLENGE': 2 });
             document.getElementById('panningBlockerCover').style.display = null;
 
             Game.history = [];
+            Game.finishers = null;
 
             Game.initialize();
         },
@@ -867,21 +873,16 @@ const GameType = Object.freeze({ 'SINGLE': 0, 'MULTI': 1, 'CHALLENGE': 2 });
             var highscores = new Map();
             highscores.set('me', Game.scoreSum);
 
-            // collect the results of users who are through the last round
+            for (var i = 0; i < Game.history.length; ++i) {
-            const round = Game.history[Game.history.length - 1];
-            if (round.allResults) {
-                for (const result of round.allResults) {
-                    highscores.set(result.userName, result.score);
-                }
-            }
-
-            // add up scores only for the finishers
-            for (var i = Game.history.length - 2; i >= 0; --i) {
                 const round = Game.history[i];
                 if (round.allResults) {
                     for (const result of round.allResults) {
-                        if (highscores.has(result.userName)) {
+                        if (Game.finishers.has(result.userName)) {
-                            highscores.set(result.userName, highscores.get(result.userName) + result.score);
+                            if (highscores.has(result.userName)) {
+                                highscores.set(result.userName, highscores.get(result.userName) + result.score);
+                            } else {
+                                highscores.set(result.userName, result.score);
+                            }
                         }
                     }
                 }

src/Controller/GameFlowController.php

@@ -199,6 +199,14 @@ class GameFlowController implements ISecured
 
         if (!isset($currentPlace)) { // game finished
             $response['finished'] = true;
+
+            // list all players who finished the challenge
+            $response['finishers'] = [];
+            foreach ($this->userInChallengeRepository->getAllByChallengeWithUsers($challenge) as $userInChallenge) {
+                if ($userInChallenge->getCurrentRound() == $currentRound && $userInChallenge->getUser()->getId() != $userId) {
+                    $response['finishers'][] = $userInChallenge->getUser()->getDisplayName();
+                }
+            }
         } else { // continue game
             $response['place'] = [
                 'panoId' => $currentPlace->getPanoIdCached(),

views/maps.php

@@ -67,7 +67,7 @@ TODO: condition!
             </div>
             <div>
                 <input type="checkbox" id="noZoom" name="noZoom" value="noZoom" />
-                <label for="noZoom">No zoom allowed</label>
+                <label for="noMove">No zoom allowed</label>
             </div>
             <div>
                 <input type="checkbox" id="noPan" name="noPan" value="noPan" />