Slight modification on the code to reduce nesting and improve readability.

Author: moonpyk

DevTrends.MvcDonutCaching/DonutOutputCacheAttribute.cs

@@ -79,20 +79,22 @@ public override void OnActionExecuting(ActionExecutingContext filterContext)
 
                     filterContext.HttpContext.Response.Output = originalWriter;
 
-                    if (!hasErrors)
+                    if (hasErrors)
                     {
-                        var cacheItem = new CacheItem
-                        {
-                            Content = cachingWriter.ToString(),
-                            ContentType = filterContext.HttpContext.Response.ContentType
-                        };
-
-                        filterContext.HttpContext.Response.Write(_donutHoleFiller.RemoveDonutHoleWrappers(cacheItem.Content, filterContext));
-
-                        if (_cacheSettings.IsServerCachingEnabled && filterContext.HttpContext.Response.StatusCode == 200)
-                        {
-                            _outputCacheManager.AddItem(cacheKey, cacheItem, DateTime.UtcNow.AddSeconds(_cacheSettings.Duration));
-                        }
+                        return;
+                    }
+
+                    var cacheItem = new CacheItem
+                    {
+                        Content = cachingWriter.ToString(),
+                        ContentType = filterContext.HttpContext.Response.ContentType
+                    };
+
+                    filterContext.HttpContext.Response.Write(_donutHoleFiller.RemoveDonutHoleWrappers(cacheItem.Content, filterContext));
+
+                    if (_cacheSettings.IsServerCachingEnabled && filterContext.HttpContext.Response.StatusCode == 200)
+                    {
+                        _outputCacheManager.AddItem(cacheKey, cacheItem, DateTime.UtcNow.AddSeconds(_cacheSettings.Duration));
                     }
                 });
             }