From 377749d79dcd22304537065409775441915b9296 Mon Sep 17 00:00:00 2001 From: James Taylor Date: Thu, 9 Aug 2018 00:59:25 -0700 Subject: music list: don't include album column if no tracks use it --- youtube/watch.py | 25 +++++++++++++++++++------ 1 file changed, 19 insertions(+), 6 deletions(-) (limited to 'youtube') diff --git a/youtube/watch.py b/youtube/watch.py index cfe827b..ea9ef4c 100644 --- a/youtube/watch.py +++ b/youtube/watch.py @@ -351,20 +351,33 @@ def get_watch_page(query_string): if len(music_list) == 0: music_list_html = '' else: + # get the set of attributes which are used by atleast 1 track + # so there isn't an empty, extraneous album column which no tracks use, for example + used_attributes = set() + for track in music_list: + used_attributes = used_attributes | track.keys() + + # now put them in the right order + ordered_attributes = [] + for attribute in ('Artist', 'Title', 'Album'): + if attribute.lower() in used_attributes: + ordered_attributes.append(attribute) + music_list_html = '''
- - - - ''' + # table headings + for attribute in ordered_attributes: + music_list_html += "\n" + music_list_html += '''\n''' + for track in music_list: music_list_html += '''\n''' - for attribute in ('artist', 'title', 'album'): + for attribute in ordered_attributes: try: - value = track[attribute] + value = track[attribute.lower()] except KeyError: music_list_html += '''''' else: -- cgit v1.2.3
Music
ArtistTitleAlbum
" + attribute + "