From df9e30eb7f8074d77a1701cc6bf2470ad82df487 Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Wed, 22 Dec 2021 13:33:11 +0100
Subject: [PATCH] raise PublishXmlValidationError in validate start frame

---
 .../plugins/publish/help/validate_start_frame.xml  | 14 ++++++++++++++
 .../plugins/publish/validate_start_frame.py        | 12 +++++++++++-
 2 files changed, 25 insertions(+), 1 deletion(-)
 create mode 100644 openpype/hosts/tvpaint/plugins/publish/help/validate_start_frame.xml

diff --git a/openpype/hosts/tvpaint/plugins/publish/help/validate_start_frame.xml b/openpype/hosts/tvpaint/plugins/publish/help/validate_start_frame.xml
new file mode 100644
index 0000000000..9052abf66c
--- /dev/null
+++ b/openpype/hosts/tvpaint/plugins/publish/help/validate_start_frame.xml
@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<root>
+<error id="main">
+<title>First frame</title>
+<description>## MarkIn is not set to 0
+
+MarkIn in your scene must start from 0 fram index but MarkIn is set to {current_start_frame}.
+
+### How to repair?
+
+You can modify MarkIn manually or hit the "Repair" button on the right which will change MarkIn to 0 (does not change MarkOut).
+</description>
+</error>
+</root>
diff --git a/openpype/hosts/tvpaint/plugins/publish/validate_start_frame.py b/openpype/hosts/tvpaint/plugins/publish/validate_start_frame.py
index d769d47736..48efd91055 100644
--- a/openpype/hosts/tvpaint/plugins/publish/validate_start_frame.py
+++ b/openpype/hosts/tvpaint/plugins/publish/validate_start_frame.py
@@ -1,5 +1,6 @@
 import pyblish.api
 from avalon.tvpaint import lib
+from openpype.pipeline import PublishXmlValidationError
 
 
 class RepairStartFrame(pyblish.api.Action):
@@ -24,4 +25,13 @@ class ValidateStartFrame(pyblish.api.ContextPlugin):
 
     def process(self, context):
         start_frame = lib.execute_george("tv_startframe")
-        assert int(start_frame) == 0, "Start frame has to be frame 0."
+        if start_frame == 0:
+            return
+
+        raise PublishXmlValidationError(
+            self,
+            "Start frame has to be frame 0.",
+            formatting_data={
+                "current_start_frame": start_frame
+            }
+        )