Move TriValidationResult to runtime assembly

Editor.Extras/Drawers/InlineEditorDrawer.cs

@@ -19,7 +19,7 @@ namespace TriInspector.Drawers
             {
                 var stack = new TriElement();
                 stack.AddChild(new TriInfoBoxElement($"InlineEditor valid only on Object fields",
-                    MessageType.Error));
+                    TriMessageType.Error));
                 stack.AddChild(next);
 
                 return stack;

Editor/Elements/TriInfoBoxElement.cs

@@ -9,9 +9,10 @@ namespace TriInspector.Elements
         private readonly GUIContent _message;
         private readonly Color _color;
 
-        public TriInfoBoxElement(string message, MessageType type = MessageType.None, Color? color = null)
+        public TriInfoBoxElement(string message, TriMessageType type = TriMessageType.None, Color? color = null)
         {
-            _message = new GUIContent(message, EditorGUIUtilityProxy.GetHelpIcon(type));
+            var messageType = GetMessageType(type);
+            _message = new GUIContent(message, EditorGUIUtilityProxy.GetHelpIcon(messageType));
             _color = color ?? GetColor(type);
         }
 
@@ -30,14 +31,14 @@ namespace TriInspector.Elements
             GUI.Label(position, _message, Styles.InfoBoxContent);
         }
 
-        private static Color GetColor(MessageType type)
+        private static Color GetColor(TriMessageType type)
         {
             switch (type)
             {
-                case MessageType.Error:
+                case TriMessageType.Error:
                     return new Color(1f, 0.4f, 0.4f);
 
-                case MessageType.Warning:
+                case TriMessageType.Warning:
                     return new Color(1f, 0.8f, 0.2f);
 
                 default:
@@ -45,6 +46,18 @@ namespace TriInspector.Elements
             }
         }
 
+        private static MessageType GetMessageType(TriMessageType type)
+        {
+            switch (type)
+            {
+                case TriMessageType.None: return MessageType.None;
+                case TriMessageType.Info: return MessageType.Info;
+                case TriMessageType.Warning: return MessageType.Warning;
+                case TriMessageType.Error: return MessageType.Error;
+                default: return MessageType.None;
+            }
+        }
+
         private static class Styles
         {
             public static readonly GUIStyle InfoBoxBg;

Editor/TriProperty.cs

@@ -13,7 +13,7 @@ namespace TriInspector
     {
         private static readonly IReadOnlyList<TriValidationResult> EmptyValidationResults =
             new List<TriValidationResult>();
-        
+
         private readonly TriPropertyDefinition _definition;
         private readonly int _propertyIndex;
         private readonly ITriPropertyParent _parent;
@@ -43,7 +43,7 @@ namespace TriInspector
 
         [PublicAPI]
         public string DisplayName => DisplayNameContent.text;
-        
+
         [PublicAPI]
         public GUIContent DisplayNameContent
         {
@@ -231,7 +231,7 @@ namespace TriInspector
             // actualize
             PropertyTree.SerializedObject.Update();
             Update();
-            
+
             PropertyTree.RequestValidation();
         }
 
@@ -325,10 +325,10 @@ namespace TriInspector
             {
                 _validationResults = _definition.Validators
                     .Select(it => it.Validate(this))
-                    .Where(it => it.MessageType != MessageType.None)
+                    .Where(it => !it.IsValid)
                     .ToList();
             }
-            
+
             if (_childrenProperties != null)
             {
                 foreach (var childrenProperty in _childrenProperties)

Editor/TriValidator.cs

@@ -38,28 +38,4 @@ namespace TriInspector
         [PublicAPI]
         public abstract TriValidationResult Validate(TriValue<T> propertyValue);
     }
-
-    public readonly struct TriValidationResult
-    {
-        public static TriValidationResult Valid => new TriValidationResult(null, MessageType.None);
-
-        private TriValidationResult(string message, MessageType messageType)
-        {
-            Message = message;
-            MessageType = messageType;
-        }
-
-        public string Message { get; }
-        public MessageType MessageType { get; }
-
-        public static TriValidationResult Error(string error)
-        {
-            return new TriValidationResult(error, MessageType.Error);
-        }
-
-        public static TriValidationResult Warning(string error)
-        {
-            return new TriValidationResult(error, MessageType.Warning);
-        }
-    }
 }

Runtime/TriValidationResult.cs

@@ -0,0 +1,36 @@
+namespace TriInspector
+{
+    public readonly struct TriValidationResult
+    {
+        public static TriValidationResult Valid => new TriValidationResult(true, null, TriMessageType.None);
+
+        private TriValidationResult(bool valid, string message, TriMessageType messageType)
+        {
+            IsValid = valid;
+            Message = message;
+            MessageType = messageType;
+        }
+
+        public bool IsValid { get; }
+        public string Message { get; }
+        public TriMessageType MessageType { get; }
+
+        public static TriValidationResult Error(string error)
+        {
+            return new TriValidationResult(false, error, TriMessageType.Error);
+        }
+
+        public static TriValidationResult Warning(string error)
+        {
+            return new TriValidationResult(false, error, TriMessageType.Warning);
+        }
+    }
+
+    public enum TriMessageType
+    {
+        None,
+        Info,
+        Warning,
+        Error,
+    }
+}

Runtime/TriValidationResult.cs.meta

@@ -0,0 +1,3 @@
+fileFormatVersion: 2
+guid: 428f8881a0d34edf8148591af6f9a0e7
+timeCreated: 1642525992