From 70bcd6bf9062df6bb72948b02b3344c153f242fc Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Tue, 9 Aug 2022 18:04:48 +0200
Subject: [PATCH] moved start and stop plugins into timers manager

---
 .../plugins/publish/start_timer.py            | 39 +++++++++++++++++++
 .../plugins/publish/stop_timer.py             | 27 +++++++++++++
 openpype/plugins/publish/start_timer.py       | 14 -------
 openpype/plugins/publish/stop_timer.py        | 17 --------
 4 files changed, 66 insertions(+), 31 deletions(-)
 create mode 100644 openpype/modules/timers_manager/plugins/publish/start_timer.py
 create mode 100644 openpype/modules/timers_manager/plugins/publish/stop_timer.py
 delete mode 100644 openpype/plugins/publish/start_timer.py
 delete mode 100644 openpype/plugins/publish/stop_timer.py

diff --git a/openpype/modules/timers_manager/plugins/publish/start_timer.py b/openpype/modules/timers_manager/plugins/publish/start_timer.py
new file mode 100644
index 0000000000..6408327ca1
--- /dev/null
+++ b/openpype/modules/timers_manager/plugins/publish/start_timer.py
@@ -0,0 +1,39 @@
+"""
+Requires:
+    context -> system_settings
+    context -> openPypeModules
+"""
+
+import pyblish.api
+
+from openpype.pipeline import legacy_io
+
+
+class StartTimer(pyblish.api.ContextPlugin):
+    label = "Start Timer"
+    order = pyblish.api.IntegratorOrder + 1
+    hosts = ["*"]
+
+    def process(self, context):
+        timers_manager = context.data["openPypeModules"]["timers_manager"]
+        if not timers_manager.enabled:
+            self.log.debug("TimersManager is disabled")
+            return
+
+        modules_settings = context.data["system_settings"]["modules"]
+        if not modules_settings["timers_manager"]["disregard_publishing"]:
+            self.log.debug("Publish is not affecting running timers.")
+            return
+
+        project_name = legacy_io.active_project()
+        asset_name = legacy_io.Session.get("AVALON_ASSET")
+        task_name = legacy_io.Session.get("AVALON_TASK")
+        if not project_name or not asset_name or not task_name:
+            self.log.info((
+                "Current context does not contain all"
+                " required information to start a timer."
+            ))
+            return
+        timers_manager.start_timer_with_webserver(
+            project_name, asset_name, task_name, self.log
+        )
diff --git a/openpype/modules/timers_manager/plugins/publish/stop_timer.py b/openpype/modules/timers_manager/plugins/publish/stop_timer.py
new file mode 100644
index 0000000000..a8674ff2ca
--- /dev/null
+++ b/openpype/modules/timers_manager/plugins/publish/stop_timer.py
@@ -0,0 +1,27 @@
+"""
+Requires:
+    context -> system_settings
+    context -> openPypeModules
+"""
+
+
+import pyblish.api
+
+
+class StopTimer(pyblish.api.ContextPlugin):
+    label = "Stop Timer"
+    order = pyblish.api.ExtractorOrder - 0.49
+    hosts = ["*"]
+
+    def process(self, context):
+        timers_manager = context.data["openPypeModules"]["timers_manager"]
+        if not timers_manager.enabled:
+            self.log.debug("TimersManager is disabled")
+            return
+
+        modules_settings = context.data["system_settings"]["modules"]
+        if not modules_settings["timers_manager"]["disregard_publishing"]:
+            self.log.debug("Publish is not affecting running timers.")
+            return
+
+        timers_manager.stop_timer_with_webserver(self.log)
diff --git a/openpype/plugins/publish/start_timer.py b/openpype/plugins/publish/start_timer.py
deleted file mode 100644
index 112d92bef0..0000000000
--- a/openpype/plugins/publish/start_timer.py
+++ /dev/null
@@ -1,14 +0,0 @@
-import pyblish.api
-
-from openpype.lib import change_timer_to_current_context
-
-
-class StartTimer(pyblish.api.ContextPlugin):
-    label = "Start Timer"
-    order = pyblish.api.IntegratorOrder + 1
-    hosts = ["*"]
-
-    def process(self, context):
-        modules_settings = context.data["system_settings"]["modules"]
-        if modules_settings["timers_manager"]["disregard_publishing"]:
-            change_timer_to_current_context()
diff --git a/openpype/plugins/publish/stop_timer.py b/openpype/plugins/publish/stop_timer.py
deleted file mode 100644
index 414e43a3c4..0000000000
--- a/openpype/plugins/publish/stop_timer.py
+++ /dev/null
@@ -1,17 +0,0 @@
-import os
-import requests
-
-import pyblish.api
-
-
-class StopTimer(pyblish.api.ContextPlugin):
-    label = "Stop Timer"
-    order = pyblish.api.ExtractorOrder - 0.49
-    hosts = ["*"]
-
-    def process(self, context):
-        modules_settings = context.data["system_settings"]["modules"]
-        if modules_settings["timers_manager"]["disregard_publishing"]:
-            webserver_url = os.environ.get("OPENPYPE_WEBSERVER_URL")
-            rest_api_url = "{}/timers_manager/stop_timer".format(webserver_url)
-            requests.post(rest_api_url)