chore: Run ruff check --fix

Author: C0rn3j

examples/collection.py

@@ -96,7 +96,7 @@ def show_collection(collection_id, ctype):
 				name=collection["name"], editor=collection["editor"], mbid=collection["id"]
 			)
 		)
-	print("")
+	print()
 	# release count is only available starting with musicbrainzngs 0.5
 	if "release-count" in collection:
 		print("{} releases".format(collection["release-count"]))
@@ -110,7 +110,7 @@ def show_collection(collection_id, ctype):
 		print("{} recordings".format(collection["recording-count"]))
 	if "work-count" in collection:
 		print("{} works".format(collection["work-count"]))
-	print("")
+	print()
 
 	if "release-list" in collection:
 		show_releases(collection)
@@ -124,7 +124,7 @@ def show_releases(collection):
 	print("Releases:")
 	releases_fetched = 0
 	while len(release_list) > 0:
-		print("")
+		print()
 		releases_fetched += len(release_list)
 		for release in release_list:
 			print("{title} ({mbid})".format(title=release["title"], mbid=release["id"]))
@@ -136,7 +136,7 @@ def show_releases(collection):
 		collection = result["collection"]
 		release_list = collection["release-list"]
 
-	print("")
+	print()
 	print("Number of fetched releases: %d" % releases_fetched)
 
 
@@ -154,7 +154,7 @@ def show_releases(collection):
 	username = args.pop(0)
 
 	# Input the password.
-	password = getpass.getpass("Password for {}: ".format(username))
+	password = getpass.getpass(f"Password for {username}: ")
 
 	# Call musicbrainzngs.auth() before making any API calls that
 	# require authentication.
@@ -175,9 +175,9 @@ def show_releases(collection):
 				sys.exit("only release collections can be modified ATM")
 		else:
 			# Print out the collection's contents.
-			print("")
+			print()
 			show_collection(collection_id, options.type)
 	else:
 		# Show all collections.
-		print("")
+		print()
 		show_collections()

examples/find_disc.py

@@ -41,13 +41,13 @@ def show_offsets(offset_list):
 			offsets = str(offset)
 		else:
 			offsets += " " + str(offset)
-	print("\toffsets: {}".format(offsets))
+	print(f"\toffsets: {offsets}")
 
 
 if __name__ == "__main__":
 	args = sys.argv[1:]
 	if len(args) != 1:
-		sys.exit("usage: {} DISC_ID".format(sys.argv[0]))
+		sys.exit(f"usage: {sys.argv[0]} DISC_ID")
 	discid = args[0]
 
 	try:
@@ -69,7 +69,7 @@ def show_offsets(offset_list):
 			print("\tTracks: {}".format(result["disc"]["offset-count"]))
 		for release in result["disc"]["release-list"]:
 			show_release_details(release)
-			print("")
+			print()
 	elif result.get("cdstub"):
 		print("cdstub:")
 		print("\tArtist: {}".format(result["cdstub"]["artist"]))

examples/releasesearch.py

@@ -33,7 +33,7 @@ def show_release_details(rel):
 if __name__ == "__main__":
 	args = sys.argv[1:]
 	if len(args) != 2:
-		sys.exit("usage: {} ARTIST ALBUM".format(sys.argv[0]))
+		sys.exit(f"usage: {sys.argv[0]} ARTIST ALBUM")
 	artist, album = args
 
 	# Keyword arguments to the "search_*" functions limit keywords to
@@ -46,6 +46,6 @@ def show_release_details(rel):
 	if not result["release-list"]:
 		sys.exit("no release found")
 	for idx, release in enumerate(result["release-list"]):
-		print("match #{}:".format(idx + 1))
+		print(f"match #{idx + 1}:")
 		show_release_details(release)
 		print()

musicbrainzngs/__init__.py

@@ -1,2 +1,2 @@
-from musicbrainzngs.musicbrainz import *
 from musicbrainzngs.caa import *
+from musicbrainzngs.musicbrainz import *

musicbrainzngs/caa.py

@@ -4,13 +4,13 @@
 # See the COPYING file for more information.
 
 __all__ = [
-	"set_caa_hostname",
+	"get_image",
+	"get_image_back",
+	"get_image_front",
 	"get_image_list",
-	"get_release_group_image_list",
 	"get_release_group_image_front",
-	"get_image_front",
-	"get_image_back",
-	"get_image",
+	"get_release_group_image_list",
+	"set_caa_hostname",
 ]
 
 import json
@@ -83,10 +83,9 @@ def _caa_request(mbid, imageid=None, size=None, entitytype="release"):
 	if imageid:
 		# If we asked for an image, return the image
 		return resp
-	else:
-		# Otherwise it's json
-		data = _unicode(resp)
-		return json.loads(data)
+	# Otherwise it's json
+	data = _unicode(resp)
+	return json.loads(data)
 
 
 def get_image_list(releaseid):

musicbrainzngs/mbxml.py

@@ -33,7 +33,8 @@ def fixtag(tag, namespaces: dict[str, str]):
 def get_error_message(error):
 	"""Given an error XML message from the webservice containing
 	<error><text>x</text><text>y</text></error>, return a list
-	of [x, y]"""
+	of [x, y]
+	"""
 	try:
 		tree = util.bytes_to_elementtree(error)
 		root = tree.getroot()
@@ -92,7 +93,7 @@ def parse_elements(valid_els: list[str], inner_els: dict, element) -> dict:
 			t = t.split(":")[1]
 		if t in valid_els:
 			result[t] = sub.text or ""
-		elif t in inner_els.keys():
+		elif t in inner_els:
 			inner_result = inner_els[t](sub)
 			if isinstance(inner_result, tuple) and inner_result[0]:
 				result.update(inner_result[1])
@@ -368,8 +369,7 @@ def parse_relation_target(tgt):
 	attributes = parse_attributes(["id"], tgt)
 	if "id" in attributes:
 		return (True, {"target-id": attributes["id"]})
-	else:
-		return (True, {"target-id": tgt.text})
+	return (True, {"target-id": tgt.text})
 
 
 def parse_relation_list(rl):
@@ -460,7 +460,8 @@ def parse_release(release):
 def parse_medium_list(ml):
 	"""medium-list results from search have an additional
 	<track-count> element containing the number of tracks
-	over all mediums. Optionally add this"""
+	over all mediums. Optionally add this
+	"""
 	medium_list = []
 	track_count = None
 	for m in ml: