Provide a ConfigEntityNormalizer [#2191903]

Updated: Comment #N

Problem/Motivation

We can normalize/denormalize content entities, why not config entities too? They are a massive part of Drupal 8 architecture. It seems very reasonable people will want to serialize/unserialize this stuff with the serializer in web services. For such a small patch, it seems better to provide this in core, rather than have x contrib modules providing a similar thing all over the place.

Proposed resolution

Extend the EntityNormalizer class and override the normalize() method to call getExportProperties() - DONE.

Remaining tasks

Patch, review.

User interface changes

None

API changes

None

Comment	File	Size	Author
#4	interdiff-2191903-4.txt	1.38 KB	damiankloip
#4	2191903-4.patch	3.41 KB	damiankloip
#4
	D8.serialization.ConfigEntityNormalizer.patch	3.33 KB	damiankloip

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

damiankloip CreditAttribution: damiankloip commented 8 February 2014 at 23:55

Title:

Provide an ConfigEntityNormalizer

» Provide a ConfigEntityNormalizer

Hmm, poor english there..

Comment #2

aspilicious CreditAttribution: aspilicious commented 9 February 2014 at 01:06

I agree this should be part of core

Comment #3

dawehner

German

CreditAttribution: dawehner commented 10 February 2014 at 20:29

+++ b/core/modules/serialization/tests/Drupal/serialization/Tests/Normalizer/ConfigNormalizerTest.php
@@ -0,0 +1,48 @@
+ * @see \Drupal\serialization\Normalizer\ConfigEntityNormalizer
+ */
+class ConfigEntityNormalizerTest extends UnitTestCase {

see the other issue ...

+++ b/core/modules/serialization/tests/Drupal/serialization/Tests/Normalizer/ConfigNormalizerTest.php
@@ -0,0 +1,48 @@
+  public static function getInfo() {

inheritdoc

+++ b/core/modules/serialization/tests/Drupal/serialization/Tests/Normalizer/ConfigNormalizerTest.php
@@ -0,0 +1,48 @@
+    $this->assertsame($test_export_properties, $normalizer->normalize($config_entity));

assertsame => assertSame