@@ -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 | } |