fix(federation): Don't send notifications for most system messages in federation

Signed-off-by: Joas Schilling <coding@schilljs.com>

1 files changed, 7 insertions, 1 deletions

diff --git a/lib/Federation/CloudFederationProviderTalk.php b/lib/Federation/CloudFederationProviderTalk.php
index 2ab2e71c2..c49a64c20 100644
--- a/lib/Federation/CloudFederationProviderTalk.php
+++ b/lib/Federation/CloudFederationProviderTalk.php
@@ -36,6 +36,7 @@ use OCA\Talk\Service\RoomService;
 use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\Http;
 use OCP\AppFramework\Services\IAppConfig;
+use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\DB\Exception as DBException;
 use OCP\EventDispatcher\IEventDispatcher;
 use OCP\Federation\Exceptions\ActionNotSupportedException;
@@ -79,6 +80,7 @@ class CloudFederationProviderTalk implements ICloudFederationProvider {
 		private ProxyCacheMessageService $pcmService,
 		private FederationChatNotifier $federationChatNotifier,
 		private UserConverter $userConverter,
+		private ITimeFactory $timeFactory,
 		ICacheFactory $cacheFactory,
 	) {
 		$this->proxyCacheMessages = $cacheFactory->isAvailable() ? $cacheFactory->createDistributed(CachePrefix::FEDERATED_PCM) : null;
@@ -385,7 +387,11 @@ class CloudFederationProviderTalk implements ICloudFederationProvider {
 				if ($notification['messageData']['remoteMessageId'] > $lastMessageId) {
 					$lastMessageId = (int) $notification['messageData']['remoteMessageId'];
 				}
-				$this->roomService->setLastMessageInfo($room, $lastMessageId, new \DateTime());
+
+				if ($notification['messageData']['systemMessage'] !== 'message_edited'
+					&& $notification['messageData']['systemMessage'] !== 'message_deleted') {
+					$this->roomService->setLastMessageInfo($room, $lastMessageId, $this->timeFactory->getDateTime());
+				}
 
 				if ($this->proxyCacheMessages instanceof ICache) {
 					$cacheKey = sha1(json_encode([$notification['remoteServerUrl'], $notification['remoteToken']]));