diff --git a/src/Microsoft.Extensions.WebEncoders/EncoderServiceCollectionExtensions.cs b/src/Microsoft.Extensions.WebEncoders/EncoderServiceCollectionExtensions.cs index df3946099c..317d88f99d 100644 --- a/src/Microsoft.Extensions.WebEncoders/EncoderServiceCollectionExtensions.cs +++ b/src/Microsoft.Extensions.WebEncoders/EncoderServiceCollectionExtensions.cs @@ -9,6 +9,9 @@ using Microsoft.Extensions.WebEncoders; namespace Microsoft.Extensions.DependencyInjection { + /// + /// Extension methods for setting up web encoding services in an . + /// public static class EncoderServiceCollectionExtensions { /// @@ -17,24 +20,7 @@ namespace Microsoft.Extensions.DependencyInjection /// /// The . /// The instance after the encoders have been added. - public static IServiceCollection AddWebEncoders(this IServiceCollection services) - { - if (services == null) - { - throw new ArgumentNullException(nameof(services)); - } - - return AddWebEncoders(services, configureOptions: null); - } - - /// - /// Adds , and - /// to the specified . - /// - /// The . - /// A callback to configure . - /// The instance after the encoders have been added. - public static IServiceCollection AddWebEncoders(this IServiceCollection services, Action configureOptions) + public static void AddWebEncoders(this IServiceCollection services) { if (services == null) { @@ -51,13 +37,28 @@ namespace Microsoft.Extensions.DependencyInjection CreateFactory(() => JavaScriptEncoder.Default, settings => JavaScriptEncoder.Create(settings))); services.TryAddSingleton( CreateFactory(() => UrlEncoder.Default, settings => UrlEncoder.Create(settings))); + } - if (configureOptions != null) + /// + /// Adds , and + /// to the specified . + /// + /// The . + /// An to configure the provided . + public static void AddWebEncoders(this IServiceCollection services, Action setupAction) + { + if (services == null) { - services.Configure(configureOptions); + throw new ArgumentNullException(nameof(services)); } - return services; + if (setupAction == null) + { + throw new ArgumentNullException(nameof(setupAction)); + } + + services.AddWebEncoders(); + services.Configure(setupAction); } private static Func CreateFactory(