From 304006e12ce4a811e809f215da5d76112ef0aa2c Mon Sep 17 00:00:00 2001 From: Erik Weiss Date: Wed, 21 Oct 2015 14:19:21 -0500 Subject: [PATCH] Execute 404 keyword searching in SQL instead of selecting all entries and filtering in linq --- Repositories/UrlTrackerRepository.cs | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/Repositories/UrlTrackerRepository.cs b/Repositories/UrlTrackerRepository.cs index 59e5077..0ac458c 100644 --- a/Repositories/UrlTrackerRepository.cs +++ b/Repositories/UrlTrackerRepository.cs @@ -312,7 +312,7 @@ public static List GetUrlTrackerEntries(int? maximumRows, int? public static List GetNotFoundEntries(int? maximumRows, int? startRowIndex, string sortExpression = "", string keyword = "") { List notFoundEntries = new List(); - List urlTrackerEntries = GetUrlTrackerEntries(maximumRows, startRowIndex, sortExpression, true); + List urlTrackerEntries = GetUrlTrackerEntries(maximumRows, startRowIndex, sortExpression, true, keyword: keyword); foreach (var notFoundEntry in urlTrackerEntries.GroupBy(x => x.OldUrl).Select(x => new { Count = x.Count(), @@ -329,19 +329,6 @@ public static List GetNotFoundEntries(int? maximumRows, int? st notFoundEntries.Add(notFoundEntry.UrlTrackerModel); } - if (!string.IsNullOrEmpty(keyword)) - { - IEnumerable filteredNotFoundEntries = notFoundEntries; - if (!string.IsNullOrEmpty(keyword)) - { - filteredNotFoundEntries = filteredNotFoundEntries.Where(x => - (x.CalculatedOldUrl != null && x.CalculatedOldUrl.ToLower().Contains(keyword)) || - (x.Referrer != null && x.Referrer.ToLower().Contains(keyword)) - ); - } - notFoundEntries = filteredNotFoundEntries.ToList(); - } - if (!string.IsNullOrEmpty(sortExpression)) { string sortBy = sortExpression;