[Map] Minor adjustements

Author: Kocal

src/Map/CHANGELOG.md

@@ -7,8 +7,7 @@
 -  Add `DistanceCalculatorInterface` interface and three implementations:
    `HaversineDistanceCalculator`, `SphericalCosineDistanceCalculator` and `VincentyDistanceCalculator`.
 -  Add `CoordinateUtils` helper, to convert decimal coordinates (`43.2109`) in DMS (`56° 78' 90"`)
-
--  Add property `$id` to `Marker`, `Polygon` and `Polyline` constructors
+-  Add parameter `id` to `Marker`, `Polygon` and `Polyline` constructors
 -  Add method `Map::removeMarker(string|Marker $markerOrId)`
 -  Add method `Map::removePolygon(string|Polygon $polygonOrId)`
 -  Add method `Map::removePolyline(string|Polyline $polylineOrId)`

src/Map/src/Element.php

@@ -11,6 +11,11 @@
 
 namespace Symfony\UX\Map;
 
+/**
+ * @author Sylvain Blondeau <[email protected]>
+ *
+ * @internal
+ */
 interface Element
 {
 }

src/Map/src/Elements.php

@@ -15,10 +15,12 @@
  * Represents a collection of map elements.
  *
  * @author Sylvain Blondeau <[email protected]>
+ *
+ * @internal
  */
 abstract class Elements
 {
-    protected \SplObjectStorage $elements;
+    private \SplObjectStorage $elements;
 
     public function __construct(
         array $elements,
@@ -29,7 +31,7 @@ public function __construct(
         }
     }
 
-    public function add(Element $element): self
+    public function add(Element $element): static
     {
         $this->elements->attach($element, $element->id ?? $this->elements->getHash($element));
 
@@ -47,7 +49,7 @@ private function getElement(string $id): ?Element
         return null;
     }
 
-    public function remove(Element|string $elementOrId): self
+    public function remove(Element|string $elementOrId): static
     {
         if (\is_string($elementOrId)) {
             $elementOrId = $this->getElement($elementOrId);

src/Map/src/Map.php

@@ -26,8 +26,8 @@ final class Map
 
     /**
      * @param Marker[]   $markers
+     * @param Polygon[]  $polygons
      * @param Polyline[] $polylines
-     * @param Polygone[] $polygons
      */
     public function __construct(
         private readonly ?string $rendererName = null,
@@ -40,8 +40,8 @@ public function __construct(
         array $polylines = [],
     ) {
         $this->markers = new Markers($markers);
-        $this->polylines = new Polylines($polylines);
         $this->polygons = new Polygons($polygons);
+        $this->polylines = new Polylines($polylines);
     }
 
     public function getRendererName(): ?string

src/Map/src/Markers.php

@@ -15,6 +15,8 @@
  * Represents a Marker collection.
  *
  * @author Sylvain Blondeau <[email protected]>
+ *
+ * @internal
  */
 final class Markers extends Elements
 {

src/Map/src/Polygons.php

@@ -15,6 +15,8 @@
  * Represents a Polygon collection.
  *
  * @author Sylvain Blondeau <[email protected]>
+ *
+ * @internal
  */
 final class Polygons extends Elements
 {

src/Map/src/Polylines.php

@@ -15,6 +15,8 @@
  * Represents a Polyline collection.
  *
  * @author Sylvain Blondeau <[email protected]>
+ *
+ * @internal
  */
 final class Polylines extends Elements
 {