@@ -16,7 +16,7 @@ discard block |
||
16 | 16 | * @param array $options |
17 | 17 | * @return bool |
18 | 18 | */ |
19 | - public function addPlaylistTracks($playlistId, array $tracks, array $options = []) : bool; |
|
19 | + public function addPlaylistTracks($playlistId, array $tracks, array $options = [ ]) : bool; |
|
20 | 20 | |
21 | 21 | /** |
22 | 22 | * @param string $playlistId |
@@ -30,14 +30,14 @@ discard block |
||
30 | 30 | * @param array $options |
31 | 31 | * @return PlaylistDataProvider |
32 | 32 | */ |
33 | - public function getPlaylist(string $playlistId, array $options = []) : PlaylistDataProvider; |
|
33 | + public function getPlaylist(string $playlistId, array $options = [ ]) : PlaylistDataProvider; |
|
34 | 34 | |
35 | 35 | /** |
36 | 36 | * @param $playlistId |
37 | 37 | * @param array $options |
38 | 38 | * @return object |
39 | 39 | */ |
40 | - public function getPlaylistTracks(string $playlistId, array $options = []); |
|
40 | + public function getPlaylistTracks(string $playlistId, array $options = [ ]); |
|
41 | 41 | |
42 | 42 | /** |
43 | 43 | * @param string $userId |
@@ -46,12 +46,12 @@ discard block |
||
46 | 46 | * @return PlaylistDataProvider |
47 | 47 | * @throws PlaylistNotFound |
48 | 48 | */ |
49 | - public function getUserPlaylistByName(string $userId, string $playlistName, array $options = []): PlaylistDataProvider; |
|
49 | + public function getUserPlaylistByName(string $userId, string $playlistName, array $options = [ ]): PlaylistDataProvider; |
|
50 | 50 | |
51 | 51 | /** |
52 | 52 | * @param TrackSearchRequestDataProvider $trackSearchRequest |
53 | 53 | * @param array $options |
54 | 54 | * @return TracksSearchDataProvider |
55 | 55 | */ |
56 | - public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = []): TracksSearchDataProvider; |
|
56 | + public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = [ ]): TracksSearchDataProvider; |
|
57 | 57 | } |
58 | 58 | \ No newline at end of file |
@@ -36,7 +36,7 @@ discard block |
||
36 | 36 | * @param array $options |
37 | 37 | * @return bool |
38 | 38 | */ |
39 | - public function addPlaylistTracks($playlistId, array $tracks, array $options = []): bool |
|
39 | + public function addPlaylistTracks($playlistId, array $tracks, array $options = [ ]): bool |
|
40 | 40 | { |
41 | 41 | return $this->baseSpotifyWebAPI->addPlaylistTracks($playlistId, $tracks, $options); |
42 | 42 | } |
@@ -48,18 +48,18 @@ discard block |
||
48 | 48 | */ |
49 | 49 | public function deletePlaylistTracks(string $playlistId, array $tracksInfo): bool |
50 | 50 | { |
51 | - $tracksToDelete = []; |
|
51 | + $tracksToDelete = [ ]; |
|
52 | 52 | foreach ($tracksInfo as $deleteTrackInfoDataProvider) { |
53 | 53 | if (!$deleteTrackInfoDataProvider instanceof DeleteTrackInfoDataProvider) { |
54 | 54 | throw new \RuntimeException( |
55 | 55 | sprintf('tracksInfo should be instanceof class: %s', DeleteTrackInfoDataProvider::class) |
56 | 56 | ); |
57 | 57 | } |
58 | - $tracksToDelete[] = $deleteTrackInfoDataProvider->toArray(); |
|
58 | + $tracksToDelete[ ] = $deleteTrackInfoDataProvider->toArray(); |
|
59 | 59 | } |
60 | 60 | |
61 | 61 | $this->baseSpotifyWebAPI->setReturnType(Request::RETURN_OBJECT); |
62 | - $delete = (bool)$this->baseSpotifyWebAPI->deletePlaylistTracks($playlistId, $tracksToDelete); |
|
62 | + $delete = (bool) $this->baseSpotifyWebAPI->deletePlaylistTracks($playlistId, $tracksToDelete); |
|
63 | 63 | $this->baseSpotifyWebAPI->setReturnType(Request::RETURN_ASSOC); |
64 | 64 | |
65 | 65 | return $delete; |
@@ -70,7 +70,7 @@ discard block |
||
70 | 70 | * @param array $options |
71 | 71 | * @return PlaylistDataProvider |
72 | 72 | */ |
73 | - public function getPlaylist(string $playlistId, array $options = []): PlaylistDataProvider |
|
73 | + public function getPlaylist(string $playlistId, array $options = [ ]): PlaylistDataProvider |
|
74 | 74 | { |
75 | 75 | $jsonObjectResult = $this->baseSpotifyWebAPI->getPlaylist($playlistId, $options); |
76 | 76 | |
@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | * @return PlaylistDataProvider |
88 | 88 | * @throws PlaylistNotFound |
89 | 89 | */ |
90 | - public function getUserPlaylistByName(string $userId, string $playlistName, array $options = []): PlaylistDataProvider |
|
90 | + public function getUserPlaylistByName(string $userId, string $playlistName, array $options = [ ]): PlaylistDataProvider |
|
91 | 91 | { |
92 | 92 | $userPlaylistsDataProvider = $this->getUserPlaylists($userId, $options); |
93 | 93 | $playlists = $userPlaylistsDataProvider->getItems(); |
@@ -105,7 +105,7 @@ discard block |
||
105 | 105 | * @param array $options |
106 | 106 | * @return PlaylistTracksDataProvider |
107 | 107 | */ |
108 | - public function getPlaylistTracks(string $playlistId, array $options = []): PlaylistTracksDataProvider |
|
108 | + public function getPlaylistTracks(string $playlistId, array $options = [ ]): PlaylistTracksDataProvider |
|
109 | 109 | { |
110 | 110 | $jsonObjectResult = $this->baseSpotifyWebAPI->getPlaylistTracks($playlistId, $options); |
111 | 111 | |
@@ -120,7 +120,7 @@ discard block |
||
120 | 120 | * @param array $options |
121 | 121 | * @return TracksSearchDataProvider |
122 | 122 | */ |
123 | - public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = []): TracksSearchDataProvider |
|
123 | + public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = [ ]): TracksSearchDataProvider |
|
124 | 124 | { |
125 | 125 | $jsonObjectResult = $this->search( |
126 | 126 | sprintf( |
@@ -134,7 +134,7 @@ discard block |
||
134 | 134 | ); |
135 | 135 | |
136 | 136 | $tracksSearchDataProvider = new TracksSearchDataProvider(); |
137 | - $tracksSearchDataProvider->fromArray($jsonObjectResult[$trackSearchRequest->getResultType()]); |
|
137 | + $tracksSearchDataProvider->fromArray($jsonObjectResult[ $trackSearchRequest->getResultType() ]); |
|
138 | 138 | |
139 | 139 | return $tracksSearchDataProvider; |
140 | 140 | } |
@@ -145,9 +145,9 @@ discard block |
||
145 | 145 | * @param array $options |
146 | 146 | * @return array |
147 | 147 | */ |
148 | - private function search(string $query, array $type, array $options = []): array |
|
148 | + private function search(string $query, array $type, array $options = [ ]): array |
|
149 | 149 | { |
150 | - return (array)$this->baseSpotifyWebAPI->search($query, $type, $options); |
|
150 | + return (array) $this->baseSpotifyWebAPI->search($query, $type, $options); |
|
151 | 151 | } |
152 | 152 | |
153 | 153 | /** |
@@ -155,9 +155,9 @@ discard block |
||
155 | 155 | * @param array $options |
156 | 156 | * @return UserPlaylistsDataProvider |
157 | 157 | */ |
158 | - private function getUserPlaylists(string $userId, array $options = []): UserPlaylistsDataProvider |
|
158 | + private function getUserPlaylists(string $userId, array $options = [ ]): UserPlaylistsDataProvider |
|
159 | 159 | { |
160 | - $userPlaylistInfo = (array)$this->baseSpotifyWebAPI->getUserPlaylists($userId, $options); |
|
160 | + $userPlaylistInfo = (array) $this->baseSpotifyWebAPI->getUserPlaylists($userId, $options); |
|
161 | 161 | |
162 | 162 | $userPlaylistsDataProvider = new UserPlaylistsDataProvider(); |
163 | 163 | $userPlaylistsDataProvider->fromArray($userPlaylistInfo); |
@@ -62,10 +62,10 @@ discard block |
||
62 | 62 | public function testGetUserPlaylists(): void |
63 | 63 | { |
64 | 64 | $spotifyPlaylist = $this->spotifyWebApi->getUserPlaylistByName( |
65 | - static::spotifyInfo['user'], |
|
66 | - static::spotifyInfo['playlistName'] |
|
65 | + static::spotifyInfo[ 'user' ], |
|
66 | + static::spotifyInfo[ 'playlistName' ] |
|
67 | 67 | ); |
68 | - $this->assertSame(static::spotifyInfo['playlistId'], $spotifyPlaylist->getId()); |
|
68 | + $this->assertSame(static::spotifyInfo[ 'playlistId' ], $spotifyPlaylist->getId()); |
|
69 | 69 | } |
70 | 70 | |
71 | 71 | public function testGetUserPlaylistsNotFoundUser(): void |
@@ -77,18 +77,18 @@ discard block |
||
77 | 77 | public function testGetUserPlaylistsNotFoundPlayList(): void |
78 | 78 | { |
79 | 79 | $this->expectException(PlaylistNotFound::class); |
80 | - $this->spotifyWebApi->getUserPlaylistByName(static::spotifyInfo['user'], 'uniTest-Not_FOUND-Playlist'); |
|
80 | + $this->spotifyWebApi->getUserPlaylistByName(static::spotifyInfo[ 'user' ], 'uniTest-Not_FOUND-Playlist'); |
|
81 | 81 | } |
82 | 82 | |
83 | 83 | public function testGetPlaylist(): void |
84 | 84 | { |
85 | 85 | $spotifyPlayList = $this->spotifyWebApi->getPlaylist( |
86 | - static::spotifyInfo['playlistId'] |
|
86 | + static::spotifyInfo[ 'playlistId' ] |
|
87 | 87 | ); |
88 | - $this->assertSame(static::spotifyInfo['playlistId'], $spotifyPlayList->getId()); |
|
89 | - $this->assertSame(static::spotifyInfo['playlistName'], $spotifyPlayList->getName()); |
|
88 | + $this->assertSame(static::spotifyInfo[ 'playlistId' ], $spotifyPlayList->getId()); |
|
89 | + $this->assertSame(static::spotifyInfo[ 'playlistName' ], $spotifyPlayList->getName()); |
|
90 | 90 | $this->assertTrue($spotifyPlayList->getPublic()); |
91 | - $this->assertSame('U2', $spotifyPlayList->getTracks()->getItems()[0]->getTrack()->getArtists()[0]->getName()); |
|
91 | + $this->assertSame('U2', $spotifyPlayList->getTracks()->getItems()[ 0 ]->getTrack()->getArtists()[ 0 ]->getName()); |
|
92 | 92 | } |
93 | 93 | |
94 | 94 | public function testGetNoExistUserPlaylist(): void |
@@ -111,18 +111,18 @@ discard block |
||
111 | 111 | { |
112 | 112 | $trackSearchRequestDataProvider = new TrackSearchRequestDataProvider(); |
113 | 113 | $trackSearchRequestDataProvider->setArtist( |
114 | - static::spotifySong['artist'] |
|
114 | + static::spotifySong[ 'artist' ] |
|
115 | 115 | ); |
116 | 116 | $trackSearchRequestDataProvider->setTrack( |
117 | - static::spotifySong['track'] |
|
117 | + static::spotifySong[ 'track' ] |
|
118 | 118 | ); |
119 | 119 | $searchResult = $this->spotifyWebApi->searchTrack($trackSearchRequestDataProvider); |
120 | 120 | |
121 | 121 | $this->assertGreaterThan(0, $searchResult->getItems()); |
122 | 122 | $this->assertSame( |
123 | - strtolower(static::spotifySong['track']), |
|
123 | + strtolower(static::spotifySong[ 'track' ]), |
|
124 | 124 | strtolower( |
125 | - $searchResult->getItems()[0]->getName() |
|
125 | + $searchResult->getItems()[ 0 ]->getName() |
|
126 | 126 | ) |
127 | 127 | ); |
128 | 128 | } |
@@ -144,27 +144,27 @@ discard block |
||
144 | 144 | public function testAddPlaylistTracks() |
145 | 145 | { |
146 | 146 | $addResult = $this->spotifyWebApi->addPlaylistTracks( |
147 | - static::spotifyInfo['playlistId'], |
|
147 | + static::spotifyInfo[ 'playlistId' ], |
|
148 | 148 | [ |
149 | - static::spotifySong['trackId'] |
|
149 | + static::spotifySong[ 'trackId' ] |
|
150 | 150 | ] |
151 | 151 | ); |
152 | 152 | $this->assertTrue($addResult); |
153 | 153 | |
154 | 154 | $spotifyPlayList = $this->spotifyWebApi->getPlaylistTracks( |
155 | - static::spotifyInfo['playlistId'] |
|
155 | + static::spotifyInfo[ 'playlistId' ] |
|
156 | 156 | ); |
157 | 157 | |
158 | 158 | $this->assertNotEmpty($spotifyPlayList->getItems()); |
159 | 159 | $this->assertNotEmpty($spotifyPlayList->getTotal()); |
160 | - $this->assertSame(static::spotifySong['trackId'], $spotifyPlayList->getItems()[1]->getTrack()->getId()); |
|
160 | + $this->assertSame(static::spotifySong[ 'trackId' ], $spotifyPlayList->getItems()[ 1 ]->getTrack()->getId()); |
|
161 | 161 | } |
162 | 162 | |
163 | 163 | public function testAddPlaylistTracksNotExistTrackId() |
164 | 164 | { |
165 | 165 | try { |
166 | 166 | $this->spotifyWebApi->addPlaylistTracks( |
167 | - static::spotifyInfo['playlistId'], |
|
167 | + static::spotifyInfo[ 'playlistId' ], |
|
168 | 168 | [ |
169 | 169 | 'uniTest-Not_FOUND--ID' |
170 | 170 | ] |
@@ -183,24 +183,24 @@ discard block |
||
183 | 183 | public function testDeleteUserPlaylistTracks() |
184 | 184 | { |
185 | 185 | $spotifyPlayList = $this->spotifyWebApi->getPlaylistTracks( |
186 | - static::spotifyInfo['playlistId'] |
|
186 | + static::spotifyInfo[ 'playlistId' ] |
|
187 | 187 | ); |
188 | 188 | |
189 | 189 | $this->assertCount(2, $spotifyPlayList->getItems()); |
190 | 190 | $this->assertSame(2, $spotifyPlayList->getTotal()); |
191 | 191 | |
192 | 192 | $deleteTrack = new DeleteTrackInfoDataProvider(); |
193 | - $deleteTrack->setId(static::spotifySong['trackId']); |
|
193 | + $deleteTrack->setId(static::spotifySong[ 'trackId' ]); |
|
194 | 194 | |
195 | 195 | $result = $this->spotifyWebApi->deletePlaylistTracks( |
196 | - static::spotifyInfo['playlistId'], |
|
197 | - [$deleteTrack] |
|
196 | + static::spotifyInfo[ 'playlistId' ], |
|
197 | + [ $deleteTrack ] |
|
198 | 198 | ); |
199 | 199 | |
200 | 200 | $this->assertTrue($result); |
201 | 201 | |
202 | 202 | $spotifyPlayList = $this->spotifyWebApi->getPlaylistTracks( |
203 | - static::spotifyInfo['playlistId'] |
|
203 | + static::spotifyInfo[ 'playlistId' ] |
|
204 | 204 | ); |
205 | 205 | |
206 | 206 | $this->assertCount(1, $spotifyPlayList->getItems()); |
@@ -213,8 +213,8 @@ discard block |
||
213 | 213 | $deleteTrack = new DeleteTrackInfoDataProvider(); |
214 | 214 | $deleteTrack->setId('uniTest-Not_FOUND--ID'); |
215 | 215 | $this->spotifyWebApi->deletePlaylistTracks( |
216 | - static::spotifyInfo['playlistId'], |
|
217 | - [$deleteTrack] |
|
216 | + static::spotifyInfo[ 'playlistId' ], |
|
217 | + [ $deleteTrack ] |
|
218 | 218 | ); |
219 | 219 | } catch (\SpotifyWebAPI\SpotifyWebAPIException $e) { |
220 | 220 | $this->assertSame( |
@@ -231,8 +231,8 @@ discard block |
||
231 | 231 | { |
232 | 232 | $this->expectException(\RuntimeException::class); |
233 | 233 | $this->spotifyWebApi->deletePlaylistTracks( |
234 | - static::spotifyInfo['playlistId'], |
|
235 | - ['id' => 'hahaha'] |
|
234 | + static::spotifyInfo[ 'playlistId' ], |
|
235 | + [ 'id' => 'hahaha' ] |
|
236 | 236 | ); |
237 | 237 | } |
238 | 238 | } |