Sales Over Time report HTTP cache seems incorrect

[AbleMods]

I'm troubleshooting an issue with the dashboard Sales Over Time report. While reviewing the code, I noticed this:

Code: Select all

Cache.Add(cacheKey, cacheWrapper, null, DateTime.UtcNow.AddMinutes(5).AddSeconds(-1), TimeSpan.Zero, System.Web.Caching.CacheItemPriority.High, null);

Shouldn't the absolute cache expiration be based upon LOCAL time, not UTC?

Especially since you're using a CacheWrapper() class which establishes a cache entry date value using LocalNow.

[ForumsAdmin]

I agree. We will have a look into this.