Move public string 'name' into Node

All of Component, Parameter and Property have 'name'.

Moving it into Node helps phpstan to understand that 'name' always exists.

lib/Component.php

@@ -20,17 +20,12 @@
  */
 class Component extends Node
 {
-    /**
-     * Component name.
-     *
-     * This will contain a string such as VEVENT, VTODO, VCALENDAR, VCARD.
-     */
-    public string $name;
-
     /**
      * A list of properties and/or sub-components.
      *
-     * @var array<string, Component<int, mixed>|Property<int, mixed>>
+     * ToDo: maybe be more specific array<string, Component<int, mixed>|Property<int, mixed>>
+     *
+     * @var array<string, array<int, mixed>>
      */
     protected array $children = [];
 

lib/Node.php

@@ -54,6 +54,15 @@ abstract class Node implements \IteratorAggregate, \ArrayAccess, \Countable, \Js
      */
     protected ?Component $root;
 
+    /**
+     * Name of the item (a Component or Parameter or Property).
+     *
+     * This will contain a component name such as VEVENT, VTODO, VCALENDAR, VCARD.
+     * Or a parameter name (vCard 2.1 allows parameters to be encoded without a name)
+     * Or a property name such as DTSTART, SUMMARY, FN.
+     */
+    public ?string $name;
+
     /**
      * Serializes the node into a mimedir format.
      */

lib/Parameter.php

@@ -18,11 +18,6 @@
  */
 class Parameter extends Node
 {
-    /**
-     * Parameter name.
-     */
-    public string $name;
-
     /**
      * vCard 2.1 allows parameters to be encoded without a name.
      *

lib/Property.php

@@ -21,13 +21,6 @@ abstract class Property extends Node
      */
     public ?Component $root;
 
-    /**
-     * Property name.
-     *
-     * This will contain a string such as DTSTART, SUMMARY, FN.
-     */
-    public ?string $name;
-
     /**
      * Property group.
      *