Merge pull request #567 from johelvisguzman/issue566

Allowed context providers to log activity
johelvisguzman · Nov 14, 2019 · 8bebe0f · 8bebe0f
2 parents b738fc3 + 604259a
commit 8bebe0f
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/src/DotNetToolkit.Repository/RepositoryBase.cs b/src/DotNetToolkit.Repository/RepositoryBase.cs
@@ -1206,6 +1206,7 @@ public abstract class InternalRepositoryBase<TEntity> where TEntity : class
         private readonly IRepositoryContextFactory _contextFactory;
         private IEnumerable<IRepositoryInterceptor> _interceptors;
         private string _currentExecutingLoggingMethod;
+        private readonly ILoggerProvider _loggerProvider;
 
         #endregion
 
@@ -1226,11 +1227,6 @@ public abstract class InternalRepositoryBase<TEntity> where TEntity : class
         /// </summary>
         protected internal ILogger Logger { get; }
 
-        /// <summary>
-        /// Gets the repository logger provider.
-        /// </summary>
-        protected internal ILoggerProvider LoggerProvider { get; }
-
         /// <summary>
         /// Gets the caching provider.
         /// </summary>
@@ -1257,9 +1253,9 @@ internal InternalRepositoryBase([NotNull] IRepositoryOptions options)
             _contextFactory = Guard.EnsureNotNull(_options.ContextFactory, "No context provider has been configured for this repository.");
 
             // Sets the default logger provider (prints all messages levels)
-            LoggerProvider = _options.LoggerProvider ?? new ConsoleLoggerProvider(LogLevel.Debug);
+            _loggerProvider = _options.LoggerProvider ?? new ConsoleLoggerProvider(LogLevel.Debug);
 
-            Logger = LoggerProvider.Create($"DotNetToolkit.Repository<{typeof(TEntity).Name}>");
+            Logger = _loggerProvider.Create($"DotNetToolkit.Repository<{typeof(TEntity).Name}>");
 
             CacheProvider = _options.CachingProvider ?? NullCacheProvider.Instance;
 
@@ -2833,8 +2829,8 @@ protected IRepositoryContext GetContext()
                 if (_options.Conventions != null)
                     conventions.Apply(_options.Conventions);
 
-                if (context.LoggerProvider == null && LoggerProvider != null)
-                    context.LoggerProvider = LoggerProvider;
+                if (_loggerProvider != null)
+                    context.LoggerProvider = _loggerProvider;
 
                 conventions.ThrowsIfInvalidPrimaryKeyDefinition<TEntity>();
             }