Fix the ability to goal the game if the goal location has been alread…

…y collected

* Always tells the game that the goal location is uncollected so the goal may still trigger

worlds/sc2/client.py

@@ -1375,13 +1375,10 @@ def get_uncollected_objectives(self) -> typing.List[int]:
             location % VICTORY_MODULO for location in
             self.ctx.uncollected_locations_in_mission(lookup_id_to_mission[self.mission_id])
         ]
-        if self.mission_id in self.ctx.final_mission_ids and not self.ctx.finished_game:
-            mission_locations = self.ctx.locations_for_mission_id(self.mission_id)
-            mission_objectives = [location % VICTORY_MODULO for location in mission_locations]
-            if 0 not in mission_objectives:
-                # Goal isn't a regular location and the game haven't been goaled yet.
-                # Mark victory as uncollected so the goal can be sent back by the game
-                result.append(0)
+        if self.mission_id in self.ctx.final_mission_ids and not self.ctx.finished_game and 0 not in result:
+            # Goal isn't a regular location or the goal location has been collected and the game haven't been goaled yet.
+            # Mark victory as uncollected so the goal can be sent back by the game
+            result.append(0)
         return result
 
     def missions_beaten_count(self):