chore: tidy up metadata writer
This commit is contained in:
parent
4b6576ec53
commit
7061f8d64c
@ -11,7 +11,8 @@ class MetadataWriter:
|
||||
def __init__(self):
|
||||
self.logger: logging.Logger = logging.getLogger("MetadataWriter")
|
||||
|
||||
def get_mkvmerge_cmd(self, metadata: dict, in_path: str, out_path: str) -> list[str]:
|
||||
@staticmethod
|
||||
def get_mkvmerge_cmd(metadata: dict, in_path: str, out_path: str) -> list[str]:
|
||||
cmd: list[str] = [
|
||||
"mkvmerge",
|
||||
"-o", out_path
|
||||
@ -49,7 +50,8 @@ class MetadataWriter:
|
||||
cmd.append(in_path)
|
||||
return cmd
|
||||
|
||||
def get_mkvpropedit_cmd(self, metadata: dict, path: str) -> list[str]:
|
||||
@staticmethod
|
||||
def get_mkvpropedit_cmd(metadata: dict, path: str) -> list[str]:
|
||||
cmd: list[str] = [
|
||||
"mkvpropedit",
|
||||
path
|
||||
@ -78,9 +80,6 @@ class MetadataWriter:
|
||||
# Set disposition flags
|
||||
flags = track.get("flags", {})
|
||||
|
||||
def yes_no(flag: str):
|
||||
return f"{track_id}:{"yes" if flags.get(flag, False) else "no"}"
|
||||
|
||||
cmd.extend(["--set", f"flag-default={int(flags.get("default", False))}"])
|
||||
cmd.extend(["--set", f"flag-forced={int(flags.get("forced", False))}"])
|
||||
cmd.extend(["--set", f"flag-original={int(flags.get("original", False))}"])
|
||||
@ -114,8 +113,6 @@ class MetadataWriter:
|
||||
self.get_mkvmerge_cmd(metadata, in_path, out_path)
|
||||
)
|
||||
|
||||
print(cmd)
|
||||
|
||||
# Execute the command
|
||||
self.logger.debug(f"Writing metadata to {os.path.basename(out_path)}")
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user