From 6246bac7742aa18f852704588ba37d228f0a1413 Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Sat, 28 Jan 2023 00:29:21 +0100
Subject: [PATCH 01/19] renamed 'reset_avalon_context' to
 'reset_current_context'

---
 openpype/pipeline/create/context.py | 6 +++---
 openpype/tools/publisher/control.py | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 9c468ae8fc..29bc32b658 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1138,7 +1138,7 @@ class CreateContext:
 
         self.reset_preparation()
 
-        self.reset_avalon_context()
+        self.reset_current_context()
         self.reset_plugins(discover_publish_plugins)
         self.reset_context_data()
 
@@ -1185,8 +1185,8 @@ class CreateContext:
         self._collection_shared_data = None
         self.refresh_thumbnails()
 
-    def reset_avalon_context(self):
-        """Give ability to reset avalon context.
+    def reset_current_context(self):
+        """Refresh current context.
 
         Reset is based on optional host implementation of `get_current_context`
         function or using `legacy_io.Session`.
diff --git a/openpype/tools/publisher/control.py b/openpype/tools/publisher/control.py
index 50a814de5c..c11d7c53d3 100644
--- a/openpype/tools/publisher/control.py
+++ b/openpype/tools/publisher/control.py
@@ -1756,7 +1756,7 @@ class PublisherController(BasePublisherController):
         self._create_context.reset_preparation()
 
         # Reset avalon context
-        self._create_context.reset_avalon_context()
+        self._create_context.reset_current_context()
 
         self._asset_docs_cache.reset()
 

From d1b41ebac0b7bbac4a1404ca0233d2a7d92e6230 Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Sat, 28 Jan 2023 00:30:24 +0100
Subject: [PATCH 02/19] AvalonMongoDB is not needed for CreateContext or
 Controller

---
 openpype/pipeline/create/context.py | 38 ++++++++++++++---------------
 openpype/tools/publisher/control.py |  5 ++--
 2 files changed, 20 insertions(+), 23 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 29bc32b658..e421a76b6e 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1003,8 +1003,6 @@ class CreateContext:
     Args:
         host(ModuleType): Host implementation which handles implementation and
             global metadata.
-        dbcon(AvalonMongoDB): Connection to mongo with context (at least
-            project).
         headless(bool): Context is created out of UI (Current not used).
         reset(bool): Reset context on initialization.
         discover_publish_plugins(bool): Discover publish plugins during reset
@@ -1012,16 +1010,8 @@ class CreateContext:
     """
 
     def __init__(
-        self, host, dbcon=None, headless=False, reset=True,
-        discover_publish_plugins=True
+        self, host, headless=False, reset=True, discover_publish_plugins=True
     ):
-        # Create conncetion if is not passed
-        if dbcon is None:
-            session = session_data_from_environment(True)
-            dbcon = AvalonMongoDB(session)
-            dbcon.install()
-
-        self.dbcon = dbcon
         self.host = host
 
         # Prepare attribute for logger (Created on demand in `log` property)
@@ -1045,6 +1035,10 @@ class CreateContext:
                 " Missing methods: {}"
             ).format(joined_methods))
 
+        self._current_project_name = None
+        self._current_asset_name = None
+        self._current_task_name = None
+
         self._host_is_valid = host_is_valid
         # Currently unused variable
         self.headless = headless
@@ -1119,9 +1113,16 @@ class CreateContext:
     def host_name(self):
         return os.environ["AVALON_APP"]
 
-    @property
-    def project_name(self):
-        return self.dbcon.active_project()
+    def get_current_project_name(self):
+        return self._current_project_name
+
+    def get_current_asset_name(self):
+        return self._current_asset_name
+
+    def get_current_task_name(self):
+        return self._current_task_name
+
+    project_name = property(get_current_project_name)
 
     @property
     def log(self):
@@ -1210,12 +1211,9 @@ class CreateContext:
         if not task_name:
             task_name = legacy_io.Session.get("AVALON_TASK")
 
-        if project_name:
-            self.dbcon.Session["AVALON_PROJECT"] = project_name
-        if asset_name:
-            self.dbcon.Session["AVALON_ASSET"] = asset_name
-        if task_name:
-            self.dbcon.Session["AVALON_TASK"] = task_name
+        self._current_project_name = project_name
+        self._current_asset_name = asset_name
+        self._current_task_name = task_name
 
     def reset_plugins(self, discover_publish_plugins=True):
         """Reload plugins.
