joshtronic/leaderbin
1
0
Fork 0
Code Issues 13 Pull requests Projects Releases Packages Wiki Activity Actions

Cleaned up response codes

Browse source 
Closes #21
This commit is contained in:
Josh Sherman 2013-12-21 19:07:07 -05:00
parent fa5ef9d858
commit a4a994dc19
3 changed files with 15 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

5
classes/APIv1.php
View file

@ -90,6 +90,11 @@ class APIv1 extends CustomModule
{
// Unsure why I had to put this here, I guess it's being overridden somewhere in PICKLES
header('Content-type: application/json');
if (isset($this->return['response_code']))
{
Browser::status($this->return['response_code']);
}
}
}

9
modules/api/v1/leaderboard.php
View file

@ -11,7 +11,7 @@ class api_v1_leaderboard extends APIv1
// Checks if we have a UID
if (!isset($_GET['uid']))
{
throw new Exception('Missing UID.');
throw new Exception('You must supply a leaderboard UID.');
}
$leaderboard_uid = $_GET['uid'];
@ -22,12 +22,12 @@ class api_v1_leaderboard extends APIv1
if (!$leaderboard)
{
throw new Exception('Leaderboard UID does not exist.');
throw new Exception('Leaderboard does not exist.');
}
if ($this->uid != $leaderboard['uid'])
{
throw new Exception('Leaderboard UID does not belong to you.');
throw new Exception('Leaderboard does not belong to you.');
}
// Sets up our key suffixes
@ -122,7 +122,8 @@ class api_v1_leaderboard extends APIv1
}
catch (Exception $e)
{
return array('response_code' => 400, 'error' => $e->getMessage());
$this->response_code = 400;
$this->error = $e->getMessage();
}
}
}

7
modules/api/v1/leaderboards.php
View file

@ -6,9 +6,12 @@ class api_v1_leaderboards extends APIv1
public function __default()
{
$leaderboards = new leaderboards();
if (!isset($this->return['error']))
{
$leaderboards = new leaderboards();
return $leaderboards->__default();
return $leaderboards->__default();
}
}
}