New default constructor for FileLoadExceptionContext

maxkoshevoi · Dec 9, 2021 · a06b504 · a06b504
1 parent a576913
commit a06b504
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 10 deletions.
diff --git a/...ons.Configuration.FileExtensions/ref/Microsoft.Extensions.Configuration.FileExtensions.cs b/...ons.Configuration.FileExtensions/ref/Microsoft.Extensions.Configuration.FileExtensions.cs
@@ -40,9 +40,9 @@ public void ResolveFileProvider() { }
     }
     public partial class FileLoadExceptionContext
     {
-        public FileLoadExceptionContext() { }
-        public System.Exception? Exception { get { throw null; } set { } }
+        public FileLoadExceptionContext(FileConfigurationProvider provider, System.Exception exception) { }
+        public System.Exception Exception { get { throw null; } set { } }
         public bool Ignore { get { throw null; } set { } }
-        public Microsoft.Extensions.Configuration.FileConfigurationProvider? Provider { get { throw null; } set { } }
+        public Microsoft.Extensions.Configuration.FileConfigurationProvider Provider { get { throw null; } set { } }
     }
 }
diff --git a/...raries/Microsoft.Extensions.Configuration.FileExtensions/src/FileConfigurationProvider.cs b/...raries/Microsoft.Extensions.Configuration.FileExtensions/src/FileConfigurationProvider.cs
@@ -136,11 +136,7 @@ private void HandleException(ExceptionDispatchInfo info)
             bool ignoreException = false;
             if (Source.OnLoadException != null)
             {
-                var exceptionContext = new FileLoadExceptionContext
-                {
-                    Provider = this,
-                    Exception = info.SourceException
-                };
+                var exceptionContext = new FileLoadExceptionContext(this, info.SourceException);
                 Source.OnLoadException.Invoke(exceptionContext);
                 ignoreException = exceptionContext.Ignore;
             }

diff --git a/...braries/Microsoft.Extensions.Configuration.FileExtensions/src/FileLoadExceptionContext.cs b/...braries/Microsoft.Extensions.Configuration.FileExtensions/src/FileLoadExceptionContext.cs
@@ -10,15 +10,21 @@ namespace Microsoft.Extensions.Configuration
     /// </summary>
     public class FileLoadExceptionContext
     {
+        public FileLoadExceptionContext(FileConfigurationProvider provider, Exception exception)
+        {
+            Provider = provider;
+            Exception = exception;
+        }
+
         /// <summary>
         /// The <see cref="FileConfigurationProvider"/> that caused the exception.
         /// </summary>
-        public FileConfigurationProvider? Provider { get; set; }
+        public FileConfigurationProvider Provider { get; set; }
 
         /// <summary>
         /// The exception that occurred in Load.
         /// </summary>
-        public Exception? Exception { get; set; }
+        public Exception Exception { get; set; }
 
         /// <summary>
         /// If true, the exception will not be rethrown.