Frame13/ayon-core
1
0
Fork 1
mirror of https://github.com/ynput/ayon-core.git synced 2025-12-24 21:04:40 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

OP-4760 - fixes after review

Browse source 
Removed unnecessary function, fixed missed arguments.
This commit is contained in:
Petr Kalis 2023-01-20 16:59:47 +01:00
parent 93eb865065
commit f00ee2784a
2 changed files with 6 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

16
openpype/hosts/traypublisher/batch_parsing.py
View file

@ -36,10 +36,6 @@ def get_asset_doc_from_file_name(source_filename, project_name,
matching_asset_doc = parse_containing(project_name, asset_name,
all_selected_asset_ids)
if matching_asset_doc is None:
raise CreatorError(
"Cannot guess asset name from {}".format(source_filename))
return matching_asset_doc, version
@ -49,7 +45,8 @@ def parse_with_version(project_name, asset_name, version_regex,
Eg. 'chair_v001.mov' >> 'chair', 1
"""
log = _handle_log(log)
if not log:
log = Logger.get_logger(__name__)
log.debug(
("Asset doc by \"{}\" was not found, trying version regex.".
format(asset_name)))
@ -80,6 +77,8 @@ def get_asset_by_name_case_not_sensitive(project_name, asset_name,
all_selected_asset_ids=None,
log=None):
"""Handle more cases in file names"""
if not log:
log = Logger.get_logger(__name__)
asset_name = re.compile(asset_name, re.IGNORECASE)
assets = list(get_assets(project_name, asset_ids=all_selected_asset_ids,
@ -91,10 +90,3 @@ def get_asset_by_name_case_not_sensitive(project_name, asset_name,
return
return assets.pop()
def _handle_log(log):
if not log:
log = Logger.get_logger(__name__)
return log

4
openpype/hosts/traypublisher/plugins/create/create_movie_batch.py
View file

@ -59,7 +59,7 @@ class BatchMovieCreator(TrayPublishCreator):
instance_data["creator_attributes"] = {"filepath": filepath}
asset_doc, version = get_asset_doc_from_file_name(
file_name, self.project_name)
file_name, self.project_name, self.version_regex)
subset_name, task_name = self._get_subset_and_task(
asset_doc, data["variant"], self.project_name)
@ -72,7 +72,7 @@ class BatchMovieCreator(TrayPublishCreator):
instance_data, self)
self._store_new_instance(new_instance)
def get_subset_and_task(self, asset_doc, variant, project_name):
def _get_subset_and_task(self, asset_doc, variant, project_name):
"""Create subset name according to standard template process"""
task_name = self._get_task_name(asset_doc)