diff --git a/openpype/tools/publisher/control.py b/openpype/tools/publisher/control.py
index c11d7c53d3..83c2dd4b1c 100644
--- a/openpype/tools/publisher/control.py
+++ b/openpype/tools/publisher/control.py
@@ -1589,20 +1589,19 @@ class PublisherController(BasePublisherController):
     Handle both creation and publishing parts.
 
     Args:
-        dbcon (AvalonMongoDB): Connection to mongo with context.
         headless (bool): Headless publishing. ATM not implemented or used.
     """
 
     _log = None
 
-    def __init__(self, dbcon=None, headless=False):
+    def __init__(self, headless=False):
         super(PublisherController, self).__init__()
 
         self._host = registered_host()
         self._headless = headless
 
         self._create_context = CreateContext(
-            self._host, dbcon, headless=headless, reset=False
+            self._host, headless=headless, reset=False
         )
 
         self._publish_plugins_proxy = None

From 0a900f8ae1e4ac3b1ba48aca017be03a7f743e89 Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Sat, 28 Jan 2023 00:30:59 +0100
Subject: [PATCH 03/19] use 'name' attribute of host implementation if is
 available

---
 openpype/pipeline/create/context.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index e421a76b6e..867809a4c1 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1111,6 +1111,8 @@ class CreateContext:
 
     @property
     def host_name(self):
+        if hasattr(self.host, "name"):
+            return self.host.name
         return os.environ["AVALON_APP"]
 
     def get_current_project_name(self):

From 8678f4e2fa36bab16aa8033bf518dd0231fa885c Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Sat, 28 Jan 2023 00:31:27 +0100
Subject: [PATCH 04/19] 'create' returns output from creator

---
 openpype/pipeline/create/context.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 867809a4c1..413580526e 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1429,6 +1429,7 @@ class CreateContext:
         failed = False
         add_traceback = False
         exc_info = None
+        result = None
         try:
             # Fake CreatorError (Could be maybe specific exception?)
             if creator is None:
@@ -1436,7 +1437,7 @@ class CreateContext:
                     "Creator {} was not found".format(identifier)
                 )
 
-            creator.create(*args, **kwargs)
+            result = creator.create(*args, **kwargs)
 
         except CreatorError:
             failed = True
@@ -1458,6 +1459,7 @@ class CreateContext:
                     identifier, label, exc_info, add_traceback
                 )
             ])
+        return result
 
     def creator_removed_instance(self, instance):
         """When creator removes instance context should be acknowledged.

From d09b7812616bfaee29c4089e2ece893ff6bd3faa Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Sat, 28 Jan 2023 00:32:53 +0100
Subject: [PATCH 05/19] implemented helper function 'create_with_context' to
 trigger standartized creation

---
 openpype/pipeline/create/context.py | 64 ++++++++++++++++++++++++++---
 1 file changed, 59 insertions(+), 5 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 413580526e..655af1b8ed 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -8,17 +8,13 @@ import inspect
 from uuid import uuid4
 from contextlib import contextmanager
 
-from openpype.client import get_assets
+from openpype.client import get_assets, get_asset_by_name
 from openpype.settings import (
     get_system_settings,
     get_project_settings
 )
 from openpype.host import IPublishHost
 from openpype.pipeline import legacy_io
-from openpype.pipeline.mongodb import (
-    AvalonMongoDB,
-    session_data_from_environment,
-)
 
 from .creator_plugins import (
     Creator,
@@ -1461,6 +1457,64 @@ class CreateContext:
             ])
         return result
 
