chore: refactor code quality issues

Author: akshgpt7

beets/dbcore/db.py

@@ -729,10 +729,10 @@ def __iter__(self):
     def _get_indexed_flex_attrs(self):
         """ Index flexible attributes by the entity id they belong to
         """
-        flex_values = dict()
+        flex_values = {}
         for row in self.flex_rows:
             if row['entity_id'] not in flex_values:
-                flex_values[row['entity_id']] = dict()
+                flex_values[row['entity_id']] = {}
 
             flex_values[row['entity_id']][row['key']] = row['value']
 

beets/importer.py

@@ -187,7 +187,7 @@ def __init__(self, lib, loghandler, paths, query):
         self.logger = self._setup_logging(loghandler)
         self.paths = paths
         self.query = query
-        self._is_resuming = dict()
+        self._is_resuming = {}
         self._merged_items = set()
         self._merged_dirs = set()
 

beets/ui/__init__.py

@@ -791,7 +791,7 @@ def _store_dict(option, opt_str, value, parser):
     if option_values is None:
         # This is the first supplied ``key=value`` pair of option.
         # Initialize empty dictionary and get a reference to it.
-        setattr(parser.values, dest, dict())
+        setattr(parser.values, dest, {})
         option_values = getattr(parser.values, dest)
 
     try:

beetsplug/bpd/__init__.py

@@ -21,6 +21,7 @@
 from __future__ import division, absolute_import, print_function
 
 import re
+import sys
 from string import Template
 import traceback
 import random
@@ -334,7 +335,7 @@ def cmd_idle(self, conn, *subsystems):
 
     def cmd_kill(self, conn):
         """Exits the server process."""
-        exit(0)
+        sys.exit(0)
 
     def cmd_close(self, conn):
         """Closes the connection."""

beetsplug/export.py

@@ -33,7 +33,7 @@
 class ExportEncoder(json.JSONEncoder):
     """Deals with dates because JSON doesn't have a standard"""
     def default(self, o):
-        if isinstance(o, datetime) or isinstance(o, date):
+        if isinstance(o, (datetime, date)):
             return o.isoformat()
         return json.JSONEncoder.default(self, o)
 

beetsplug/fetchart.py

@@ -504,7 +504,7 @@ def get(self, album, plugin, paths):
 
         matches = []
         # can there be more than one releasegroupid per response?
-        for mbid, art in data.get(u'albums', dict()).items():
+        for mbid, art in data.get(u'albums', {}).items():
             # there might be more art referenced, e.g. cdart, and an albumcover
             # might not be present, even if the request was successful
             if album.mb_releasegroupid == mbid and u'albumcover' in art:

beetsplug/fish.py

@@ -110,7 +110,7 @@ def run(self, lib, opts, args):
         # Collect commands, their aliases, and their help text
         cmd_names_help = []
         for cmd in beetcmds:
-            names = [alias for alias in cmd.aliases]
+            names = list(cmd.aliases)
             names.append(cmd.name)
             for name in names:
                 cmd_names_help.append((name, cmd.help))
@@ -238,7 +238,7 @@ def get_all_commands(beetcmds):
     # Formatting for Fish to complete command options
     word = ""
     for cmd in beetcmds:
-        names = [alias for alias in cmd.aliases]
+        names = list(cmd.aliases)
         names.append(cmd.name)
         for name in names:
             name = _escape(name)

beetsplug/importadded.py

@@ -27,7 +27,7 @@ def __init__(self):
         # album.path for old albums that were replaced by a reimported album
         self.replaced_album_paths = None
         # item path in the library to the mtime of the source file
-        self.item_mtime = dict()
+        self.item_mtime = {}
 
         register = self.register_listener
         register('import_task_created', self.check_config)

beetsplug/info.py

@@ -235,7 +235,7 @@ def filter_(data):
         matchers.append(re.compile(key + '$'))
 
     def filter_(data):
-        filtered = dict()
+        filtered = {}
         for key, value in data.items():
             if any([m.match(key) for m in matchers]):
                 filtered[key] = value

beetsplug/mbsync.py

@@ -123,7 +123,7 @@ def albums(self, lib, query, move, pretend, write):
             # Map release track and recording MBIDs to their information.
             # Recordings can appear multiple times on a release, so each MBID
             # maps to a list of TrackInfo objects.
-            releasetrack_index = dict()
+            releasetrack_index = {}
             track_index = defaultdict(list)
             for track_info in album_info.tracks:
                 releasetrack_index[track_info.release_track_id] = track_info