skobkin/point-tools
1
1
Fork 0
Code Issues 4 Pull requests 1 Releases Activity

urlencode() for user logins in API requests.

Browse source
This commit is contained in:
Alexey Skobkin 2015-10-05 04:10:07 +03:00
parent 295b089616
commit 4bd570e910
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/Skobkin/Bundle/PointToolsBundle/Service/UserApi.php
View file

@ -63,7 +63,7 @@ class UserApi extends AbstractApi
public function getUserSubscribersByLogin($login) public function getUserSubscribersByLogin($login)
{ {
try { try {
$usersList = $this->getGetRequestData('/api/user/'.$login.'/subscribers', [], true); $usersList = $this->getGetRequestData('/api/user/'.urlencode($login).'/subscribers', [], true);
} catch (ClientErrorResponseException $e) { } catch (ClientErrorResponseException $e) {
if (Response::HTTP_NOT_FOUND === $e->getResponse()->getStatusCode()) { if (Response::HTTP_NOT_FOUND === $e->getResponse()->getStatusCode()) {
throw new UserNotFoundException('User not found', 0, $e, null, $login); throw new UserNotFoundException('User not found', 0, $e, null, $login);
@ -115,7 +115,7 @@ class UserApi extends AbstractApi
public function getUserSubscriptionsByLogin($login) public function getUserSubscriptionsByLogin($login)
{ {
try { try {
$usersList = $this->getGetRequestData('/api/user/'.$login.'/subscriptions', [], true); $usersList = $this->getGetRequestData('/api/user/'.urlencode($login).'/subscriptions', [], true);
} catch (ClientErrorResponseException $e) { } catch (ClientErrorResponseException $e) {
if (Response::HTTP_NOT_FOUND === $e->getResponse()->getStatusCode()) { if (Response::HTTP_NOT_FOUND === $e->getResponse()->getStatusCode()) {
throw new UserNotFoundException('User not found', 0, $e, null, $login); throw new UserNotFoundException('User not found', 0, $e, null, $login);
@ -166,7 +166,7 @@ class UserApi extends AbstractApi
public function getUserByLogin($login) public function getUserByLogin($login)
{ {
try { try {
$userInfo = $this->getGetRequestData('/api/user/login/'.$login, [], true); $userInfo = $this->getGetRequestData('/api/user/login/'.urlencode($login), [], true);
} catch (ClientErrorResponseException $e) { } catch (ClientErrorResponseException $e) {
if (Response::HTTP_NOT_FOUND === $e->getResponse()->getStatusCode()) { if (Response::HTTP_NOT_FOUND === $e->getResponse()->getStatusCode()) {
throw new UserNotFoundException('User not found', 0, $e, null, $login); throw new UserNotFoundException('User not found', 0, $e, null, $login);
@ -306,6 +306,6 @@ class UserApi extends AbstractApi
throw new \InvalidArgumentException('Avatar size must be one of restricted variants. See UserApi class AVATAR_SIZE_* constants.'); throw new \InvalidArgumentException('Avatar size must be one of restricted variants. See UserApi class AVATAR_SIZE_* constants.');
} }
return $this->avatarsBaseUrl.$user->getLogin().'/'.$size; return $this->avatarsBaseUrl.urlencode($user->getLogin()).'/'.$size;
} }
} }