Using Base Specification Class (#56)

* Ardalis/upgrade1 (#44)

* Upgrading to netcore 2.0
Updating repository to support async options and refactoring to use it.

* Starting work on tracking customer orders feature.

* Cleaning up some bugs
Working on basket view component implementation

* Fixing up styles, especially for basket in header.

* Adding Order Features (#47)

* Working on order model binding from checkout page - WIP

* Small layout tweaks (#43)

* Updating quantities implemented.

* Fixed basket widget count

* Order History (#49)

* working on creating and viewing orders.
* Working on wiring up listing of orders
* List orders page works as expected. Needed to support ThenInclude scenarios. Currently using strings.

* Remove non-icon basket link from header
Add comments to EF query logic

* Refactoring to use base specification type

* minor cleanup

src/Infrastructure/Data/EfRepository.cs

@@ -43,25 +43,34 @@ namespace Infrastructure.Data
 
         public IEnumerable<T> List(ISpecification<T> spec)
         {
+            // fetch a Queryable that includes all expression-based includes
             var queryableResultWithIncludes = spec.Includes
                 .Aggregate(_dbContext.Set<T>().AsQueryable(),
                     (current, include) => current.Include(include));
+
+            // modify the IQueryable to include any string-based include statements
             var secondaryResult = spec.IncludeStrings
                 .Aggregate(queryableResultWithIncludes,
                     (current, include) => current.Include(include));
+
+            // return the result of the query using the specification's criteria expression
             return secondaryResult
                             .Where(spec.Criteria)
                             .AsEnumerable();
         }
         public async Task<List<T>> ListAsync(ISpecification<T> spec)
         {
+            // fetch a Queryable that includes all expression-based includes
             var queryableResultWithIncludes = spec.Includes
                 .Aggregate(_dbContext.Set<T>().AsQueryable(),
                     (current, include) => current.Include(include));
+
+            // modify the IQueryable to include any string-based include statements
             var secondaryResult = spec.IncludeStrings
                 .Aggregate(queryableResultWithIncludes,
                     (current, include) => current.Include(include));
 
+            // return the result of the query using the specification's criteria expression
             return await secondaryResult
                             .Where(spec.Criteria)
                             .ToListAsync();