php-cs-fixer

php.ini

@@ -1,4 +1,4 @@
 upload_max_filesize = 40M
 post_max_size = 40M
-memory_limit = 256M
+memory_limit = 512M
 

src/Controller/BadgeController.php

@@ -23,7 +23,7 @@ class BadgeController extends AbstractController
         }
 
         $stats = $taskLifecycleManager->getProjectStats($project);
-        if (!isset($stats[$status])) {
+        if (empty($stats) || !isset($stats[$status])) {
             throw new NotFoundHttpException('Status not found');
         }
 

src/Controller/MapController.php

@@ -14,7 +14,7 @@ class MapController extends AbstractController
     #[Route('/popup', name: 'app_map_popup')]
     public function popup(Request $request): Response
     {
-        // Grosso modo, on récuère les données de la feature correspondante
+        // Grosso modo, on récupère les données de la feature correspondante
         // Cf `assets/controllers/map_controller.js`
         $element = json_decode($request->query->get('element'), true);
 
@@ -26,7 +26,7 @@ class MapController extends AbstractController
 
         // TODO dans le cas d’un nœud il n’y a pas de bounds mais on doit
         // pouvoir trouver les coins d’une bbox autour sans trop de difficultés
-        if (isset($element['bounds'])) {
+        if (isset($element['bounds']) and isset($element['type'])) {
             $josmCommands['load_and_zoom'] = [
                 'bottom' => $element['bounds']['minlat'],
                 'top' => $element['bounds']['maxlat'],

src/Controller/TaskController.php

@@ -110,7 +110,7 @@ class TaskController extends AbstractController
         $josmCommands = [
             // Charge l’imagerie (fond OSM par défaut)
             'imagery' => [
-                'id' => $project->hasImagery() ? $project->getImagery() : 'osmfr',
+                (($project->hasImagery() and 'http' === substr($project->getImagery(), 0, 4)) ? 'url' : 'id') => ($project->hasImagery() ? $project->getImagery() : 'osmfr'),
             ],
         ];
 

src/Entity/Task.php

@@ -166,7 +166,11 @@ class Task
 
     public function hasGeojson(): bool
     {
-        return isset($this->geojson);
+        return
+            isset($this->geojson)
+            and !empty($this->geojson)
+            and ('{}' !== $this->geojson)
+        ;
     }
 
     public function getGeojson(): ?string

src/Form/CsvType.php

@@ -17,7 +17,7 @@ class CsvType extends AbstractType
                 'label' => 'Fichier CSV',
                 'constraints' => [
                     new File([
-                        'maxSize' => '10M',
+                        'maxSize' => '40M',
                         'mimeTypes' => [
                             'text/csv',
                             'text/plain',

src/Form/TaskType.php

@@ -30,6 +30,7 @@ class TaskType extends AbstractType
                 'required' => false,
                 'help_html' => true,
                 'help' => 'XML décrivant ce qu’il faut charger dans la feuille de données de JOSM (cf <a href="https://wiki.openstreetmap.org/wiki/FR:OSM_XML" target="_blank">FR:OSM XML - OpenStreetMap Wiki</a>)',
+                'empty_data' => '',
             ])
             ->add('status', TaskLifecycleType::class, [
                 'label' => 'État',

src/Service/OverpassClient.php

@@ -12,7 +12,7 @@ class OverpassClient
     ) {
     }
 
-    public function query($query, $prefix = '[out:json][timeout:25];', $suffix = 'out geom;')
+    public function query($query, $prefix = '[out:json][timeout:60];', $suffix = 'out geom;')
     {
         $response = $this->client->request('GET', 'https://overpass-api.de/api/interpreter', [
             'query' => [

src/Service/TaskLifecycleManager.php

@@ -36,6 +36,7 @@ class TaskLifecycleManager
         foreach ($places as $place => $id) {
             $stats[$place] = array_merge($metas->getPlaceMetadata($place), [
                 'value' => 0,
+                'percentage' => 0,
             ]);
         }