upstream/icingadb-web
1
0
Fork 0
mirror of https://github.com/Icinga/icingadb-web.git synced 2026-02-03 20:40:15 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

VolatileStateResults: Fetch individual redis results for the actual current item
Some checks are pending
L10n Update / update (push) Waiting to run
Details
CI / PHP (push) Waiting to run
Details

Browse source 
Previously, only the second and other subsequent item was updated with redis
results. Not sure why this wasn't discovered until recently, as the issue
was already part of the initial fix meant for a very similar issue, affecting
*all* items. Hell, maybe because my initial analysis this time also led me
into the wrong direction, so when reviewing this, proper testing and result
verification is key!

fixes #1318
This commit is contained in:
Johannes Meyer 2026-01-21 12:27:53 +01:00
parent 54765ed17a
commit f70f60bf66
1 changed files with 4 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
library/Icingadb/Redis/VolatileStateResults.php
View file

@ -84,6 +84,10 @@ class VolatileStateResults extends ResultSet
}
$result = parent::current();
if ($this->isCacheDisabled && ! $this->redisUnavailable) {
$this->applyRedisUpdates([$result]);
}
if (! $this->includeModelID) {
unset($result['id']);
}
@ -91,15 +95,6 @@ class VolatileStateResults extends ResultSet
return $result;
}
public function next(): void
{
parent::next();
if (! $this->redisUnavailable && $this->isCacheDisabled && $this->valid()) {
$this->applyRedisUpdates([parent::current()]);
}
}
public function key(): int
{
if (! $this->redisUnavailable && ! $this->updatesApplied && ! $this->isCacheDisabled) {