From 549f0555abfdbe1a9ada48cc9dcb59d83e1562a4 Mon Sep 17 00:00:00 2001
From: Shannon <sdeminick@gmail.com>
Date: Mon, 8 Sep 2014 19:16:30 +1000
Subject: [PATCH] don't cache too much when using get all

---
 src/Umbraco.Core/Persistence/Repositories/RepositoryBase.cs | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/Umbraco.Core/Persistence/Repositories/RepositoryBase.cs b/src/Umbraco.Core/Persistence/Repositories/RepositoryBase.cs
index ccf946f9e2..1424861d52 100644
--- a/src/Umbraco.Core/Persistence/Repositories/RepositoryBase.cs
+++ b/src/Umbraco.Core/Persistence/Repositories/RepositoryBase.cs
@@ -174,6 +174,12 @@ namespace Umbraco.Core.Persistence.Repositories
                 .WhereNotNull()
                 .ToArray();
 
+            //We need to put a threshold here! IF there's an insane amount of items
+            // coming back here we don't want to chuck it all into memory, this added cache here
+            // is more for convenience when paging stuff temporarily
+
+            if (entityCollection.Length > 100) return entityCollection;
+
             foreach (var entity in entityCollection)
             {
                 if (entity != null)