PickleRick/jellyplist
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Activity

feat: refactor add_jellyfin_user_to_playlist to use internal method for user assignment

Browse Source
This commit is contained in:
Kamil
2024-12-11 14:15:28 +00:00
parent 67d2b3cb9e
commit a44c5b5209
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/routes/routes.py
View File

@@ -506,6 +506,10 @@ def remove_jellyfin_user_from_playlist():
def add_jellyfin_user_to_playlist(): def add_jellyfin_user_to_playlist():
playlist_id = request.args.get('playlist') playlist_id = request.args.get('playlist')
user_id = request.args.get('user') user_id = request.args.get('user')
return add_jellyfin_user_to_playlist_internal(user_id, playlist_id)
def add_jellyfin_user_to_playlist_internal(user_id, playlist_id):
# assign this playlist also to the user in the database # assign this playlist also to the user in the database
# get the playlist from the db # get the playlist from the db
playlist = Playlist.query.filter_by(jellyfin_id=playlist_id).first() playlist = Playlist.query.filter_by(jellyfin_id=playlist_id).first()
@@ -522,8 +526,8 @@ def add_jellyfin_user_to_playlist():
if playlist not in user.playlists: if playlist not in user.playlists:
user.playlists.append(playlist) user.playlists.append(playlist)
db.session.commit() db.session.commit()
if not playlist_id or not user_id: if not playlist_id or not user_id:
return jsonify({'error': 'Playlist or User not specified'}), 400 return jsonify({'error': 'Playlist or User not specified'}), 400
jellyfin.add_users_to_playlist(session_token=functions._get_api_token(), playlist_id=playlist_id, user_id=functions._get_admin_id(), user_ids=[user_id]) jellyfin.add_users_to_playlist(session_token=functions._get_api_token(), playlist_id=playlist_id, user_id=functions._get_admin_id(), user_ids=[user_id])