[PR #2018] [MERGED] chore: remove "own songs only" option #2020

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/2018
Author: @phanan
Created: 7/11/2025
Status: ✅ Merged
Merged: 7/11/2025
Merged by: @phanan

Base: master ← Head: rm-own-songs-only

📝 Commits (1)

cc3407f chore: remove "own songs only" option

📊 Changes

25 files changed (+21 additions, -374 deletions)

View changed files

📝 app/Http/Controllers/API/PlaylistController.php (+0 -3)
📝 app/Http/Controllers/API/SongController.php (+0 -1)
📝 app/Http/Requests/API/PlaylistStoreRequest.php (+0 -2)
📝 app/Http/Requests/API/PlaylistUpdateRequest.php (+0 -2)
📝 app/Http/Requests/API/SongListRequest.php (+0 -1)
📝 app/Http/Resources/PlaylistResource.php (+0 -2)
📝 app/Models/Playlist.php (+0 -2)
📝 app/Repositories/SongRepository.php (+0 -5)
📝 app/Services/PlaylistService.php (+1 -15)
📝 app/Services/SmartPlaylistService.php (+1 -8)
➕ database/migrations/2025_07_11_100738_remove_own_songs_only_setting.php (+15 -0)
📝 resources/assets/js/__tests__/factory/playlistFactory.ts (+0 -1)
📝 resources/assets/js/components/playlist/smart-playlist/CreateSmartPlaylistForm.vue (+0 -12)
📝 resources/assets/js/components/playlist/smart-playlist/EditSmartPlaylistForm.vue (+0 -10)
📝 resources/assets/js/components/screens/AllSongsScreen.spec.ts (+0 -1)
📝 resources/assets/js/components/screens/AllSongsScreen.vue (+1 -29)
📝 resources/assets/js/stores/playlistStore.ts (+2 -5)
📝 resources/assets/js/stores/songStore.spec.ts (+1 -2)
📝 resources/assets/js/stores/songStore.ts (+0 -1)
📝 resources/assets/js/types.d.ts (+0 -1)

...and 5 more files

📄 Description

This option is now obsolete and has been replaced by a user-based preference.

_{🔄 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/2018 **Author:** [@phanan](https://github.com/phanan) **Created:** 7/11/2025 **Status:** ✅ Merged **Merged:** 7/11/2025 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `rm-own-songs-only` --- ### 📝 Commits (1) - [`cc3407f`](https://github.com/koel/koel/commit/cc3407f15e510069fa832354e0204f06134df82b) chore: remove "own songs only" option ### 📊 Changes **25 files changed** (+21 additions, -374 deletions) <details> <summary>View changed files</summary> 📝 `app/Http/Controllers/API/PlaylistController.php` (+0 -3) 📝 `app/Http/Controllers/API/SongController.php` (+0 -1) 📝 `app/Http/Requests/API/PlaylistStoreRequest.php` (+0 -2) 📝 `app/Http/Requests/API/PlaylistUpdateRequest.php` (+0 -2) 📝 `app/Http/Requests/API/SongListRequest.php` (+0 -1) 📝 `app/Http/Resources/PlaylistResource.php` (+0 -2) 📝 `app/Models/Playlist.php` (+0 -2) 📝 `app/Repositories/SongRepository.php` (+0 -5) 📝 `app/Services/PlaylistService.php` (+1 -15) 📝 `app/Services/SmartPlaylistService.php` (+1 -8) ➕ `database/migrations/2025_07_11_100738_remove_own_songs_only_setting.php` (+15 -0) 📝 `resources/assets/js/__tests__/factory/playlistFactory.ts` (+0 -1) 📝 `resources/assets/js/components/playlist/smart-playlist/CreateSmartPlaylistForm.vue` (+0 -12) 📝 `resources/assets/js/components/playlist/smart-playlist/EditSmartPlaylistForm.vue` (+0 -10) 📝 `resources/assets/js/components/screens/AllSongsScreen.spec.ts` (+0 -1) 📝 `resources/assets/js/components/screens/AllSongsScreen.vue` (+1 -29) 📝 `resources/assets/js/stores/playlistStore.ts` (+2 -5) 📝 `resources/assets/js/stores/songStore.spec.ts` (+1 -2) 📝 `resources/assets/js/stores/songStore.ts` (+0 -1) 📝 `resources/assets/js/types.d.ts` (+0 -1) _...and 5 more files_ </details> ### 📄 Description This option is now obsolete and has been replaced by a user-based preference. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>