Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

AfterEffects: Use client query functions #3374

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions openpype/hosts/aftereffects/api/pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,14 +65,14 @@ def on_pyblish_instance_toggled(instance, old_value, new_value):
instance[0].Visible = new_value


def get_asset_settings():
def get_asset_settings(asset_doc):
"""Get settings on current asset from database.

Returns:
dict: Scene data.

"""
asset_data = lib.get_asset()["data"]
asset_data = asset_doc["data"]
fps = asset_data.get("fps")
frame_start = asset_data.get("frameStart")
frame_end = asset_data.get("frameEnd")
Expand Down
11 changes: 3 additions & 8 deletions openpype/hosts/aftereffects/plugins/create/workfile_creator.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import openpype.hosts.aftereffects.api as api
from openpype.client import get_asset_by_name
from openpype.pipeline import (
AutoCreator,
CreatedInstance,
Expand Down Expand Up @@ -41,10 +42,7 @@ def create(self, options=None):
host_name = legacy_io.Session["AVALON_APP"]

if existing_instance is None:
asset_doc = legacy_io.find_one({
"type": "asset",
"name": asset_name
})
asset_doc = get_asset_by_name(project_name, asset_name)
subset_name = self.get_subset_name(
variant, task_name, asset_doc, project_name, host_name
)
Expand All @@ -69,10 +67,7 @@ def create(self, options=None):
existing_instance["asset"] != asset_name
or existing_instance["task"] != task_name
):
asset_doc = legacy_io.find_one({
"type": "asset",
"name": asset_name
})
asset_doc = get_asset_by_name(project_name, asset_name)
subset_name = self.get_subset_name(
variant, task_name, asset_doc, project_name, host_name
)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# -*- coding: utf-8 -*-
"""Validate scene settings."""
"""Validate scene settings.
Requires:
instance -> assetEntity
instance -> anatomyData
"""
import os
import re

Expand Down Expand Up @@ -67,7 +71,8 @@ def process(self, instance):
if not self.is_active(instance.data):
return

expected_settings = get_asset_settings()
asset_doc = instance.data["assetEntity"]
expected_settings = get_asset_settings(asset_doc)
self.log.info("config from DB::{}".format(expected_settings))

task_name = instance.data["anatomyData"]["task"]["name"]
Expand Down