danvics/ytdl-sub
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

do not over complicate

Browse source
This commit is contained in:
Jesse Bannon 2026-01-19 12:26:03 -08:00
parent 71f43e37b7
commit 82dc29b0d5
1 changed files with 2 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/ytdl_sub/script/script.py
View file

@ -281,10 +281,10 @@ class Script:
self._variables[variable_name] = ResolvedSyntaxTree(ast=[resolved]) self._variables[variable_name] = ResolvedSyntaxTree(ast=[resolved])
def _recursive_get_unresolved_output_filter_variables( def _recursive_get_unresolved_output_filter_variables(
self, current_var: SyntaxTree, subset_to_resolve: Set[str], unresolvable: Set[Variable], allow_partial_resolve: bool = False self, current_var: SyntaxTree, subset_to_resolve: Set[str], unresolvable: Set[Variable]
) -> Set[str]: ) -> Set[str]:
for var_dep in current_var.variables: for var_dep in current_var.variables:
if var_dep in unresolvable and not allow_partial_resolve: if var_dep in unresolvable:
raise ScriptVariableNotResolved( raise ScriptVariableNotResolved(
f"Output filter variable contains the variable {var_dep} " f"Output filter variable contains the variable {var_dep} "
f"which is set as unresolvable" f"which is set as unresolvable"
@ -299,14 +299,12 @@ class Script:
current_var=self._variables[var_dep.name], current_var=self._variables[var_dep.name],
subset_to_resolve=subset_to_resolve, subset_to_resolve=subset_to_resolve,
unresolvable=unresolvable, unresolvable=unresolvable,
allow_partial_resolve=allow_partial_resolve,
) )
for custom_func_dep in current_var.custom_functions: for custom_func_dep in current_var.custom_functions:
subset_to_resolve |= self._recursive_get_unresolved_output_filter_variables( subset_to_resolve |= self._recursive_get_unresolved_output_filter_variables(
current_var=self._functions[custom_func_dep.name], current_var=self._functions[custom_func_dep.name],
subset_to_resolve=subset_to_resolve, subset_to_resolve=subset_to_resolve,
unresolvable=unresolvable, unresolvable=unresolvable,
allow_partial_resolve=allow_partial_resolve,
) )
for lambda_func in current_var.lambdas: for lambda_func in current_var.lambdas:
@ -315,7 +313,6 @@ class Script:
current_var=self._functions[lambda_func.value], current_var=self._functions[lambda_func.value],
subset_to_resolve=subset_to_resolve, subset_to_resolve=subset_to_resolve,
unresolvable=unresolvable, unresolvable=unresolvable,
allow_partial_resolve=allow_partial_resolve,
) )
return subset_to_resolve return subset_to_resolve
@ -324,7 +321,6 @@ class Script:
self, self,
output_filter: Set[str], output_filter: Set[str],
unresolvable: Set[Variable], unresolvable: Set[Variable],
allow_partial_resolvable: bool = False,
) -> Set[str]: ) -> Set[str]:
""" """
When an output filter is applied, only a subset of variables that the filter When an output filter is applied, only a subset of variables that the filter
@ -344,7 +340,6 @@ class Script:
current_var=self._variables[output_filter_variable], current_var=self._variables[output_filter_variable],
subset_to_resolve=subset_to_resolve, subset_to_resolve=subset_to_resolve,
unresolvable=unresolvable, unresolvable=unresolvable,
allow_partial_resolve=allow_partial_resolvable,
) )
return subset_to_resolve return subset_to_resolve