throw for EventInvokerNames contained no items

Fody · Apr 9, 2018 · 689c7b2 · 689c7b2
1 parent c621d60
commit 689c7b2
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/Directory.Build.props b/Directory.Build.props
@@ -2,7 +2,7 @@
 <Project>
   <PropertyGroup>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
-    <Version>2.4.1</Version>
+    <Version>2.5.0</Version>
     <LangVersion>latest</LangVersion>
   </PropertyGroup>
-</Project>
+</Project>
diff --git a/PropertyChanged.Fody/EventInvokerNameResolver.cs b/PropertyChanged.Fody/EventInvokerNameResolver.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using Fody;
 
 public partial class ModuleWeaver
 {
@@ -18,10 +19,18 @@ public partial class ModuleWeaver
     public void ResolveEventInvokerName()
     {
         var eventInvokerAttribute = Config?.Attributes("EventInvokerNames").FirstOrDefault();
-        if (eventInvokerAttribute != null)
+        if (eventInvokerAttribute == null)
         {
-            EventInvokerNames.Clear();
-            EventInvokerNames.AddRange(eventInvokerAttribute.Value.Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries).Select(x => x.Trim()).Where(x => x.Length > 0));
+            return;
+        }
+
+        EventInvokerNames = eventInvokerAttribute.Value.Split(new[] {','}, StringSplitOptions.RemoveEmptyEntries)
+            .Select(x => x.Trim())
+            .Where(x => x.Length > 0)
+            .ToList();
+        if (!EventInvokerNames.Any())
+        {
+            throw new WeavingException("EventInvokerNames contained no items.");
         }
     }
 }