Unzip albums, tag songs with album data

free_bandcamp_downloader/__init__.py

@@ -1,20 +1,26 @@
-import os, logging, atexit
+import atexit
+import logging
+import os
 from configparser import ConfigParser
 
 logging.basicConfig(level=os.environ.get('LOGLEVEL', 'INFO'))
 logger = logging.getLogger(__name__)
 
-__version__ = 'v0.0.3'
+__version__ = 'v0.0.4'
 
 if 'XDG_CONFIG_HOME' in os.environ:
-    config_dir = os.path.join(os.environ['XDG_CONFIG_HOME'], 'free-bandcamp-downloader')
+    config_dir = os.path.join(
+        os.environ['XDG_CONFIG_HOME'], 'free-bandcamp-downloader')
 else:
-    config_dir = os.path.join(os.path.expanduser('~'), '.config', 'free-bandcamp-downloader')
+    config_dir = os.path.join(os.path.expanduser(
+        '~'), '.config', 'free-bandcamp-downloader')
 
 if 'XDG_DATA_HOME' in os.environ:
-    data_dir = os.path.join(os.environ['XDG_DATA_HOME'], 'free-bandcamp-downloader')
+    data_dir = os.path.join(
+        os.environ['XDG_DATA_HOME'], 'free-bandcamp-downloader')
 else:
-    data_dir = os.path.join(os.path.expanduser('~'), '.local', 'share', 'free-bandcamp-downloader')
+    data_dir = os.path.join(os.path.expanduser(
+        '~'), '.local', 'share', 'free-bandcamp-downloader')
 
 download_history_file = os.path.join(data_dir, 'downloaded.txt')
 
@@ -44,6 +50,7 @@ if not os.path.exists(download_history_file):
 parser = ConfigParser()
 parser.read(config_file)
 
+
 class Config:
     def __init__(self, parser: ConfigParser):
         self.parser = parser
@@ -62,4 +69,5 @@ class Config:
     def __str__(self):
         return str(dict(self.parser['free-bandcamp-downloader']))
 
+
 config = Config(parser)

free_bandcamp_downloader/main.py

