From b6b6db6f99da5d3eb729b99a0683502adb6fe461 Mon Sep 17 00:00:00 2001
From: Stein Magnus Jodal <stein.magnus@jodal.no>
Date: Thu, 24 Jun 2010 21:15:47 +0200
Subject: [PATCH] MPD: Clearify what SONGID in playlistmove is

---
 mopidy/mpd/frontend.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/mopidy/mpd/frontend.py b/mopidy/mpd/frontend.py
index 39bd1eeb..2d1a8bdf 100644
--- a/mopidy/mpd/frontend.py
+++ b/mopidy/mpd/frontend.py
@@ -1575,8 +1575,8 @@ class MpdFrontend(object):
         raise MpdNotImplemented # TODO
 
     @handle_pattern(r'^playlistmove "(?P<name>[^"]+)" '
-        r'"(?P<songid>\d+)" "(?P<songpos>\d+)"$')
-    def _stored_playlist_playlistmove(self, name, songid, songpos):
+        r'"(?P<from_pos>\d+)" "(?P<to_pos>\d+)"$')
+    def _stored_playlist_playlistmove(self, name, from_pos, to_pos):
         """
         *musicpd.org, stored playlists section:*
 
@@ -1584,6 +1584,12 @@ class MpdFrontend(object):
 
             Moves ``SONGID`` in the playlist ``NAME.m3u`` to the position
             ``SONGPOS``.
+
+        *Clarifications:*
+
+            - The second argument is not a ``SONGID`` as used elsewhere in the
+              protocol documentation, but just the ``SONGPOS`` to move *from*,
+              i.e. ``playlistmove {NAME} {FROM_SONGPOS} {TO_SONGPOS}``.
         """
         raise MpdNotImplemented # TODO