diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2021-05-18 22:50:51 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2021-05-18 22:50:51 +0200 |
| commit | 8d8b792007acd7c49e49ecc4e512ff594436656d (patch) | |
| tree | fc3286b50391670c3c48ac77586da74389388048 /app/Models/EntryDAO.php | |
| parent | ec10dfc2f3bd81e512c02000cb6cd5587052cb82 (diff) | |
Fix array_chunk key index bug (#3631)
* Fix array_chunk key index bug
#fix https://github.com/FreshRSS/FreshRSS/issues/3630
We still want array entries indexed sequentially from zero https://php.net/function.array-chunk
* One missing
Diffstat (limited to 'app/Models/EntryDAO.php')
| -rw-r--r-- | app/Models/EntryDAO.php | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/app/Models/EntryDAO.php b/app/Models/EntryDAO.php index a95b9a422..046dc4afc 100644 --- a/app/Models/EntryDAO.php +++ b/app/Models/EntryDAO.php @@ -265,7 +265,7 @@ SQL; if (count($ids) > FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER) { // Split a query with too many variables parameters $affected = 0; - $idsChunks = array_chunk($ids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER, true); + $idsChunks = array_chunk($ids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER); foreach ($idsChunks as $idsChunk) { $affected += $this->markFavorite($idsChunk, $is_favorite); } @@ -356,7 +356,7 @@ SQL; } elseif (count($ids) > FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER) { // Split a query with too many variables parameters $affected = 0; - $idsChunks = array_chunk($ids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER, true); + $idsChunks = array_chunk($ids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER); foreach ($idsChunks as $idsChunk) { $affected += $this->markRead($idsChunk, $is_read); } @@ -981,7 +981,7 @@ SQL; yield false; } elseif (count($ids) > FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER) { // Split a query with too many variables parameters - $idsChunks = array_chunk($ids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER, true); + $idsChunks = array_chunk($ids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER); foreach ($idsChunks as $idsChunk) { foreach ($this->listByIds($idsChunk, $order) as $entry) { yield $entry; @@ -1020,7 +1020,7 @@ SQL; return $result; } elseif (count($guids) > FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER) { // Split a query with too many variables parameters - $guidsChunks = array_chunk($guids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER, true); + $guidsChunks = array_chunk($guids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER); foreach ($guidsChunks as $guidsChunk) { $result += $this->listHashForFeedGuids($id_feed, $guidsChunk); } @@ -1055,7 +1055,7 @@ SQL; } elseif (count($guids) > FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER) { // Split a query with too many variables parameters $affected = 0; - $guidsChunks = array_chunk($guids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER, true); + $guidsChunks = array_chunk($guids, FreshRSS_DatabaseDAO::MAX_VARIABLE_NUMBER); foreach ($guidsChunks as $guidsChunk) { $affected += $this->updateLastSeen($id_feed, $guidsChunk, $mtime); } |