Skip to content

Commit 28c010c

Browse files
author
Evgeniy Guseletov
committed
Merge pull request #78 from KnpLabs/feature/rawurlencodefix
rawurlencode instead of urlencode, fixes #77
2 parents 3787a1b + 9f53ced commit 28c010c

37 files changed

+198
-172
lines changed

lib/Github/Api/Authorizations.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ public function all()
1818

1919
public function show($number)
2020
{
21-
return $this->get('authorizations/'.urlencode($number));
21+
return $this->get('authorizations/'.rawurlencode($number));
2222
}
2323

2424
public function create(array $params)
@@ -28,16 +28,16 @@ public function create(array $params)
2828

2929
public function update($id, array $params)
3030
{
31-
return $this->patch('authorizations/'.urlencode($id), $params);
31+
return $this->patch('authorizations/'.rawurlencode($id), $params);
3232
}
3333

3434
public function remove($id)
3535
{
36-
return $this->delete('authorizations/'.urlencode($id));
36+
return $this->delete('authorizations/'.rawurlencode($id));
3737
}
3838

3939
public function check($id, $token)
4040
{
41-
return $this->get('authorizations/'.urlencode($id).'/tokens/'.urlencode($token));
41+
return $this->get('authorizations/'.rawurlencode($id).'/tokens/'.rawurlencode($token));
4242
}
4343
}

lib/Github/Api/CurrentUser/DeployKeys.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ public function all()
3131
*/
3232
public function show($id)
3333
{
34-
return $this->get('user/keys/'.urlencode($id));
34+
return $this->get('user/keys/'.rawurlencode($id));
3535
}
3636

3737
/**
@@ -68,7 +68,7 @@ public function update($id, array $params)
6868
throw new MissingArgumentException(array('title', 'key'));
6969
}
7070

71-
return $this->patch('user/keys/'.urlencode($id), $params);
71+
return $this->patch('user/keys/'.rawurlencode($id), $params);
7272
}
7373

7474
/**
@@ -80,6 +80,6 @@ public function update($id, array $params)
8080
*/
8181
public function remove($id)
8282
{
83-
return $this->delete('user/keys/'.urlencode($id));
83+
return $this->delete('user/keys/'.rawurlencode($id));
8484
}
8585
}

lib/Github/Api/CurrentUser/Followers.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public function all($page = 1)
3333
*/
3434
public function check($username)
3535
{
36-
return $this->get('user/following/'.urlencode($username));
36+
return $this->get('user/following/'.rawurlencode($username));
3737
}
3838

3939
/**
@@ -45,7 +45,7 @@ public function check($username)
4545
*/
4646
public function follow($username)
4747
{
48-
return $this->put('user/following/'.urlencode($username));
48+
return $this->put('user/following/'.rawurlencode($username));
4949
}
5050

5151
/**
@@ -57,6 +57,6 @@ public function follow($username)
5757
*/
5858
public function unfollow($username)
5959
{
60-
return $this->delete('user/following/'.urlencode($username));
60+
return $this->delete('user/following/'.rawurlencode($username));
6161
}
6262
}

lib/Github/Api/CurrentUser/Notifications.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public function all(array $params = array())
3333
*/
3434
public function allInRepository($username, $repository, array $params = array())
3535
{
36-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/notifications', $params);
36+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/notifications', $params);
3737
}
3838

3939
/**
@@ -57,7 +57,7 @@ public function markAsReadAll(array $params = array())
5757
*/
5858
public function markAsReadInRepository($username, $repository, array $params = array())
5959
{
60-
return $this->put('repos/'.urlencode($username).'/'.urlencode($repository).'/notifications', $params);
60+
return $this->put('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/notifications', $params);
6161
}
6262

6363
/**
@@ -69,7 +69,7 @@ public function markAsReadInRepository($username, $repository, array $params = a
6969
*/
7070
public function markAsRead($id, array $params)
7171
{
72-
return $this->patch('notifications/threads/'.urlencode($id), $params);
72+
return $this->patch('notifications/threads/'.rawurlencode($id), $params);
7373
}
7474

7575
/**
@@ -80,7 +80,7 @@ public function markAsRead($id, array $params)
8080
*/
8181
public function show($id)
8282
{
83-
return $this->get('notifications/threads/'.urlencode($id));
83+
return $this->get('notifications/threads/'.rawurlencode($id));
8484
}
8585

8686
/**
@@ -91,7 +91,7 @@ public function show($id)
9191
*/
9292
public function showSubscription($id)
9393
{
94-
return $this->get('notifications/threads/'.urlencode($id).'/subscription');
94+
return $this->get('notifications/threads/'.rawurlencode($id).'/subscription');
9595
}
9696

9797
/**
@@ -103,7 +103,7 @@ public function showSubscription($id)
103103
*/
104104
public function createSubscription($id, array $params)
105105
{
106-
return $this->put('notifications/threads/'.urlencode($id).'/subscription', $params);
106+
return $this->put('notifications/threads/'.rawurlencode($id).'/subscription', $params);
107107
}
108108

