fix: 🐛 fix crash if no cache is configured
Some checks failed
repod / xml (push) Successful in 1m10s
repod / php (push) Failing after 44s
repod / nodejs (push) Successful in 1m25s
repod / release (push) Has been skipped

This commit is contained in:
Michel Roux 2024-06-14 19:32:26 +02:00
parent bc8db22fce
commit 42bfdb530b

View File

@ -10,12 +10,14 @@ use OCA\RePod\AppInfo\Application;
use OCP\AppFramework\Controller;
use OCP\AppFramework\Http\JSONResponse;
use OCP\Http\Client\IClientService;
use OCP\ICacheFactory;
use OCP\IRequest;
class PodcastController extends Controller
{
public function __construct(
IRequest $request,
private ICacheFactory $cacheFactory,
private IClientService $clientService,
private PodcastDataReader $podcastDataReader
) {
@ -27,11 +29,13 @@ class PodcastController extends Controller
* @NoCSRFRequired
*/
public function index(string $url): JSONResponse {
if ($this->cacheFactory->isLocalCacheAvailable()) {
try {
$podcast = $this->podcastDataReader->tryGetCachedPodcastData($url);
} catch (\Exception $e) {
$podcast = null;
}
}
if ($podcast) {
return new JSONResponse($podcast);
@ -41,10 +45,12 @@ class PodcastController extends Controller
$feed = $client->get($url);
$podcast = PodcastData::parseRssXml((string) $feed->getBody());
if ($this->cacheFactory->isLocalCacheAvailable()) {
try {
$this->podcastDataReader->trySetCachedPodcastData($url, $podcast);
} catch (\Exception $e) {
}
}
return new JSONResponse($podcast, $feed->getStatusCode());
}