From 1ee1fcce91d64c8cee6b8621aaedbdeae0c9c330 Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Tue, 2 Feb 2021 00:03:59 +0100 Subject: INSERT ... ON CONFLIT DO NOTHING for addEntry (#3409) #fix https://github.com/FreshRSS/FreshRSS/issues/3402 Explicit `INSERT OR IGNORE` / `ON CONFLICT DO NOTHING` for the `addEntry()` method, which does expect some duplicates. --- app/Models/EntryDAO.php | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'app/Models/EntryDAO.php') diff --git a/app/Models/EntryDAO.php b/app/Models/EntryDAO.php index 28a9b8bde..6f7a4b48f 100644 --- a/app/Models/EntryDAO.php +++ b/app/Models/EntryDAO.php @@ -18,6 +18,10 @@ class FreshRSS_EntryDAO extends Minz_ModelPdo implements FreshRSS_Searchable { return 'hex(' . $x . ')'; } + public function sqlIgnoreConflict($sql) { + return str_replace('INSERT INTO ', 'INSERT IGNORE INTO ', $sql); + } + //TODO: Move the database auto-updates to DatabaseDAO protected function createEntryTempTable() { $ok = false; @@ -83,14 +87,15 @@ SQL; public function addEntry($valuesTmp, $useTmpTable = true) { if ($this->addEntryPrepared == null) { - $sql = 'INSERT INTO `_' . ($useTmpTable ? 'entrytmp' : 'entry') . '` (id, guid, title, author, ' + $sql = $this->sqlIgnoreConflict( + 'INSERT INTO `_' . ($useTmpTable ? 'entrytmp' : 'entry') . '` (id, guid, title, author, ' . ($this->isCompressed() ? 'content_bin' : 'content') . ', link, date, `lastSeen`, hash, is_read, is_favorite, id_feed, tags) ' . 'VALUES(:id, :guid, :title, :author, ' . ($this->isCompressed() ? 'COMPRESS(:content)' : ':content') . ', :link, :date, :last_seen, ' . $this->sqlHexDecode(':hash') - . ', :is_read, :is_favorite, :id_feed, :tags)'; + . ', :is_read, :is_favorite, :id_feed, :tags)'); $this->addEntryPrepared = $this->pdo->prepare($sql); } if ($this->addEntryPrepared) { -- cgit v1.2.3