From 71e8faf8cd05b1f1f321edca6f30919f4a18bc90 Mon Sep 17 00:00:00 2001 From: Kalle Fagerberg Date: Sun, 17 Jul 2022 20:00:34 +0200 Subject: [PATCH] Smallfix errors --- lib/Controller/PersonalSettingsController.php | 22 +++++++++---------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/lib/Controller/PersonalSettingsController.php b/lib/Controller/PersonalSettingsController.php index 44de8b9..6ceccf3 100644 --- a/lib/Controller/PersonalSettingsController.php +++ b/lib/Controller/PersonalSettingsController.php @@ -58,10 +58,7 @@ class PersonalSettingsController extends Controller { $subStats = array(); foreach ($episodeActions as $ep) { $url = $ep->getPodcast(); - $stats = $subStats[$url] ?? [ - 'listenedSeconds' => 0, - 'actionCounts' => $this->defaultActionCounts(), - ]; + $stats = $subStats[$url] ?? $this->defaultSubscriptionData(); $actionCounts = $stats['actionCounts']; $actionLower = strtolower($ep->getAction()); if (array_key_exists($actionLower, $actionCounts)) { @@ -79,7 +76,7 @@ class PersonalSettingsController extends Controller { $subscriptions = array_map(function (SubscriptionChangeEntity $sub) use ($subStats) { $url = $sub->getUrl(); - $stats = $subStats[$url] ?? array(); + $stats = $subStats[$url] ?? $this->defaultSubscriptionData(); $sub = [ 'url' => $url ?? '', 'listenedSeconds' => $stats['listenedSeconds'], @@ -103,13 +100,16 @@ class PersonalSettingsController extends Controller { ]); } - private function defaultActionCounts(): array { + private function defaultSubscriptionData(): array { return [ - 'download' => 0, - 'delete' => 0, - 'play' => 0, - 'new' => 0, - 'flattr' => 0, + 'listenedSeconds' => 0, + 'actionCounts' => [ + 'download' => 0, + 'delete' => 0, + 'play' => 0, + 'new' => 0, + 'flattr' => 0, + ], ]; } }