Skip to content

Commit a6a3aa5

Browse files
committed
Fix typing issues
1 parent a8c2b78 commit a6a3aa5

1 file changed

Lines changed: 15 additions & 13 deletions

File tree

beetsplug/musicbrainz.py

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -379,17 +379,17 @@ def track_info(
379379
if r["type"] == "remixer"
380380
] or None
381381

382-
if recording.get("length"):
383-
info.length = int(recording["length"]) / 1000.0
382+
if length := recording["length"]:
383+
info.length = length / 1000.0
384384

385385
info.trackdisambig = recording.get("disambiguation")
386386

387387
if recording.get("isrcs"):
388388
info.isrc = ";".join(recording["isrcs"])
389389

390-
lyricists = []
391-
composer = []
392-
composer_sort = []
390+
lyricists: list[str] = []
391+
composers: list[str] = []
392+
composer_sort: list[str] = []
393393
for work_relation in recording.get("work_relations", ()):
394394
if work_relation["type"] != "performance":
395395
continue
@@ -406,14 +406,14 @@ def track_info(
406406
if type == "lyricist":
407407
lyricists.append(artist_relation["artist"]["name"])
408408
elif type == "composer":
409-
composer.append(artist_relation["artist"]["name"])
409+
composers.append(artist_relation["artist"]["name"])
410410
composer_sort.append(
411411
artist_relation["artist"]["sort_name"]
412412
)
413413
if lyricists:
414414
info.lyricists = lyricists
415-
if composer:
416-
info.composers = composer
415+
if composers:
416+
info.composers = composers
417417
info.composer_sort = ", ".join(composer_sort)
418418

419419
arrangers = []
@@ -547,8 +547,8 @@ def album_info(self, release: Release) -> AlbumInfo:
547547

548548
ti.artists_ids = _artist_ids(track["artist_credit"])
549549
ti.artist_id = ti.artists_ids[0]
550-
if track.get("length"):
551-
ti.length = int(track["length"]) / (1000.0)
550+
if length := track["length"]:
551+
ti.length = length / 1000.0
552552

553553
track_infos.append(ti)
554554

@@ -614,7 +614,7 @@ def album_info(self, release: Release) -> AlbumInfo:
614614

615615
# Release events.
616616
info.country, release_date = _preferred_release_event(release)
617-
release_group_date = release["release_group"].get("first_release_date")
617+
release_group_date = release["release_group"]["first_release_date"]
618618
if not release_date:
619619
# Fall back if release-specific date is not available.
620620
release_date = release_group_date
@@ -682,9 +682,11 @@ def album_info(self, release: Release) -> AlbumInfo:
682682
url_source.capitalize(),
683683
)
684684

685-
for source, url in urls.items():
685+
for source_name, url in urls.items():
686686
setattr(
687-
info, f"{source}_album_id", extract_release_id(source, url)
687+
info,
688+
f"{source}_album_id",
689+
extract_release_id(source_name, url),
688690
)
689691

690692
extra_albumdatas = plugins.send("mb_album_extract", data=release)

0 commit comments

Comments
 (0)