@@ -33,22 +33,41 @@ Formats:
     - AIFF
 """
 
-import atexit, time, sys, urllib.request, os, re
+import atexit
+import glob
+import os
+import re
+import sys
+import time
+import urllib.request
+import zipfile
+from urllib.parse import urlsplit
+
+import bs4
+import mutagen
+from bs4 import BeautifulSoup, SoupStrainer
 from docopt import docopt
 from guerrillamail import GuerrillaMailSession
 from selenium import webdriver
-from selenium.webdriver.support.ui import Select
-from selenium.webdriver.support import expected_conditions as EC
-from selenium.webdriver.support.wait import WebDriverWait
 from selenium.webdriver.firefox.options import Options
+from selenium.webdriver.support import expected_conditions as EC
+from selenium.webdriver.support.ui import Select
+from selenium.webdriver.support.wait import WebDriverWait
 
 from free_bandcamp_downloader import __version__, config, logger
 
+
+def kill_geckodriver():
+    os.system("taskkill /f /im geckodriver.exe /T")
+
+
+atexit.register(kill_geckodriver)
+
 # Global variables
 arguments = None
-driver = None
 mail_session = None
 expected_emails = 0
+mail_album_data = {}
 downloaded = set()
 options = {
     'country': None,
@@ -73,7 +92,11 @@ xpath = {
     'preparing': '//span[@class="preparing-title"]',
     'formats': '//div[@class="item-format button"]',
     'download': '//a[@class="item-button"]',
-    'albums': '//a[./p[@class="title"]]'
+    'albums': '//a[./p[@class="title"]]',
+    'album-link': '//div[@class="download-artwork"]/a',
+    'album-tags': '//div[contains(@class, "tralbum-tags-nu")]',
+    'album-credits': '//div[@class="tralbumData tralbum-credits"]',
+    'album-about': '//div[@class="tralbumData tralbum-about"]',
 }
 
 formats = {
@@ -87,44 +110,56 @@ formats = {
     'AIFF': 'AIFF'
 }
 
+
 def wait():
     time.sleep(1)
 
+
 def init_email():
     global mail_session
     if not mail_session and (not options['email'] or options['email'] == 'auto'):
         mail_session = GuerrillaMailSession()
         options['email'] = mail_session.get_session_state()['email_address']
 
-def init_driver():
-    global driver
-    if not driver:
+
+def get_driver():
     options = Options()
     options.add_argument('--headless')
-        driver = webdriver.Firefox(firefox_options=options, service_log_path=os.devnull)
-        atexit.register(driver.quit)
+    driver = webdriver.Firefox(
+        firefox_options=options, service_log_path=os.devnull)
     driver.implicitly_wait(10)
+    return driver
+
 
 def init_downloaded():
     with open(config.get('download_history_file'), 'r') as f:
         for line in f:
             downloaded.add(line.strip())
 
-def download_file(page_url=None):
+
+def download_file(driver, album_data=None):
     logger.info('On download page')
+    page_url = driver.find_element_by_xpath(
+        xpath['album-link']).get_attribute('href')
+    page_url = urlsplit(page_url).geturl()
+    if album_data is None:
+        album_data = mail_album_data[page_url]
     driver.find_element_by_xpath(xpath['formats']).click()
     wait()
-    driver.find_element_by_xpath(f'//*[text() = "{formats[options["format"]]}"]').click()
+    driver.find_element_by_xpath(
+        f'//*[text() = "{formats[options["format"]]}"]').click()
     logger.info(f'Set format to {formats[options["format"]]}')
     wait()
     button = driver.find_element_by_xpath(xpath['download'])
     WebDriverWait(driver, 60).until(EC.visibility_of(button))
     url = button.get_attribute('href')
     response = urllib.request.urlopen(url)
-    name = os.path.basename(response.headers.get_filename().encode('latin-1').decode('utf-8'))
+    name = os.path.basename(
+        response.headers.get_filename().encode('latin-1').decode('utf-8'))
     length = int(response.getheader('content-length'))
     block_size = length // 10
-    with open(os.path.join(options['dir'], name), 'wb') as f:
+    file_name = os.path.join(options['dir'], name)
+    with open(file_name, 'wb') as f:
         size = 0
         while True:
             buf = response.read(block_size)
@@ -134,78 +169,158 @@ def download_file(page_url=None):
             size += len(buf)
             logger.info(f'Downloading {name}, {size / length * 100: .1f}%')
     response.close()
-    logger.info(f'Downloaded {os.path.join(options["dir"], name)}')
+    logger.info(f'Downloaded {file_name}')
+    if file_name.endswith('zip'):
+        # Unzip archive
+        dir_name = file_name[:-4]
+        with zipfile.ZipFile(file_name, 'r') as f:
+            f.extractall(dir_name)
+        logger.info(f'Unzipped to {dir_name}')
+        os.remove(file_name)
+        files = glob.glob(os.path.join(dir_name, "*"))
+    else:
+        files = [file_name]
+    # Tag downloaded audio files with url & comment
+    logger.info("Setting tags...")
+    for file in files:
+        try:
+            f = mutagen.File(file)
+            if f is None:
+                continue
+            f['website'] = page_url
+            if album_data['tags']:
+                f['genre_tags'] = album_data['tags']
+            if album_data['about'] or album_data['credits']:
+                comment = ''
+                if album_data['about']:
+                    comment += album_data['about']
+                if album_data['about'] and album_data['credits']:
+                    comment += '\n\n'
+                if album_data['credits']:
+                    comment += album_data['credits']
+                f['comment'] = comment
+            f.save()
+        except Exception as e:
+            logger.info(f"Could not tag {file} - {e.__class__}: {e}")
     # successfully downloaded file, add to download history
-    if page_url:
     downloaded.add(page_url)
     with open(config.get('download_history_file'), 'a') as f:
         f.write(f'{page_url}\n')
 
-def download_album(url):
+
+def download_files(driver, urls):
+    for url in urls:
+        driver.get(url)
+        download_file(driver)
+
+
+def get_text(text):
+    text = ''.join(line.strip() for line in text.split('\n'))
+    text = text.replace('<br>', '\n')
+    return BeautifulSoup(text, features='html.parser').get_text()
+
+
+def download_album(driver, url):
+    # Remove url params
+    url = urlsplit(url).geturl()
     if url in downloaded and not arguments['--force']:
+        logger.error(
+            f'{url} already downloaded. To download anyways, use option --force')
         return f'{url} already downloaded. To download anyways, use option --force'
-    init_driver()
     driver.get(url)
     wait()
     try:
+        # Get album data
+        album_data = {
+            'about': None,
+            'credits': None,
+            'tags': None
+        }
+        try:
+            s = driver.find_element_by_xpath(
+                xpath['album-about']).get_attribute('innerHTML')
+            s = get_text(s)
+            album_data['about'] = s
+        except Exception as e:
+            logger.info(f"Could not get album about - {e.__class__}: {e}")
+        try:
+            s = driver.find_element_by_xpath(
+                xpath['album-credits']).get_attribute('innerHTML')
+            s = get_text(s)
+            album_data['credits'] = s
+        except Exception as e:
+            logger.info(f"Could not get album credits - {e.__class__}: {e}")
+        try:
+            s = driver.find_element_by_xpath(
+                xpath['album-tags']).get_attribute('innerHTML')
+            tags = {a.text for a in BeautifulSoup(
+                s, features='html.parser', parse_only=SoupStrainer('a'))}
+            album_data['tags'] = ','.join(sorted(tags))
+        except Exception as e:
+            logger.info(f"Could not get album tags - {e.__class__}: {e}")
+
+        logger.info(f"Album data: {album_data}")
+
         button = driver.find_element_by_xpath(xpath['buy'])
         if button.text == 'Free Download':
-            logger.info('Album is Free Download')
+            logger.info(f'{url} is Free Download')
             button.click()
             wait()
-            return download_file(url)
+            return download_file(driver, album_data)
         else:
             # name your price download
-            logger.info('Album is not Free Download')
+            logger.info(f'{url} is not Free Download')
             button.click()
             wait()
             price_input = driver.find_element_by_xpath(xpath['price'])
             price_input.click()
             price_input.send_keys('0')
-            logger.info('Set payment to 0')
             wait()
             try:
                 driver.find_element_by_xpath(xpath['download-nyp']).click()
             except:
-                return f'Album {url} is not free'
+                logger.error(f'{url} is not free')
+                return f'{url} is not free'
             checkout = driver.find_element_by_xpath(xpath['checkout'])
             if checkout.text == 'Download Now':
                 checkout.click()
                 wait()
-                return download_file(url)
+                return download_file(driver, album_data)
             else:
                 init_email()
-                logger.info('Album requires email')
+                logger.info(f'{url} requires email')
                 # fill out info
-                driver.find_element_by_xpath(xpath['email']).send_keys(options['email'])
+                driver.find_element_by_xpath(
+                    xpath['email']).send_keys(options['email'])
                 wait()
-                driver.find_element_by_xpath(xpath['zipcode']).send_keys(options['zipcode'])
+                driver.find_element_by_xpath(
+                    xpath['zipcode']).send_keys(options['zipcode'])
                 wait()
-                Select(driver.find_element_by_xpath(xpath['country'])).select_by_visible_text(options['country'])
+                Select(driver.find_element_by_xpath(
+                    xpath['country'])).select_by_visible_text(options['country'])
                 wait()
                 checkout.click()
-                logger.info(f'Download link sent to {options["email"]}')
                 global expected_emails
                 expected_emails += 1
-                downloaded.add(url)
-                with open(config.get('download_history_file'), 'a') as f:
-                    f.write(f'{url}\n')
-
+                mail_album_data[url] = album_data
     except Exception as e:
+        logger.error(f"Error downloading {url} - {e.__class__}: {e}")
         return e
 
 
-def download_label(url):
-    init_driver()
+def download_albums(driver, urls):
+    for link in urls:
+        logger.info(f'Downloading {link}')
+        download_album(driver, link)
+
+
+def download_label(driver, url):
     driver.get(url)
     links = []
     for album in driver.find_elements_by_xpath(xpath['albums']):
         links.append(album.get_attribute('href'))
-    for link in links:
-        logger.info(f'Downloading album at {link}')
-        err = download_album(link)
-        if err:
-            logger.info(err)
+    download_albums(driver, links)
+
 
 def main():
     global arguments
@@ -219,19 +334,22 @@ def main():
             else:
                 options[option] = config.get(option)
             if not options[option]:
-                logger.error(f'{option} is not set, use "bcdl-free setdefault {arg} <{option}>"')
+                logger.error(
+                    f'{option} is not set, use "bcdl-free setdefault {arg} <{option}>"')
                 sys.exit(1)
         if options['format'] not in formats:
-            logger.error(f'{options["format"]} is not a valid format. See "bcdl-free -h" for valid formats')
+            logger.error(
+                f'{options["format"]} is not a valid format. See "bcdl-free -h" for valid formats')
             sys.exit(1)
         init_downloaded()
+    driver = get_driver()
+    try:
         if arguments['-a']:
-        err = download_album(arguments['-a'])
+            err = download_album(driver, arguments['-a'])
             if err:
-            logger.error(err)
                 sys.exit(1)
         elif arguments['-l']:
-        download_label(arguments['-l'])
+            download_label(driver, arguments['-l'])
         elif arguments['setdefault']:
             # write arguments to config
             for option in options:
@@ -245,6 +363,7 @@ def main():
                 pass
         # download emailed albums
         checked_ids = set()
+        album_urls = set()
         global expected_emails
         logger.info(f'Waiting for {expected_emails} emails from bandcamp')
         while expected_emails > 0:
@@ -259,11 +378,17 @@ def main():
                             match = link_regex.search(email.body)
                             if match:
                                 download_url = match.group('url')
-                            driver.get(download_url)
-                            download_file()
+                                album_urls.add(download_url)
                                 expected_emails -= 1
             except Exception as e:
                 logger.error(e)
+        logger.info(f'Downloading {len(album_urls)} albums...')
+        download_files(driver, album_urls)
+    except:
+        raise
+    finally:
+        driver.quit()
+
 
 if __name__ == '__main__':
     main()

setup.py

@@ -1,4 +1,4 @@
-from setuptools import setup, find_packages
+from setuptools import find_packages, setup
 
 import free_bandcamp_downloader
 
@@ -10,7 +10,8 @@ setup(
     install_requires=[
         'selenium',
         'docopt',
-        'python-guerrillamail'
+        'python-guerrillamail',
+        'mutagen'
     ],
     entry_points={
         'console_scripts': [