Merge pull request #12 from michaelb/dev

Dev

Author: michaelb

README.md

@@ -1,6 +1,6 @@
 # auto-ytdl
 
-![](https://img.shields.io/badge/Release-v1.1.1-green.svg)
+![](https://img.shields.io/badge/Release-v1.1.2-green.svg)
 
 I know, _yet another_ youtube-dl wrapper. But this one does something others don't: ability to automate your new music downloads, just like package managers automate your updates.
 
@@ -87,6 +87,7 @@ see also [Usage](#usage) and [Examples](#examples)
 
 - youtube-dl
 - ffmpeg
+- kid3-cli [AUR](https://aur.archlinux.org/packages/kid3-cli/)
 - python3-pip (for manual install / build)
 
 For every feature to work, you need a XDG-compliant desktop environment.

autoytdl/AYTDL.py

@@ -68,7 +68,7 @@ def prepare_ytdl_commmand_line(dateafter):
         if not ((url[0] == "\"" or url[0] == "'") and (url[-1] == "'" or url[-1] == "\"")):
             url = "\""+url+"\""
 
-        # here
+        # here we run youtube-dl
         exit_code = os.system("youtube-dl " +
                               prepare_ytdl_commmand_line(dateafter) + " " + url)
         if exit_code != 0:  # may be a youtube-dl error taht is recoverable

autoytdl/config.py

@@ -95,6 +95,7 @@ def __init__(self):
         self.comments += ["Not user modifiable"]
         self.comments += [
             "Ideally, should only be \"best\" (default) or \"mp3\" so the eventual thumbnail is correctly embedded most of the time (opus and mp3 format are the main focus of dev)"]
+        self.comments += ["You should not change this"]
         self.comments += ["Not user modifiable"]
         self.comments += ["Do not download more than this much music from one source"]
         self.youtube_dl_args = {"ignore-errors": True,
@@ -108,6 +109,7 @@ def __init__(self):
                                 "force-ipv4": True,
                                 "extract-audio": True,
                                 "audio-format": "best",
+                                "audio-quality": 0,
                                 "write-thumbnail": True,
                                 "playlist-end": 150}
 

autoytdl/name_cleaner.py

@@ -12,8 +12,11 @@ def remove_brackets(string):
     string = re.sub(r'\(\)', '', string)
     string = re.sub(r'\( \)', '', string)
     string = re.sub(r'\(  \)', '', string)
-    # dont want to test more compelx regex for this
     string = re.sub(r'\(   \)', '', string)
+    string = re.sub(r'\(\ \)', '', string)
+    string = re.sub(r'\(\ \ \)', '', string)
+    string = re.sub(r'\(\ \ \ \)', '', string)
+    # dont want to test more compelx regex for this
     string = re.sub("[\{\[].*?[\}\]]", "", string)
     return string
 

autoytdl/version.py

@@ -1,2 +1,2 @@
 # follows semantic versionning
-__version__ = "1.1.1"
+__version__ = "1.1.2"