Merge pull request #359 from johelvisguzman/cache

Simplified the logic of getting a caching result

Author: johelvisguzman

src/DotNetToolkit.Repository/Extensions/CachingProviderExtensions.cs

@@ -19,7 +19,7 @@
     /// </summary>
     internal static class CachingProviderExtensions
     {
-        private static bool TryGetValue<T>(this ICacheProvider cacheProvider, string key, out T value)
+        private static bool TryGetValue<T>(this ICacheProvider cacheProvider, string key, out QueryResult<T> value)
         {
             if (cacheProvider == null)
                 throw new ArgumentNullException(nameof(cacheProvider));
@@ -29,12 +29,14 @@ private static bool TryGetValue<T>(this ICacheProvider cacheProvider, string key
 
             if (!cacheProvider.Cache.TryGetValue(key, out var obj))
             {
-                value = default(T);
+                value = default(QueryResult<T>);
 
                 return false;
             }
 
-            value = (T)obj;
+            value = (QueryResult<T>)obj;
+
+            value.CacheUsed = true;
 
             return true;
         }
@@ -87,16 +89,12 @@ private static QueryResult<T> GetOrSet<T>(this ICacheProvider cacheProvider, str
 
             var hashedKey = FormatHashKey(key);
 
-            if (!cacheProvider.TryGetValue<QueryResult<T>>(hashedKey, out var value))
+            if (!cacheProvider.TryGetValue<T>(hashedKey, out var value))
             {
                 value = getter();
 
                 cacheProvider.SetValue(hashedKey, key, value, priority, cacheExpiration, logger);
             }
-            else
-            {
-                value.CacheUsed = true;
-            }
 
             return value;
         }
@@ -243,16 +241,12 @@ private static async Task<QueryResult<T>> GetOrSetAsync<T>(this ICacheProvider c
 
             var hashedKey = FormatHashKey(key);
 
-            if (!cacheProvider.TryGetValue<QueryResult<T>>(hashedKey, out var value))
+            if (!cacheProvider.TryGetValue<T>(hashedKey, out var value))
             {
                 value = await getter();
 
                 cacheProvider.SetValue(hashedKey, key, value, priority, cacheExpiration, logger);
             }
-            else
-            {
-                value.CacheUsed = true;
-            }
 
             return value;
         }