From bc86cd279c40933c6f9ed84a5724bd2997df1196 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Wed, 23 Feb 2022 18:47:09 +0100
Subject: [PATCH] showing report does not use publishing logic

---
 openpype/tools/publisher/control.py | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/openpype/tools/publisher/control.py b/openpype/tools/publisher/control.py
index 04158ad05e..5a84b1d8ca 100644
--- a/openpype/tools/publisher/control.py
+++ b/openpype/tools/publisher/control.py
@@ -184,11 +184,21 @@ class PublishReport:
 
         self._stored_plugins.append(plugin)
 
+        plugin_data_item = self._create_plugin_data_item(plugin)
+
+        self._plugin_data_with_plugin.append({
+            "plugin": plugin,
+            "data": plugin_data_item
+        })
+        self._plugin_data.append(plugin_data_item)
+        return plugin_data_item
+
+    def _create_plugin_data_item(self, plugin):
         label = None
         if hasattr(plugin, "label"):
             label = plugin.label
 
-        plugin_data_item = {
+        return {
             "name": plugin.__name__,
             "label": label,
             "order": plugin.order,
@@ -197,12 +207,6 @@ class PublishReport:
             "skipped": False,
             "passed": False
         }
-        self._plugin_data_with_plugin.append({
-            "plugin": plugin,
-            "data": plugin_data_item
-        })
-        self._plugin_data.append(plugin_data_item)
-        return plugin_data_item
 
     def set_plugin_skipped(self):
         """Set that current plugin has been skipped."""
@@ -252,7 +256,7 @@ class PublishReport:
         if publish_plugins:
             for plugin in publish_plugins:
                 if plugin not in self._stored_plugins:
-                    plugins_data.append(self._add_plugin_data_item(plugin))
+                    plugins_data.append(self._create_plugin_data_item(plugin))
 
         crashed_file_paths = {}
         if self._publish_discover_result is not None: