fix(extensions): 2 trigger classes implementing the same interface not being picked up correctly by AddAssemblyTriggers

Carl-Johan Blomqvist · Carl-Johan Blomqvist · commit c646709c5d38 · 2022-10-21T13:30:46.000+02:00
diff --git a/src/EntityFrameworkCore.Triggered.Extensions/ServiceCollectionExtensions.cs b/src/EntityFrameworkCore.Triggered.Extensions/ServiceCollectionExtensions.cs
@@ -105,7 +105,7 @@ public static IServiceCollection AddAssemblyTriggers(this IServiceCollection ser
                         registered = true;
                     }
 
-                    services.TryAdd(new ServiceDescriptor(triggerType, sp => sp.GetRequiredService(assemblyType), ServiceLifetime.Transient));
+                    services.Add(new ServiceDescriptor(triggerType, sp => sp.GetRequiredService(assemblyType), ServiceLifetime.Transient));
                 }
             }
 

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ public static IServiceCollection AddAssemblyTriggers(this IServiceCollection ser`
`105`	`105`	`registered = true;`
`106`	`106`	`}`
`107`	`107`
`108`		`- services.TryAdd(new ServiceDescriptor(triggerType, sp => sp.GetRequiredService(assemblyType), ServiceLifetime.Transient));`
	`108`	`+ services.Add(new ServiceDescriptor(triggerType, sp => sp.GetRequiredService(assemblyType), ServiceLifetime.Transient));`
`109`	`109`	`}`
`110`	`110`	`}`
`111`	`111`