109109
/**
@@ -114,6 +114,6 @@ public function createSubscription($id, array $params)
114114
*/
115115
public function removeSubscription($id)
116116
{
117-
return $this->delete('notifications/threads/'.urlencode($id).'/subscription');
117+
return $this->delete('notifications/threads/'.rawurlencode($id).'/subscription');
118118
}
119119
}

lib/Github/Api/CurrentUser/Watchers.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public function all($page = 1)
3434
*/
3535
public function check($username, $repository)
3636
{
37-
return $this->get('user/watched/'.urlencode($username).'/'.urlencode($repository));
37+
return $this->get('user/watched/'.rawurlencode($username).'/'.rawurlencode($repository));
3838
}
3939

4040
/**
@@ -47,7 +47,7 @@ public function check($username, $repository)
4747
*/
4848
public function watch($username, $repository)
4949
{
50-
return $this->put('user/watched/'.urlencode($username).'/'.urlencode($repository));
50+
return $this->put('user/watched/'.rawurlencode($username).'/'.rawurlencode($repository));
5151
}
5252

5353
/**
@@ -60,6 +60,6 @@ public function watch($username, $repository)
6060
*/
6161
public function unwatch($username, $repository)
6262
{
63-
return $this->delete('user/watched/'.urlencode($username).'/'.urlencode($repository));
63+
return $this->delete('user/watched/'.rawurlencode($username).'/'.rawurlencode($repository));
6464
}
6565
}

lib/Github/Api/Gists.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,12 @@ public function all($type = null)
2020
return $this->get('gists');
2121
}
2222

23-
return $this->get('gists/'.urlencode($type));
23+
return $this->get('gists/'.rawurlencode($type));
2424
}
2525

2626
public function show($number)
2727
{
28-
return $this->get('gists/'.urlencode($number));
28+
return $this->get('gists/'.rawurlencode($number));
2929
}
3030

3131
public function create(array $params)
@@ -41,31 +41,31 @@ public function create(array $params)
4141

4242
public function update($id, array $params)
4343
{
44-
return $this->patch('gists/'.urlencode($id), $params);
44+
return $this->patch('gists/'.rawurlencode($id), $params);
4545
}
4646

4747
public function fork($id)
4848
{
49-
return $this->post('gists/'.urlencode($id).'/fork');
49+
return $this->post('gists/'.rawurlencode($id).'/fork');
5050
}
5151

5252
public function remove($id)
5353
{
54-
return $this->delete('gists/'.urlencode($id));
54+
return $this->delete('gists/'.rawurlencode($id));
5555
}
5656

5757
public function check($id)
5858
{
59-
return $this->get('gists/'.urlencode($id).'/star');
59+
return $this->get('gists/'.rawurlencode($id).'/star');
6060
}
6161

6262
public function star($id)
6363
{
64-
return $this->put('gists/'.urlencode($id).'/star');
64+
return $this->put('gists/'.rawurlencode($id).'/star');
6565
}
6666

6767
public function unstar($id)
6868
{
69-
return $this->delete('gists/'.urlencode($id).'/star');
69+
return $this->delete('gists/'.rawurlencode($id).'/star');
7070
}
7171
}

lib/Github/Api/GitData/Blobs.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public function configure($bodyType = null)
2222

2323
public function show($username, $repository, $sha)
2424
{
25-
$response = $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/blobs/'.urlencode($sha));
25+
$response = $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/blobs/'.rawurlencode($sha));
2626

2727
return $response;
2828
}
@@ -33,6 +33,6 @@ public function create($username, $repository, array $params)
3333
throw new MissingArgumentException(array('content', 'encoding'));
3434
}
3535

36-
return $this->post('repos/'.urlencode($username).'/'.urlencode($repository).'/git/blobs', $params);
36+
return $this->post('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/blobs', $params);
3737
}
3838
}

lib/Github/Api/GitData/Commits.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ class Commits extends AbstractApi
1313
{
1414
public function show($username, $repository, $sha)
1515
{
16-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/commits/'.urlencode($sha));
16+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/commits/'.rawurlencode($sha));
1717
}
1818

1919
public function create($username, $repository, array $params)
@@ -22,6 +22,6 @@ public function create($username, $repository, array $params)
2222
throw new MissingArgumentException(array('message', 'tree', 'parents'));
2323
}
2424

25-
return $this->post('repos/'.urlencode($username).'/'.urlencode($repository).'/git/commits', $params);
25+
return $this->post('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/commits', $params);
2626
}
2727
}

lib/Github/Api/GitData/References.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,22 +13,22 @@ class References extends AbstractApi
1313
{
1414
public function all($username, $repository)
1515
{
16-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs');
16+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs');
1717
}
1818

1919
public function branches($username, $repository)
2020
{
21-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs/heads');
21+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs/heads');
2222
}
2323

2424
public function tags($username, $repository)
2525
{
26-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs/tags');
26+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs/tags');
2727
}
2828

2929
public function show($username, $repository, $reference)
3030
{
31-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs/'.urlencode($reference));
31+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs/'.rawurlencode($reference));
3232
}
3333

3434
public function create($username, $repository, array $params)
@@ -37,7 +37,7 @@ public function create($username, $repository, array $params)
3737
throw new MissingArgumentException(array('ref', 'sha'));
3838
}
3939

