feat(error): make some properties on SimpleError public (#128)

CHANGELOG.md

@@ -12,6 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 
 ### Changed
+- Made a number of properties on `SimpleError` public (#128)
 
 ### Removed
 

Editor/ErrorReporting/InlineError.cs

@@ -61,12 +61,12 @@ private void AddContext(IEnumerable args, List<string> substitutions)
             }
         }
 
-        protected override Localizer Localizer { get; }
+        public override Localizer Localizer { get; }
         public override ErrorSeverity Severity { get; }
-        protected override string TitleKey { get; }
+        public override string TitleKey { get; }
 
-        protected override string[] TitleSubst => _subst;
-        protected override string[] DetailsSubst => _subst;
-        protected override string[] HintSubst => _subst;
+        public override string[] TitleSubst => _subst;
+        public override string[] DetailsSubst => _subst;
+        public override string[] HintSubst => _subst;
     }
 }

Editor/ErrorReporting/SimpleError.cs

@@ -19,46 +19,46 @@ public abstract class SimpleError : IError
         /// <summary>
         /// The Localizer to use to look up strings.
         /// </summary>
-        protected abstract Localizer Localizer { get; }
+        public abstract Localizer Localizer { get; }
 
         /// <summary>
         /// The key to use for the title of the error. By default, all other keys are derived from this TitleKey.
         /// </summary>
-        protected abstract string TitleKey { get; }
+        public abstract string TitleKey { get; }
 
         /// <summary>
         /// String substitutions to insert into the title of the error display. You can reference these with
         /// e.g. `{0}`.
         /// </summary>
-        protected virtual string[] TitleSubst => Array.Empty<string>();
+        public virtual string[] TitleSubst => Array.Empty<string>();
 
         /// <summary>
         /// The key to use for the details section of the error display. By default, this is the TitleKey + `:description`.
         /// </summary>
-        protected virtual string DetailsKey => TitleKey + ":description";
+        public virtual string DetailsKey => TitleKey + ":description";
 
         /// <summary>
         /// String substitutions to insert into the details section of the error display. You can reference these with
         /// e.g. `{0}`.
         /// </summary>
-        protected virtual string[] DetailsSubst => Array.Empty<string>();
+        public virtual string[] DetailsSubst => Array.Empty<string>();
 
         /// <summary>
         /// The key to use for the hint section of the error display. By default, this is the TitleKey + `:hint`.
         /// This section should be used to provide a hint to the user about how to resolve the error.
         /// </summary>
-        protected virtual string HintKey => TitleKey + ":hint";
+        public virtual string HintKey => TitleKey + ":hint";
 
         /// <summary>
         /// String substitutions to insert into the hint section of the error display. You can reference these with
         /// e.g. `{0}`.
         /// </summary>
-        protected virtual string[] HintSubst => Array.Empty<string>();
+        public virtual string[] HintSubst => Array.Empty<string>();
 
         /// <summary>
         /// Any ObjectReferences to display to the user; the user will be able to click to jump to these objects.
         /// </summary>
-        protected List<ObjectReference> _references = new List<ObjectReference>();
+        public List<ObjectReference> _references = new List<ObjectReference>();
 
         /// <summary>
         /// Any ObjectReferences to display to the user; the user will be able to click to jump to these objects.

Editor/ErrorReporting/StackTraceError.cs

@@ -25,11 +25,11 @@ public StackTraceError(Exception e, string additionalStackTrace = null)
             }
         }
 
-        protected override Localizer Localizer => NDMFLocales.L;
-        protected override string TitleKey => "Errors:InternalError";
+        public override Localizer Localizer => NDMFLocales.L;
+        public override string TitleKey => "Errors:InternalError";
         public override ErrorSeverity Severity => ErrorSeverity.InternalError;
 
-        protected override string[] DetailsSubst => new []
+        public override string[] DetailsSubst => new []
         {
             _e.GetType().Name
         };