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

Carl-Johan Blomqvist · koenbeuk · commit b30af9042314 · 2022-10-24T12:27:25.000+01:00
diff --git a/src/EntityFrameworkCore.Triggered.Extensions/ServiceCollectionExtensions.cs b/src/EntityFrameworkCore.Triggered.Extensions/ServiceCollectionExtensions.cs
@@ -96,7 +96,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
`@@ -96,7 +96,7 @@ public static IServiceCollection AddAssemblyTriggers(this IServiceCollection ser`
`96`	`96`	`registered = true;`
`97`	`97`	`}`
`98`	`98`
`99`		`- services.TryAdd(new ServiceDescriptor(triggerType, sp => sp.GetRequiredService(assemblyType), ServiceLifetime.Transient));`
	`99`	`+ services.Add(new ServiceDescriptor(triggerType, sp => sp.GetRequiredService(assemblyType), ServiceLifetime.Transient));`
`100`	`100`	`}`
`101`	`101`	`}`
`102`	`102`