40-
return $this->post('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs', $params);
40+
return $this->post('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs', $params);
4141
}
4242

4343
public function update($username, $repository, $reference, array $params)
@@ -46,11 +46,11 @@ public function update($username, $repository, $reference, array $params)
4646
throw new MissingArgumentException('sha');
4747
}
4848

49-
return $this->patch('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs/'.urlencode($reference), $params);
49+
return $this->patch('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs/'.rawurlencode($reference), $params);
5050
}
5151

5252
public function remove($username, $repository, $reference)
5353
{
54-
return $this->delete('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs/'.urlencode($reference));
54+
return $this->delete('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs/'.rawurlencode($reference));
5555
}
5656
}

lib/Github/Api/GitData/Tags.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,12 @@ class Tags extends AbstractApi
1313
{
1414
public function all($username, $repository)
1515
{
16-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/refs/tags');
16+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/refs/tags');
1717
}
1818

1919
public function show($username, $repository, $sha)
2020
{
21-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/tags/'.urlencode($sha));
21+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/tags/'.rawurlencode($sha));
2222
}
2323

2424
public function create($username, $repository, array $params)
@@ -35,6 +35,6 @@ public function create($username, $repository, array $params)
3535
throw new MissingArgumentException(array('tagger.name', 'tagger.email', 'tagger.date'));
3636
}
3737

38-
return $this->post('repos/'.urlencode($username).'/'.urlencode($repository).'/git/tags', $params);
38+
return $this->post('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/tags', $params);
3939
}
4040
}

lib/Github/Api/GitData/Trees.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ class Trees extends AbstractApi
1313
{
1414
public function show($username, $repository, $sha, $recursive = false)
1515
{
16-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/git/trees/'.urlencode($sha), array('recursive' => $recursive ? 1 : null));
16+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/trees/'.rawurlencode($sha), array('recursive' => $recursive ? 1 : null));
1717
}
1818

1919
public function create($username, $repository, array $params)
@@ -37,6 +37,6 @@ public function create($username, $repository, array $params)
3737
}
3838
}
3939

40-
return $this->post('repos/'.urlencode($username).'/'.urlencode($repository).'/git/trees', $params);
40+
return $this->post('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/git/trees', $params);
4141
}
4242
}

lib/Github/Api/Issue.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class Issue extends AbstractApi
2828
*/
2929
public function all($username, $repository, array $params = array())
3030
{
31-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/issues', array_merge(array('page' => 1), $params));
31+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/issues', array_merge(array('page' => 1), $params));
3232
}
3333

3434
/**
@@ -48,7 +48,7 @@ public function find($username, $repository, $state, $keyword)
4848
$state = 'open';
4949
}
5050

51-
return $this->get('legacy/issues/search/'.urlencode($username).'/'.urlencode($repository).'/'.urlencode($state).'/'.urlencode($keyword));
51+
return $this->get('legacy/issues/search/'.rawurlencode($username).'/'.rawurlencode($repository).'/'.rawurlencode($state).'/'.rawurlencode($keyword));
5252
}
5353

5454
/**
@@ -66,7 +66,7 @@ public function org($organization, $state, array $params = array())
6666
$state = 'open';
6767
}
6868

69-
return $this->get('orgs/'.urlencode($organization).'/issues', array_merge(array('page' => 1, 'state' => $state), $params));
69+
return $this->get('orgs/'.rawurlencode($organization).'/issues', array_merge(array('page' => 1, 'state' => $state), $params));
7070
}
7171

7272
/**
@@ -80,7 +80,7 @@ public function org($organization, $state, array $params = array())
8080
*/
8181
public function show($username, $repository, $id)
8282
{
83-
return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/issues/'.urlencode($id));
83+
return $this->get('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/issues/'.rawurlencode($id));
8484
}
8585

8686
/**
@@ -101,7 +101,7 @@ public function create($username, $repository, array $params)
101101
throw new MissingArgumentException(array('title', 'body'));
102102
}
103103

104-
return $this->post('repos/'.urlencode($username).'/'.urlencode($repository).'/issues', $params);
104+
return $this->post('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/issues', $params);
105105
}
106106

107107
/**
@@ -117,7 +117,7 @@ public function create($username, $repository, array $params)
117117
*/
118118
public function update($username, $repository, $id, array $params)
119119
{
120-
return $this->patch('repos/'.urlencode($username).'/'.urlencode($repository).'/issues/'.urlencode($id), $params);
120+
return $this->patch('repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/issues/'.rawurlencode($id), $params);
121121
}
122122

123123
/**

0 commit comments

Comments
 (0)