#25 fix: RSS endpoint again

Signed-off-by: Patrizio Bekerle <patrizio@bekerle.com>

Author: pbek

src/Controller/AppReleaseRSSFeedController.php

@@ -3,9 +3,9 @@
 namespace App\Controller;
 
 use App\Entity\AppRelease;
-use App\Service\ReleaseApi;
 use Doctrine\Persistence\ManagerRegistry;
 use Michelf\Markdown;
+use Psr\Cache\CacheItemPoolInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Routing\Attribute\Route;
@@ -20,11 +20,11 @@ class AppReleaseRSSFeedController extends AbstractController
      * http://api.qownnotes.org/rss/app-releases
      */
     #[Route('/rss/app-releases')]
-    public function appReleases(ManagerRegistry $doctrine): Response
+    public function appReleases(ManagerRegistry $doctrine, CacheItemPoolInterface $cache): Response
     {
-        $cacheDriver = ReleaseApi::getCacheDriver();
+        $cacheItem = $cache->getItem(self::CACHE_KEY);
 
-        if (!$cacheDriver->contains(self::CACHE_KEY)) {
+        if (!$cacheItem->isHit()) {
             $projectUrl = 'https://www.qownnotes.org';
 
             $xml = new \DOMDocument('1.0', 'UTF-8'); // Create new DOM document.
@@ -117,9 +117,11 @@ public function appReleases(ManagerRegistry $doctrine): Response
 
             $xmlString = $xml->saveXML();
 
-            $cacheDriver->save(self::CACHE_KEY, $xmlString, 60);
+            $cacheItem->set($xmlString);
+            $cacheItem->expiresAfter(60);
+            $cache->save($cacheItem);
         } else {
-            $xmlString = $cacheDriver->fetch(self::CACHE_KEY);
+            $xmlString = $cacheItem->get();
         }
 
         return new Response($xmlString, 200, ['Content-Type' => 'text/xml']);

src/Service/ReleaseApi.php

@@ -9,7 +9,6 @@
 
 use App\Entity\AppRelease;
 use App\Entity\LatestRelease;
-use Doctrine\Common\Cache\PhpFileCache;
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\ORM\EntityManagerInterface;
 use GuzzleHttp\Client;
@@ -500,14 +499,4 @@ public function getEnv(string $varName, $default = '')
 
         return false === $value ? ($_ENV[$varName] ?? $default) : $value;
     }
-
-    /**
-     * @return PhpFileCache
-     */
-    public static function getCacheDriver()
-    {
-        return new PhpFileCache(
-            '/tmp'
-        );
-    }
 }