+    def create_with_context(
+        self,
+        creator_identifier,
+        variant=None,
+        asset_doc=None,
+        task_name=None,
+        pre_create_data=None
+    ):
+        """Trigger create of plugins with standartized
+
+        Args:
+            creator_identifier (str):
+            asset_doc (Dict[str, Any]):
+            task_name (str): Name of task to which is context related.
+            variant (str): Variant used for subset name.
+            pre_create_data (Dict[str, Any]): Pre-create attribute values.
+
+        Returns:
+            Any: Output of triggered creator's 'create' method.
+
+        Raises:
+            CreatorsCreateFailed: When creation fails.
+        """
+
+        if pre_create_data is None:
+            pre_create_data = {}
+
+        project_name = self.project_name
+        if asset_doc is None:
+            asset_name = self.get_current_asset_name()
+            asset_doc = get_asset_by_name(project_name, asset_name)
+            task_name = self.get_current_task_name()
+
+        creator = self.creators.get(creator_identifier)
+        family = None
+        subset_name = None
+        if creator is not None:
+            family = creator.family
+            subset_name = creator.get_subset_name(
+                variant,
+                task_name,
+                asset_doc,
+                project_name,
+                self.host_name
+            )
+        instance_data = {
+            "asset": asset_doc["name"],
+            "task": task_name,
+            "variant": variant,
+            "family": family
+        }
+        return self.raw_create(
+            creator_identifier,
+            subset_name,
+            instance_data,
+            pre_create_data
+        )
+
     def creator_removed_instance(self, instance):
         """When creator removes instance context should be acknowledged.
 

From 430fe6aed42d8c08f57c7b91dd2eb9185a3d1fde Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Sat, 28 Jan 2023 00:33:33 +0100
Subject: [PATCH 06/19] renamed 'create'->'raw_create' and
 'create_with_context'->'create'

---
 openpype/pipeline/create/context.py | 9 ++++++---
 openpype/tools/publisher/control.py | 2 +-
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 655af1b8ed..a9f8ae3ce1 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1407,8 +1407,8 @@ class CreateContext:
         with self.bulk_instances_collection():
             self._bulk_instances_to_process.append(instance)
 
-    def create(self, identifier, *args, **kwargs):
-        """Wrapper for creators to trigger created.
+    def raw_create(self, identifier, *args, **kwargs):
+        """Wrapper for creators to trigger 'create' method.
 
         Different types of creators may expect different arguments thus the
         hints for args are blind.
@@ -1417,6 +1417,9 @@ class CreateContext:
             identifier (str): Creator's identifier.
             *args (Tuple[Any]): Arguments for create method.
             **kwargs (Dict[Any, Any]): Keyword argument for create method.
+
+        Raises:
+            CreatorsCreateFailed: When creation fails.
         """
 
         error_message = "Failed to run Creator with identifier \"{}\". {}"
@@ -1457,7 +1460,7 @@ class CreateContext:
             ])
         return result
 
