Merge pull request #4166 from nextcloud/fix/set-logs-to-processed

Fix setting log entries to processed on sent notification

Author: dartcafe

CHANGELOG.md

@@ -5,13 +5,14 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
-## [Unreleased]
+## [8.1.4] - 2025-07-15
 ### Fixes
  - Fixed some typos
  - Removed inaccessible polls from polls overview
  - Removed link target from inaccessible polls in navigation
  - Removed clone action from inaccessible polls in navigation
  - Fixed visual bug when scrolling in list view
+ - Fixed exception on notifications which may cause resending notification mails
 
 ### changes
  - Center poll table

lib/Db/LogMapper.php

@@ -9,6 +9,7 @@
 namespace OCA\Polls\Db;
 
 use OCP\AppFramework\Db\QBMapper;
+use OCP\DB\QueryBuilder\IQueryBuilder;
 use OCP\IDBConnection;
 
 /**
@@ -31,7 +32,8 @@ public function findUnprocessed(): array {
 
 		$qb->select('*')
 			->from($this->getTableName())
-			->where($qb->expr()->isNotNull('poll_id'));
+			->where($qb->expr()->isNotNull('poll_id'))
+			->andWhere($qb->expr()->eq('processed', $qb->expr()->literal(0, IQueryBuilder::PARAM_INT)));
 		return $this->findEntities($qb);
 	}
 

lib/Service/MailService.php

@@ -172,8 +172,9 @@ public function sendNotifications(): void {
 		}
 
 		foreach ($this->logs as $logItem) {
-			$logItem->setProcessed(time());
+			$logItem->setProcessed(intval(microtime(true) * 1000));
 			$this->logMapper->update($logItem);
+			usleep(5000);
 		}
 	}