Merge pull request #116 from Gizra/115-clone-entity

Clone the entity prior to adding to the queue.
Gizra · May 7, 2018 · 64f093b · 64f093b
2 parents ea91975 + b6f2ab4
commit 64f093b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Subscribers.php b/src/Subscribers.php
@@ -177,7 +177,9 @@ public function sendMessage(EntityInterface $entity, MessageInterface $message,
       // Add item to the queue.
       $task = [
         'message' => $message,
-        'entity' => $entity,
+        // Clone the entity first to avoid any oddness with serialization.
+        // @see https://www.drupal.org/project/drupal/issues/2971157
+        'entity' => clone $entity,
         'notify_options' => $notify_options,
         'subscribe_options' => $subscribe_options,
         'context' => $context,

diff --git a/tests/src/Kernel/SubscribersTest.php b/tests/src/Kernel/SubscribersTest.php
@@ -242,7 +242,7 @@ public function testEntityAccess() {
     $message = Message::create(['template' => $this->template->id()]);
 
     $node = $this->nodes[0];
-    $node->setPublished(FALSE);
+    $node->setUnpublished();
     $node->save();
 
     // Add permission to view own unpublished content.