From 67e8c3ece5b9a12411d25b25b0c619e099ad0e02 Mon Sep 17 00:00:00 2001 From: Martin Wagner Date: Wed, 4 Aug 2021 23:34:00 +0200 Subject: [PATCH] denested "if" in Song --- bin/mpdevil | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/bin/mpdevil b/bin/mpdevil index 8ab422b..f846336 100755 --- a/bin/mpdevil +++ b/bin/mpdevil @@ -491,21 +491,21 @@ class MultiTag(list): class Song(collections.UserDict): def __setitem__(self, key, value): - # time is deprecated https://mpd.readthedocs.io/en/latest/protocol.html#other-metadata - if key != "time": - if key == "duration": - super().__setitem__(key, Duration(value)) - elif key == "format": - super().__setitem__(key, Format(value)) - elif key == "last-modified": - super().__setitem__(key, LastModified(value)) - elif key in ("range", "file", "pos", "id"): - super().__setitem__(key, value) + if key == "time": # time is deprecated https://mpd.readthedocs.io/en/latest/protocol.html#other-metadata + pass + elif key == "duration": + super().__setitem__(key, Duration(value)) + elif key == "format": + super().__setitem__(key, Format(value)) + elif key == "last-modified": + super().__setitem__(key, LastModified(value)) + elif key in ("range", "file", "pos", "id"): + super().__setitem__(key, value) + else: + if isinstance(value, list): + super().__setitem__(key, MultiTag(value)) else: - if isinstance(value, list): - super().__setitem__(key, MultiTag(value)) - else: - super().__setitem__(key, MultiTag([value])) + super().__setitem__(key, MultiTag([value])) def __missing__(self, key): if key == "title":