From fe8d6aa4e837421922481442b41f138961bb14a0 Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Sun, 22 Mar 2015 21:28:35 +0100 Subject: [PATCH] core: Use 'must' instead of 'should' where appropriate --- mopidy/core/playlists.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mopidy/core/playlists.py b/mopidy/core/playlists.py index 5680c018..15d35aa9 100644 --- a/mopidy/core/playlists.py +++ b/mopidy/core/playlists.py @@ -45,7 +45,7 @@ class PlaylistsController(object): :class:`None` or doesn't match a current backend, the first backend is asked to create the playlist. - All new playlists should be created by calling this method, and **not** + All new playlists must be created by calling this method, and **not** by creating new instances of :class:`mopidy.models.Playlist`. :param name: name of the new playlist @@ -150,14 +150,14 @@ class PlaylistsController(object): Save the playlist. For a playlist to be saveable, it must have the ``uri`` attribute set. - You should not set the ``uri`` atribute yourself, but use playlist + You must not set the ``uri`` atribute yourself, but use playlist objects returned by :meth:`create` or retrieved from :attr:`playlists`, which will always give you saveable playlists. The method returns the saved playlist. The return playlist may differ from the saved playlist. E.g. if the playlist name was changed, the returned playlist may have a different URI. The caller of this method - should throw away the playlist sent to this method, and use the + must throw away the playlist sent to this method, and use the returned playlist instead. If the playlist's URI isn't set or doesn't match the URI scheme of a