missing parameters for playlist_edit

Author: lachlan-00

requirements.txt

@@ -1,2 +1,2 @@
-requests~=2.28.1
+requests>=2.28.1
 setuptools>=65.6.3

src/ampache.py

@@ -64,7 +64,7 @@ def __init__(self):
     def set_format(self, myformat: str):
         """ set_format
 
-            Allow forcing a default format
+            Allow forcing a default API output format
 
             INPUTS
             * myformat = (string) 'xml'|'json'
@@ -1518,27 +1518,39 @@ def playlist_create(self, playlist_name, playlist_type):
         return self.return_data(ampache_response)
 
     def playlist_edit(self, filter_id: int, name=False,
-                      object_type=False):
+                      playlist_type=False, owner=False, items=False, tracks=False):
         """ playlist_edit
             MINIMUM_API_VERSION=400001
 
             This modifies name and type of a playlist
 
             INPUTS
-            * filter_id   = (integer)
-            * name        = (string) playlist name //optional
-            * object_type = (string) 'public'|'private'
+            * filter_id     = (integer)
+            * name          = (string) playlist name //optional
+            * playlist_type = (string) 'public'|'private' //optional
+            * owner       = (string) Change playlist owner to the user id (-1 = System playlist) //optional
+            * items       = (string) comma-separated song_id's (replaces existing items with a new id) //optional
+            * tracks      = (string) comma-separated playlisttrack numbers matched to 'items' in order //optional
         """
         ampache_url = self.AMPACHE_URL + '/server/' + self.AMPACHE_API + '.server.php'
         data = {'action': 'playlist_edit',
                 'auth': self.AMPACHE_SESSION,
                 'filter': filter_id,
                 'name': name,
-                'type': object_type}
+                'type': playlist_type,
+                'owner': owner,
+                'items': items,
+                'tracks': tracks}
         if not name:
             data.pop('name')
-        if not object_type:
+        if not playlist_type:
             data.pop('type')
+        if not owner:
+            data.pop('owner')
+        if not items:
+            data.pop('items')
+        if not tracks:
+            data.pop('tracks')
         data = urllib.parse.urlencode(data)
         full_url = ampache_url + '?' + data
         ampache_response = self.fetch_url(full_url, self.AMPACHE_API, 'playlist_edit')
@@ -4491,11 +4503,18 @@ def execute(self, method: str, params=None):
             case 'playlist_delete':
                 return self.playlist_delete(params["filter_id"])
             case 'playlist_edit':
-                if not "playlist_name" in params:
-                    params["playlist_name"] = False
+                if not "name" in params:
+                    params["name"] = False
                 if not "playlist_type" in params:
                     params["playlist_type"] = False
-                return self.playlist_edit(params["filter_id"], params["playlist_name"], params["playlist_type"])
+                if not "owner" in params:
+                    params["owner"] = False
+                if not "items" in params:
+                    params["items"] = False
+                if not "tracks" in params:
+                    params["tracks"] = False
+                return self.playlist_edit(params["filter_id"], params["name"], params["playlist_type"],
+                                          params["owner"], params["items"], params["tracks"])
             case 'playlist_generate':
                 if not "mode" in params:
                     params["mode"] = 'random'