danvics/ytptube
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Refactor arg_converter usage to remove logging of removed options in Item and YTDLPOpts classes

Browse source
This commit is contained in:
arabcoders 2025-09-01 19:35:35 +03:00
parent 28e3062bef
commit c9af30edda
2 changed files with 2 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/library/ItemDTO.py
View file

@ -151,11 +151,7 @@ class Item:
from .Utils import arg_converter
try:
removed_options: list = []
arg_converter(args=cli, level=True, removed_options=removed_options)
if len(removed_options) > 0:
LOG.warning("Removed the following options '%s' for '%s'.", ", ".join(removed_options), url)
arg_converter(args=cli, level=True)
data["cli"] = cli
except Exception as e:
msg = f"Failed to parse command options for yt-dlp. {e!s}"

16
app/library/YTDLPOpts.py
View file

@ -76,18 +76,12 @@ class YTDLPOpts:
if from_user:
bad_options: dict[str, str] = {k: v for d in REMOVE_KEYS for k, v in d.items()}
removed_options: list = []
for key, value in config.items():
if from_user and key in bad_options:
removed_options.append(bad_options[key])
continue
self._item_opts[key] = value
if len(removed_options) > 0:
LOG.warning("Removed the following options: '%s'.", ", ".join(removed_options))
return self
def preset(self, name: str) -> "YTDLPOpts":
@ -175,15 +169,7 @@ class YTDLPOpts:
if len(self._item_cli) > 0:
try:
removed_options: list = []
user_cli: dict = arg_converter(
args="\n".join(self._item_cli),
level=True,
removed_options=removed_options,
)
if len(removed_options) > 0:
LOG.warning("Removed the following options: '%s'.", ", ".join(removed_options))
user_cli: dict = arg_converter(args="\n".join(self._item_cli), level=True)
except Exception as e:
msg = f"Invalid command options for yt-dlp were given. '{e!s}'."
raise ValueError(msg) from e