From bd0fb1da9901d0c91b1a68a8a50046a434440caf Mon Sep 17 00:00:00 2001 From: sysadminstory Date: Sun, 3 Nov 2024 18:28:32 +0100 Subject: [PATCH] [IdealoBridge] Fix (#4316) When a product was available before as used product in the past, and now it's not available used anymore, a price update article was generated on every feed loading, because the old used price was still stored in the cache, and therefore different as "no price". The issue was also present in the cas of a New product price that becomes unavailable. Now, when either there is no New or Used price available, the previous price is delete from the cache. --- bridges/IdealoBridge.php | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/bridges/IdealoBridge.php b/bridges/IdealoBridge.php index f426a45c..7432f78a 100644 --- a/bridges/IdealoBridge.php +++ b/bridges/IdealoBridge.php @@ -152,14 +152,21 @@ class IdealoBridge extends BridgeAbstract $PriceNew = $FirstButton->find('strong', 0)->plaintext; // Save current price $this->saveCacheValue($KeyNEW, $PriceNew); + } else if ($FirstButton === null) { + // In case there is no actual New Price delete the previous value in the cache + $this->cache->delete($this->getShortName() . '_' . $KeyNEW); } + // Second Button contains the used product price $SecondButton = $html->find('.oopStage-conditionButton-wrapper-text', 1); if ($SecondButton) { $PriceUsed = $SecondButton->find('strong', 0)->plaintext; // Save current price $this->saveCacheValue($KeyUSED, $PriceUsed); + } else if ($SecondButton === null) { + // In case there is no actual Used Price delete the previous value in the cache + $this->cache->delete($this->getShortName() . '_' . $KeyUSED); } // Only continue if a price has changed