openinfra
/
openstackid-resources
Code Issues Proposed changes

Added get my speaker endpoint 

GET /api/v1/speakers/me

Change-Id: I342ae1c47b68f9f9f505ecfd539eaf5c646faeaf
This commit is contained in:
Sebastian Marcet 2018-06-20 08:18:00 -07:00
parent 3e0424d54d
commit 23f49897cd
4 changed files with 80 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

43
app/Http/Controllers/Apis/Protected/Summit/OAuth2SummitSpeakersApiController.php
View File

@ -313,6 +313,49 @@ final class OAuth2SummitSpeakersApiController extends OAuth2ProtectedController
}
}
public function getMySpeaker(){
try
{
$current_member_id = $this->resource_server_context->getCurrentUserExternalId();
if(is_null($current_member_id))
return $this->error403();
$member = $this->member_repository->getById($current_member_id);
if(is_null($member))
return $this->error403();
$speaker = $this->speaker_repository->getByMember($member);
if (is_null($speaker)) return $this->error404();
$serializer_type = $this->serializer_type_selector->getSerializerType();
return $this->ok
(
SerializerRegistry::getInstance()->getSerializer($speaker, $serializer_type)->serialize
(
Request::input('expand', ''),
[],
[],
[]
)
);
}
catch(ValidationException $ex1){
Log::warning($ex1);
return $this->error412($ex1->getMessages());
}
catch(EntityNotFoundException $ex2)
{
Log::warning($ex2);
return $this->error404(array('message'=> $ex2->getMessage()));
}
catch (Exception $ex) {
Log::error($ex);
return $this->error500($ex);
}
}
/**
* @param $speaker_id
* @return mixed

1
app/Http/routes.php
View File

@ -515,6 +515,7 @@ Route::group([
Route::get('', 'OAuth2SummitSpeakersApiController@getAll');
Route::post('', [ 'middleware' => 'auth.user:administrators|summit-front-end-administrators', 'uses' => 'OAuth2SummitSpeakersApiController@addSpeaker']);
Route::put('merge/{speaker_from_id}/{speaker_to_id}', 'OAuth2SummitSpeakersApiController@merge');
Route::get('me', 'OAuth2SummitSpeakersApiController@getMySpeaker');
Route::group(['prefix' => '{speaker_id}'], function () {
Route::put('',[ 'middleware' => 'auth.user:administrators|summit-front-end-administrators', 'uses' => 'OAuth2SummitSpeakersApiController@updateSpeaker'])->where('speaker_id', 'me|[0-9]+');
Route::delete('',[ 'middleware' => 'auth.user:administrators|summit-front-end-administrators', 'uses' => 'OAuth2SummitSpeakersApiController@deleteSpeaker'])->where('speaker_id', 'me|[0-9]+');

9
database/seeds/ApiEndpointsSeeder.php
View File

@ -311,6 +311,15 @@ class ApiEndpointsSeeder extends Seeder
sprintf(SummitScopes::ReadAllSummitData, $current_realm)
],
),
array(
'name' => 'get-my-speaker',
'route' => '/api/v1/speakers/me',
'http_method' => 'GET',
'scopes' => [
sprintf(SummitScopes::ReadSummitData, $current_realm),
sprintf(SummitScopes::ReadAllSummitData, $current_realm)
],
),
array(
'name' => 'merge-speakers',
'route' => '/api/v1/speakers/merge/{speaker_from_id}/{speaker_to_id}',

28
tests/OAuth2SpeakersApiTest.php
View File

@ -58,7 +58,7 @@ final class OAuth2SpeakersApiTest extends ProtectedApiTest
$headers = [
"HTTP_Authorization" => " Bearer " . $this->access_token,
"CONTENT_TYPE" => "application/json"
"CONTENT_TYPE" => "application/json"
];
$data = [
@ -420,6 +420,32 @@ final class OAuth2SpeakersApiTest extends ProtectedApiTest
$this->assertTrue(!is_null($speaker));
}
public function testGetMySpeaker(){
$params = [
];
$headers = [
"HTTP_Authorization" => " Bearer " . $this->access_token,
"CONTENT_TYPE" => "application/json"
];
$response = $this->action(
"GET",
"OAuth2SummitSpeakersApiController@getMySpeaker",
$params,
[],
[],
[],
$headers
);
$content = $response->getContent();
$this->assertResponseStatus(200);
$speaker = json_decode($content);
$this->assertTrue(!is_null($speaker));
}
public function testMergeSpeakers(){
$params = [