Issue #2229181 by pwolanin, Berdir, fago: MapItem::propertyDefinitions()...

Issue #2229181 by pwolanin, Berdir, fago: MapItem::propertyDefinitions() claims it has a static value property, which is wrong.

core/lib/Drupal/Core/Field/Plugin/Field/FieldType/MapItem.php

@@ -27,10 +27,8 @@ class MapItem extends FieldItemBase {
    * {@inheritdoc}
    */
   public static function propertyDefinitions(FieldStorageDefinitionInterface $field_definition) {
-    $properties['value'] = DataDefinition::create('string')
-      ->setLabel(t('Serialized values'));
-
-    return $properties;
+    // The properties are dynamic and can not be defined statically.
+    return array();
   }
 
   /**
@@ -48,6 +46,15 @@ public static function schema(FieldStorageDefinitionInterface $field_definition)
     );
   }
 
+  /**
+   * {@inheritdoc}
+   */
+  public function toArray() {
+    // The default implementation of toArray() only returns known properties.
+    // For a map, return everything as the properties are not pre-defined.
+    return $this->getValue();
+  }
+
   /**
    * {@inheritdoc}
    */