[PR #2091] [MERGED] refactor: turn some method args into DTO #2064

New issue

Closed

opened 2026-02-26 03:33:10 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 03:33:10 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/2091
Author: @phanan
Created: 9/9/2025
Status: ✅ Merged
Merged: 9/9/2025
Merged by: @phanan

Base: master ← Head: dto

📝 Commits (1)

105e35e refactor: turn some method args into DTO

📊 Changes

125 files changed (+548 additions, -383 deletions)

View changed files

📝 app/Casts/UserPreferencesCast.php (+1 -1)
📝 app/Http/Controllers/API/AlbumController.php (+1 -2)
📝 app/Http/Controllers/API/Artist/ArtistController.php (+1 -2)
📝 app/Http/Controllers/API/PlaylistController.php (+2 -9)
📝 app/Http/Controllers/API/ProfileController.php (+1 -9)
📝 app/Http/Controllers/API/Radio/RadioStationController.php (+6 -22)
📝 app/Http/Controllers/API/SongController.php (+1 -2)
📝 app/Http/Controllers/API/UserController.php (+2 -13)
📝 app/Http/Controllers/Demo/NewSessionController.php (+7 -7)
📝 app/Http/Controllers/SSO/GoogleCallbackController.php (+1 -1)
📝 app/Http/Integrations/Lastfm/Requests/GetAlbumInfoRequest.php (+2 -2)
📝 app/Http/Integrations/Lastfm/Requests/GetArtistInfoRequest.php (+2 -2)
📝 app/Http/Integrations/Lastfm/Requests/GetSessionKeyRequest.php (+1 -1)
📝 app/Http/Integrations/Lastfm/Requests/ScrobbleRequest.php (+1 -1)
📝 app/Http/Integrations/Lastfm/Requests/ToggleLoveTrackRequest.php (+1 -1)
📝 app/Http/Integrations/Lastfm/Requests/UpdateNowPlayingRequest.php (+1 -1)
📝 app/Http/Integrations/iTunes/Requests/GetTrackRequest.php (+1 -1)
📝 app/Http/Requests/API/AcceptUserInvitationRequest.php (+1 -1)
📝 app/Http/Requests/API/ActivateLicenseRequest.php (+1 -1)
📝 app/Http/Requests/API/Album/AlbumUpdateRequest.php (+12 -2)

...and 80 more files

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/koel/koel/pull/2091 **Author:** [@phanan](https://github.com/phanan) **Created:** 9/9/2025 **Status:** ✅ Merged **Merged:** 9/9/2025 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `dto` --- ### 📝 Commits (1) - [`105e35e`](https://github.com/koel/koel/commit/105e35eaef84e865c8584588af84a3c69fd3a348) refactor: turn some method args into DTO ### 📊 Changes **125 files changed** (+548 additions, -383 deletions) <details> <summary>View changed files</summary> 📝 `app/Casts/UserPreferencesCast.php` (+1 -1) 📝 `app/Http/Controllers/API/AlbumController.php` (+1 -2) 📝 `app/Http/Controllers/API/Artist/ArtistController.php` (+1 -2) 📝 `app/Http/Controllers/API/PlaylistController.php` (+2 -9) 📝 `app/Http/Controllers/API/ProfileController.php` (+1 -9) 📝 `app/Http/Controllers/API/Radio/RadioStationController.php` (+6 -22) 📝 `app/Http/Controllers/API/SongController.php` (+1 -2) 📝 `app/Http/Controllers/API/UserController.php` (+2 -13) 📝 `app/Http/Controllers/Demo/NewSessionController.php` (+7 -7) 📝 `app/Http/Controllers/SSO/GoogleCallbackController.php` (+1 -1) 📝 `app/Http/Integrations/Lastfm/Requests/GetAlbumInfoRequest.php` (+2 -2) 📝 `app/Http/Integrations/Lastfm/Requests/GetArtistInfoRequest.php` (+2 -2) 📝 `app/Http/Integrations/Lastfm/Requests/GetSessionKeyRequest.php` (+1 -1) 📝 `app/Http/Integrations/Lastfm/Requests/ScrobbleRequest.php` (+1 -1) 📝 `app/Http/Integrations/Lastfm/Requests/ToggleLoveTrackRequest.php` (+1 -1) 📝 `app/Http/Integrations/Lastfm/Requests/UpdateNowPlayingRequest.php` (+1 -1) 📝 `app/Http/Integrations/iTunes/Requests/GetTrackRequest.php` (+1 -1) 📝 `app/Http/Requests/API/AcceptUserInvitationRequest.php` (+1 -1) 📝 `app/Http/Requests/API/ActivateLicenseRequest.php` (+1 -1) 📝 `app/Http/Requests/API/Album/AlbumUpdateRequest.php` (+12 -2) _...and 80 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 03:33:10 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/koel-koel#2064

No description provided.

Rows
Columns