openinfra/openstackid-resources
Code Issues Proposed changes

Fix on GetSummitById endpoint

Browse Source 
formerly GetSummitById endpoint was not honoring
Summit field AvailableOnApi returning summit instead
of returning 404.

Change-Id: I667c0071d976237e98bc44cc93d03a1a6136e8e9
This commit is contained in:
Sebastian Marcet 2017-08-22 12:46:15 -03:00
parent 0888309234
commit 32c377a0c2
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
app/Http/Controllers/apis/protected/summit/strategies/CurrentSummitFinderStrategy.php
View File

@ -41,6 +41,7 @@ class CurrentSummitFinderStrategy implements ISummitFinderStrategy
public function find($summit_id) public function find($summit_id)
{ {
$summit = $summit_id === 'current' ? $this->repository->getCurrent() : $this->repository->getById(intval($summit_id)); $summit = $summit_id === 'current' ? $this->repository->getCurrent() : $this->repository->getById(intval($summit_id));
if(!$summit->isAvailableOnApi()) return null;
return $summit; return $summit;
} }
} }