Merge pull request #2309 from nextcloud/feat/clear-previews-last-preview

feat(ClearPreviews): Clear last_preview column in addition to cache

Author: marcelklehr

lib/Command/ClearPreviews.php

@@ -8,20 +8,18 @@
 
 namespace OCA\Bookmarks\Command;
 
+use OCA\Bookmarks\Db\BookmarkMapper;
 use OCA\Bookmarks\Service\FileCache;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ClearPreviews extends Command {
-	/**
-	 * @var FileCache
-	 */
-	private $fileCache;
-
-	public function __construct(FileCache $fileCache) {
+	public function __construct(
+		private FileCache $fileCache,
+		private BookmarkMapper $bookmarkMapper,
+	) {
 		parent::__construct();
-		$this->fileCache = $fileCache;
 	}
 
 	/**
@@ -45,6 +43,7 @@ protected function configure() {
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		try {
 			$this->fileCache->clear();
+			$this->bookmarkMapper->clearLastPreviews();
 		} catch (\Exception $ex) {
 			$output->writeln('<error>Failed to clear previews</error>');
 			$output->writeln($ex->getMessage());

lib/Db/BookmarkMapper.php

@@ -724,12 +724,24 @@ public function findPendingPreviews(int $limit, int $stalePeriod): array {
 		return $this->findEntities($qb);
 	}
 
+	/**
+	 * @return void
+	 * @throws Exception
+	 */
+	public function clearLastPreviews(): void {
+		$qb = $this->db->getQueryBuilder();
+		$qb->update('bookmarks');
+		$qb->set('last_preview', $qb->createNamedParameter(0));
+		$qb->executeStatement();
+	}
+
 	/**
 	 * @psalm-param Bookmark $entity
 	 * @param Entity $entity
 	 *
 	 * @return Bookmark
 	 * @psalm-return Bookmark
+	 * @throws Exception
 	 */
 	public function delete(Entity $entity): Bookmark {
 		$this->eventDispatcher->dispatch(