Added error handling for StringEditorControl

Organized the code so that the signaling of the errors happens in the base PropertyEditorControl class. Each subclass can override how errors can be displayed. This can be moved to the base class later on, depending on how we want to display errors in the future. This means we will most likely have to convert PropertyEditorControl back from being an Abstract class, and make its methods virtual.

Right now to display an error, the editor turns red and all relevant errors are outputted to the console. Once we decide what we want our error-reporting system to look like, we can modifiy what we have now to be that.

1 files changed, 12 insertions, 0 deletions

diff --git a/Xamarin.PropertyEditing.Mac/Controls/PropertyEditorControl.cs b/Xamarin.PropertyEditing.Mac/Controls/PropertyEditorControl.cs
index 853b089..d0cff6d 100644
--- a/Xamarin.PropertyEditing.Mac/Controls/PropertyEditorControl.cs
+++ b/Xamarin.PropertyEditing.Mac/Controls/PropertyEditorControl.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Collections;
 using AppKit;
 using Xamarin.PropertyEditing.ViewModels;
 
@@ -25,11 +26,22 @@ namespace Xamarin.PropertyEditing.Mac
 				viewModel = value;
 				UpdateModelValue ();
 				viewModel.PropertyChanged += HandlePropertyChanged;
+
+				// FIXME: figure out what we want errors to display as (tooltip, etc.)
+				viewModel.ErrorsChanged += (object sender, System.ComponentModel.DataErrorsChangedEventArgs e) => {
+					UpdateErrorsDisplayed (viewModel.GetErrors (viewModel.Property.Name));
+				};
 			}
 		}
 
 		protected abstract void UpdateModelValue ();
 
 		protected abstract void HandlePropertyChanged (object sender, System.ComponentModel.PropertyChangedEventArgs e);
+
+		/// <summary>
+		/// Update the display with any errors we need to show or remove
+		/// </summary>
+		/// <param name="errors">The error messages to display to the user</param>
+		protected abstract void UpdateErrorsDisplayed (IEnumerable errors);
 	}
 }