From 32c377a0c28ce13e7eaf0cf8b791d847131bf052 Mon Sep 17 00:00:00 2001 From: Sebastian Marcet Date: Tue, 22 Aug 2017 12:46:15 -0300 Subject: [PATCH] Fix on GetSummitById endpoint formerly GetSummitById endpoint was not honoring Summit field AvailableOnApi returning summit instead of returning 404. Change-Id: I667c0071d976237e98bc44cc93d03a1a6136e8e9 --- .../protected/summit/strategies/CurrentSummitFinderStrategy.php | 1 + 1 file changed, 1 insertion(+) diff --git a/app/Http/Controllers/apis/protected/summit/strategies/CurrentSummitFinderStrategy.php b/app/Http/Controllers/apis/protected/summit/strategies/CurrentSummitFinderStrategy.php index 667b36b5..7f67c26c 100644 --- a/app/Http/Controllers/apis/protected/summit/strategies/CurrentSummitFinderStrategy.php +++ b/app/Http/Controllers/apis/protected/summit/strategies/CurrentSummitFinderStrategy.php @@ -41,6 +41,7 @@ class CurrentSummitFinderStrategy implements ISummitFinderStrategy public function find($summit_id) { $summit = $summit_id === 'current' ? $this->repository->getCurrent() : $this->repository->getById(intval($summit_id)); + if(!$summit->isAvailableOnApi()) return null; return $summit; } } \ No newline at end of file