Permalink
Browse files

MQ: important change: do not use the node->user as notification targe…

…tId, as it is now always set, and thus breaks the cross-users messaging.
  • Loading branch information...
1 parent 8d51378 commit 7766fb4ccc565fe414f3159c56803427c6de0ff4 @cdujeu cdujeu committed Sep 24, 2016
Showing with 4 additions and 2 deletions.
  1. +4 −2 core/src/plugins/core.mq/src/MqManager.php
@@ -170,7 +170,8 @@ public function publishNodeChange($origNode = null, $newNode = null, $copy = fal
$ctx = null;
if ($newNode != null) {
$ctx = $newNode->getContext();
- $targetUserId = $newNode->getUserId();
+ //$targetUserId = $newNode->getUserId();
+ $targetUserId = null;
$nodePaths[] = $newNode->getPath();
$update = false;
$data = array();
@@ -185,7 +186,8 @@ public function publishNodeChange($origNode = null, $newNode = null, $copy = fal
if ($origNode != null && ! $update && !$copy) {
$ctx = $origNode->getContext();
- $targetUserId = $origNode->getUserId();
+ //$targetUserId = $origNode->getUserId();
+ $targetUserId = null;
$nodePaths[] = $origNode->getPath();
$content = XMLWriter::writeNodesDiff(array("REMOVE" => array($origNode->getPath())));

0 comments on commit 7766fb4

Please sign in to comment.