patch #2

Open
Xefir wants to merge 3 commits from patch into main
Showing only changes of commit 496b5dfac0 - Show all commits

View File

@ -41,6 +41,7 @@ class EpisodeActionController extends Controller {
public function create(): JSONResponse { public function create(): JSONResponse {
$episodeActionsArray = $this->filterEpisodesFromRequestParams($this->request->getParams()); $episodeActionsArray = $this->filterEpisodesFromRequestParams($this->request->getParams());
$episodeActionsArray = $this->filterOnlyPlays($episodeActionsArray);
$this->episodeActionSaver->saveEpisodeActions($episodeActionsArray, $this->userId); $this->episodeActionSaver->saveEpisodeActions($episodeActionsArray, $this->userId);
return new JSONResponse(["timestamp" => time()]); return new JSONResponse(["timestamp" => time()]);
@ -74,4 +75,12 @@ class EpisodeActionController extends Controller {
public function filterEpisodesFromRequestParams(array $data): array { public function filterEpisodesFromRequestParams(array $data): array {
return array_filter($data, "is_numeric", ARRAY_FILTER_USE_KEY); return array_filter($data, "is_numeric", ARRAY_FILTER_USE_KEY);
} }
/**
* @param array $data
* @return array $episodeActionsArray
*/
private function filterOnlyPlays(array $data): array {
return array_filter($data, fn($ep) => isset($ep['action']) && strtolower($ep['action']) === 'play');
}
} }