-    def create_with_context(
+    def create(
         self,
         creator_identifier,
         variant=None,
diff --git a/openpype/tools/publisher/control.py b/openpype/tools/publisher/control.py
index 83c2dd4b1c..670c22a43e 100644
--- a/openpype/tools/publisher/control.py
+++ b/openpype/tools/publisher/control.py
@@ -2017,7 +2017,7 @@ class PublisherController(BasePublisherController):
 
         success = True
         try:
-            self._create_context.create(
+            self._create_context.raw_create(
                 creator_identifier, subset_name, instance_data, options
             )
         except CreatorsOperationFailed as exc:

From 498c8564f71c4d85ad88a101d6de7ae11357bb7d Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Sat, 28 Jan 2023 00:40:08 +0100
Subject: [PATCH 07/19] swapped argments order in docstring

---
 openpype/pipeline/create/context.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index a9f8ae3ce1..702731f8b2 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1472,9 +1472,9 @@ class CreateContext:
 
         Args:
             creator_identifier (str):
+            variant (str): Variant used for subset name.
             asset_doc (Dict[str, Any]):
             task_name (str): Name of task to which is context related.
-            variant (str): Variant used for subset name.
             pre_create_data (Dict[str, Any]): Pre-create attribute values.
 
         Returns:

From 40712089d94ce22a5d34981584dc7db8beed9554 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Mon, 30 Jan 2023 10:48:57 +0100
Subject: [PATCH 08/19] Validate creator and asset doc

---
 openpype/pipeline/create/context.py | 33 +++++++++++++++++------------
 1 file changed, 19 insertions(+), 14 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 702731f8b2..35024b5af8 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1484,27 +1484,32 @@ class CreateContext:
             CreatorsCreateFailed: When creation fails.
         """
 
-        if pre_create_data is None:
-            pre_create_data = {}
+        creator = self.creators.get(creator_identifier)
+        if creator is None:
+            raise CreatorError(
+                "Creator {} was not found".format(creator_identifier)
+            )
 
         project_name = self.project_name
         if asset_doc is None:
             asset_name = self.get_current_asset_name()
             asset_doc = get_asset_by_name(project_name, asset_name)
             task_name = self.get_current_task_name()
+            if asset_doc is None:
+                raise CreatorError(
+                    "Asset with name {} was not found".format(asset_name)
+                )
 
-        creator = self.creators.get(creator_identifier)
-        family = None
-        subset_name = None
-        if creator is not None:
-            family = creator.family
-            subset_name = creator.get_subset_name(
-                variant,
-                task_name,
-                asset_doc,
-                project_name,
-                self.host_name
-            )
+        if pre_create_data is None:
+            pre_create_data = {}
+
+        subset_name = creator.get_subset_name(
+            variant,
+            task_name,
+            asset_doc,
+            project_name,
+            self.host_name
+        )
         instance_data = {
             "asset": asset_doc["name"],
             "task": task_name,

From 75bffb4daeacc8a31dc584edf3b76b3989a0607d Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Mon, 30 Jan 2023 10:49:17 +0100
Subject: [PATCH 09/19] removed unnecessary family from instance data

---
 openpype/pipeline/create/context.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 35024b5af8..dbbde9218f 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1513,8 +1513,7 @@ class CreateContext:
         instance_data = {
             "asset": asset_doc["name"],
             "task": task_name,
-            "variant": variant,
-            "family": family
+            "variant": variant
         }
         return self.raw_create(
             creator_identifier,

From daa961d24976a4b9a1d5f51320017fa346bbfc84 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Mon, 30 Jan 2023 10:49:27 +0100
Subject: [PATCH 10/19] variant is required argument

---
 openpype/pipeline/create/context.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index dbbde9218f..b10bbc17de 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1463,7 +1463,7 @@ class CreateContext:
     def create(
         self,
         creator_identifier,
-        variant=None,
+        variant,
         asset_doc=None,
         task_name=None,
         pre_create_data=None

From 4d990e6f87964cb4f5fb2c61cebfcaea47ac3151 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Mon, 30 Jan 2023 11:03:51 +0100
Subject: [PATCH 11/19] Updated docstrings

---
 openpype/pipeline/create/context.py | 24 ++++++++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index b10bbc17de..190d542724 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1191,7 +1191,15 @@ class CreateContext:
         function or using `legacy_io.Session`.
 
         Some hosts have ability to change context file without using workfiles
-        tool but that change is not propagated to
+        tool but that change is not propagated to 'legacy_io.Session'
+        nor 'os.environ'.
+
+        Todos:
+            UI: Current context should be also checked on save - compare
+                initial values vs. current values.
+            Related to UI checks: Current workfile can be also considered
+                as current context information as that's where the metadata
+                are stored. We should store the workfile (if is available) too.
         """
 
         project_name = asset_name = task_name = None
@@ -1468,12 +1476,19 @@ class CreateContext:
         task_name=None,
         pre_create_data=None
     ):
-        """Trigger create of plugins with standartized
+        """Trigger create of plugins with standartized arguments.
+
+        Arguments 'asset_doc' and 'task_name' use current context as default
+        values. If only 'task_name' is provided it will be overriden by
+        task name from current context. If 'task_name' is not provided
+        when 'asset_doc' is, it is considered that task name is not specified,
+        which can lead to error if subset name template requires task name.
 
         Args:
-            creator_identifier (str):
+            creator_identifier (str): Identifier of creator plugin.
             variant (str): Variant used for subset name.
-            asset_doc (Dict[str, Any]):
+            asset_doc (Dict[str, Any]): Asset document which define context of
+                creation (possible context of created instance/s).
             task_name (str): Name of task to which is context related.
             pre_create_data (Dict[str, Any]): Pre-create attribute values.
 
@@ -1481,6 +1496,7 @@ class CreateContext:
             Any: Output of triggered creator's 'create' method.
 
         Raises:
+            CreatorError: If creator was not found or asset is empty.
             CreatorsCreateFailed: When creation fails.
         """
 

From 15d7a7589fa119010510232e3ae243f0e6834383 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 9 Feb 2023 12:04:06 +0100
Subject: [PATCH 12/19] fix context collection from create context

---
 .../publish/collect_from_create_context.py       | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/openpype/plugins/publish/collect_from_create_context.py b/openpype/plugins/publish/collect_from_create_context.py
index d3398c885e..5fcf8feb56 100644
--- a/openpype/plugins/publish/collect_from_create_context.py
+++ b/openpype/plugins/publish/collect_from_create_context.py
@@ -32,7 +32,7 @@ class CollectFromCreateContext(pyblish.api.ContextPlugin):
             thumbnail_paths_by_instance_id.get(None)
         )
 
-        project_name = create_context.project_name
+        project_name = create_context.get_current_project_name()
         if project_name:
             context.data["projectName"] = project_name
 
@@ -53,11 +53,15 @@ class CollectFromCreateContext(pyblish.api.ContextPlugin):
         context.data.update(create_context.context_data_to_store())
         context.data["newPublishing"] = True
         # Update context data
-        for key in ("AVALON_PROJECT", "AVALON_ASSET", "AVALON_TASK"):
-            value = create_context.dbcon.Session.get(key)
-            if value is not None:
-                legacy_io.Session[key] = value
-                os.environ[key] = value
+        asset_name = create_context.get_current_asset_name()
+        task_name = create_context.get_current_task_name()
+        for key, value in (
+            ("AVALON_PROJECT", project_name),
+            ("AVALON_ASSET", asset_name),
+            ("AVALON_TASK", task_name)
+        ):
+            legacy_io.Session[key] = value
+            os.environ[key] = value
 
     def create_instance(
         self,

From b148dec04843137622960b50c484d94ad9b0e82b Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 9 Feb 2023 12:58:17 +0100
Subject: [PATCH 13/19] Added helper method to return unified information on
 create error

---
 openpype/pipeline/create/context.py | 70 +++++++++++++++++------------
 1 file changed, 41 insertions(+), 29 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 190d542724..dfe60d438b 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1538,6 +1538,44 @@ class CreateContext:
             pre_create_data
         )
 
+    def _create_with_unified_error(
+        self, identifier, creator, *args, **kwargs
+    ):
+        error_message = "Failed to run Creator with identifier \"{}\". {}"
+
+        label = None
+        add_traceback = False
+        result = None
+        fail_info = None
+        success = False
+
+        try:
+            # Try to get creator and his label
+            if creator is None:
+                creator = self._get_creator_in_create(identifier)
+            label = getattr(creator, "label", label)
+
+            # Run create
+            result = creator.create(*args, **kwargs)
+            success = True
+
+        except CreatorError:
+            exc_info = sys.exc_info()
+            self.log.warning(error_message.format(identifier, exc_info[1]))
+
+        except:
+            add_traceback = True
+            exc_info = sys.exc_info()
+            self.log.warning(
+                error_message.format(identifier, ""),
+                exc_info=True
+            )
+
+        if not success:
+            fail_info = prepare_failed_creator_operation_info(
+                identifier, label, exc_info, add_traceback
+            )
+        return result, fail_info
     def creator_removed_instance(self, instance):
         """When creator removes instance context should be acknowledged.
 
@@ -1663,37 +1701,11 @@ class CreateContext:
         Reset instances if any autocreator executed properly.
         """
 
-        error_message = "Failed to run AutoCreator with identifier \"{}\". {}"
         failed_info = []
         for identifier, creator in self.autocreators.items():
-            label = creator.label
-            failed = False
-            add_traceback = False
-            try:
-                creator.create()
-
-            except CreatorError:
-                failed = True
-                exc_info = sys.exc_info()
-                self.log.warning(error_message.format(identifier, exc_info[1]))
-
-            # Use bare except because some hosts raise their exceptions that
-            #   do not inherit from python's `BaseException`
-            except:
-                failed = True
-                add_traceback = True
-                exc_info = sys.exc_info()
-                self.log.warning(
-                    error_message.format(identifier, ""),
-                    exc_info=True
-                )
-
-            if failed:
-                failed_info.append(
-                    prepare_failed_creator_operation_info(
-                        identifier, label, exc_info, add_traceback
-                    )
-                )
+            _, fail_info = self._create_with_unified_error(identifier, creator)
+            if fail_info is not None:
+                failed_info.append(fail_info)
 
         if failed_info:
             raise CreatorsCreateFailed(failed_info)

From fac10d26337157bc253a1af90ae0a1040ff49533 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 9 Feb 2023 12:58:51 +0100
Subject: [PATCH 14/19] added public method 'create_with_unified_error' used in
 publisher

---
 openpype/pipeline/create/context.py | 25 +++++++++++++++++++++++++
 openpype/tools/publisher/control.py |  3 ++-
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index dfe60d438b..2a92d21225 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1576,6 +1576,31 @@ class CreateContext:
                 identifier, label, exc_info, add_traceback
             )
         return result, fail_info
+
+    def create_with_unified_error(self, identifier, *args, **kwargs):
+        """Trigger create but raise only one error if anything fails.
+
+        Added to raise unified exception. Capture any possible issues and
+        reraise it with unified information.
+
+        Args:
+            identifier (str): Identifier of creator.
+            *args (Tuple[Any]): Arguments for create method.
+            **kwargs (Dict[Any, Any]): Keyword argument for create method.
+
+        Raises:
+            CreatorsCreateFailed: When creation fails due to any possible
+                reason. If anything goes wrong this is only possible exception
+                the method should raise.
+        """
+
+        result, fail_info = self._create_with_unified_error(
+            identifier, None, *args, **kwargs
+        )
+        if fail_info is not None:
+            raise CreatorsCreateFailed([fail_info])
+        return result
+
     def creator_removed_instance(self, instance):
         """When creator removes instance context should be acknowledged.
 
diff --git a/openpype/tools/publisher/control.py b/openpype/tools/publisher/control.py
index 670c22a43e..11215b5ff8 100644
--- a/openpype/tools/publisher/control.py
+++ b/openpype/tools/publisher/control.py
@@ -2017,9 +2017,10 @@ class PublisherController(BasePublisherController):
 
         success = True
         try:
-            self._create_context.raw_create(
+            self._create_context.create_with_unified_error(
                 creator_identifier, subset_name, instance_data, options
             )
+
         except CreatorsOperationFailed as exc:
             success = False
             self._emit_event(

From cb84cf769eea9d5a018ef67dd6e4cb0d6d7276b8 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 9 Feb 2023 13:00:28 +0100
Subject: [PATCH 15/19] 'create' method is not triggering 'raw_create'

---
 openpype/pipeline/create/context.py | 34 ++++++++++++++++++++++-------
 1 file changed, 26 insertions(+), 8 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 2a92d21225..3287141970 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1415,6 +1415,30 @@ class CreateContext:
         with self.bulk_instances_collection():
             self._bulk_instances_to_process.append(instance)
 
+    def _get_creator_in_create(self, identifier):
+        """Creator by identifier with unified error.
+
+        Helper method to get creator by identifier with same error when creator
+        is not available.
+
+        Args:
+            identifier (str): Identifier of creator plugin.
+
+        Returns:
+            BaseCreator: Creator found by identifier.
+
+        Raises:
+            CreatorError: When identifier is not known.
+        """
+
+        creator = self.creators.get(identifier)
+        # Fake CreatorError (Could be maybe specific exception?)
+        if creator is None:
+            raise CreatorError(
+                "Creator {} was not found".format(identifier)
+            )
+        return creator
+
     def raw_create(self, identifier, *args, **kwargs):
         """Wrapper for creators to trigger 'create' method.
 
@@ -1497,14 +1521,9 @@ class CreateContext:
 
         Raises:
             CreatorError: If creator was not found or asset is empty.
-            CreatorsCreateFailed: When creation fails.
         """
 
-        creator = self.creators.get(creator_identifier)
-        if creator is None:
-            raise CreatorError(
-                "Creator {} was not found".format(creator_identifier)
-            )
+        creator = self._get_creator_in_create(creator_identifier)
 
         project_name = self.project_name
         if asset_doc is None:
@@ -1531,8 +1550,7 @@ class CreateContext:
             "task": task_name,
             "variant": variant
         }
-        return self.raw_create(
-            creator_identifier,
+        return creator.create(
             subset_name,
             instance_data,
             pre_create_data

From 0cb78a10e6cd7b0c7307c70be1827e2e8c1d1f2e Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 9 Feb 2023 13:00:49 +0100
Subject: [PATCH 16/19] removed unused 'raw_create' method

---
 openpype/pipeline/create/context.py | 53 -----------------------------
 1 file changed, 53 deletions(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 3287141970..078c50acc2 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1439,59 +1439,6 @@ class CreateContext:
             )
         return creator
 
-    def raw_create(self, identifier, *args, **kwargs):
-        """Wrapper for creators to trigger 'create' method.
-
-        Different types of creators may expect different arguments thus the
-        hints for args are blind.
-
-        Args:
-            identifier (str): Creator's identifier.
-            *args (Tuple[Any]): Arguments for create method.
-            **kwargs (Dict[Any, Any]): Keyword argument for create method.
-
-        Raises:
-            CreatorsCreateFailed: When creation fails.
-        """
-
-        error_message = "Failed to run Creator with identifier \"{}\". {}"
-        creator = self.creators.get(identifier)
-        label = getattr(creator, "label", None)
-        failed = False
-        add_traceback = False
-        exc_info = None
-        result = None
-        try:
-            # Fake CreatorError (Could be maybe specific exception?)
-            if creator is None:
-                raise CreatorError(
-                    "Creator {} was not found".format(identifier)
-                )
-
-            result = creator.create(*args, **kwargs)
-
-        except CreatorError:
-            failed = True
-            exc_info = sys.exc_info()
-            self.log.warning(error_message.format(identifier, exc_info[1]))
-
-        except:
-            failed = True
-            add_traceback = True
-            exc_info = sys.exc_info()
-            self.log.warning(
-                error_message.format(identifier, ""),
-                exc_info=True
-            )
-
-        if failed:
-            raise CreatorsCreateFailed([
-                prepare_failed_creator_operation_info(
-                    identifier, label, exc_info, add_traceback
-                )
-            ])
-        return result
-
     def create(
         self,
         creator_identifier,

From 5903dbce9e176d26d45681a86efe92f607a512e2 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Tue, 14 Feb 2023 17:03:05 +0100
Subject: [PATCH 17/19] autofill precreate attributes if are not passed

---
 openpype/pipeline/create/context.py | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index ba566f93d4..1567acdb79 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -17,6 +17,7 @@ from openpype.lib.attribute_definitions import (
     UnknownDef,
     serialize_attr_defs,
     deserialize_attr_defs,
+    get_default_values,
 )
 from openpype.host import IPublishHost
 from openpype.pipeline import legacy_io
@@ -1866,6 +1867,13 @@ class CreateContext:
         if pre_create_data is None:
             pre_create_data = {}
 
+        precreate_attr_defs = creator.get_pre_create_attr_defs() or []
+        # Create default values of precreate data
+        _pre_create_data = get_default_values(precreate_attr_defs)
+        # Update passed precreate data to default values
+        # TODO validate types
+        _pre_create_data.update(pre_create_data)
+
         subset_name = creator.get_subset_name(
             variant,
             task_name,
@@ -1881,7 +1889,7 @@ class CreateContext:
         return creator.create(
             subset_name,
             instance_data,
-            pre_create_data
+            _pre_create_data
         )
 
     def _create_with_unified_error(

From df532268a2e05b6b48074336453ab3e18b86e08f Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Wed, 15 Feb 2023 12:00:15 +0100
Subject: [PATCH 18/19] add family to instance data

---
 openpype/pipeline/create/context.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 1567acdb79..79c9805604 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1884,6 +1884,7 @@ class CreateContext:
         instance_data = {
             "asset": asset_doc["name"],
             "task": task_name,
+            "family": self.family,
             "variant": variant
         }
         return creator.create(

From fb93780640ed5de588cb11499ccd68d1f6a91d75 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Wed, 15 Feb 2023 12:08:45 +0100
Subject: [PATCH 19/19] use family form creator

---
 openpype/pipeline/create/context.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/pipeline/create/context.py b/openpype/pipeline/create/context.py
index 79c9805604..89eec52676 100644
--- a/openpype/pipeline/create/context.py
+++ b/openpype/pipeline/create/context.py
@@ -1884,7 +1884,7 @@ class CreateContext:
         instance_data = {
             "asset": asset_doc["name"],
             "task": task_name,
-            "family": self.family,
+            "family": creator.family,
             "variant": variant
         }
         return creator.create(