Support IQueryable<T>.AsTracking<T> methods

[codelovercc]

Support IQueryable.AsTracking methods

from specification.

[codelovercc]

We can support AsTracking like this:

In static class SpecificationBuilderExtensions , add two methods

    /// <summary>
    /// If the entity instances are modified, this will be detected
    /// by the change tracker.
    /// </summary>
    /// <param name="specificationBuilder"></param>
    public static ISpecificationBuilder<T> AsTracking<T>(
        this ISpecificationBuilder<T> specificationBuilder) where T : class
        => AsTracking(specificationBuilder, true);

    /// <summary>
    /// If the entity instances are modified, this will be detected
    /// by the change tracker.
    /// </summary>
    /// <param name="specificationBuilder"></param>
    /// <param name="condition">If false, the setting will be discarded.</param>
    public static ISpecificationBuilder<T> AsTracking<T>(
        this ISpecificationBuilder<T> specificationBuilder,
        bool condition) where T : class
    {
      if (condition)
      {
        specificationBuilder.Specification.AsNoTracking = false;
      }

      return specificationBuilder;
    }

In class AsNoTrackingEvaluator, modify GetQuery<T> method.

    public IQueryable<T> GetQuery<T>(IQueryable<T> query, ISpecification<T> specification) where T : class
    {
      return specification.AsNoTracking ?  query.AsNoTracking() : query.AsTracking();
    }

[codelovercc]

In situation one, We can let the repository class control the query is tracked or not and ignore specification's AsNoTracking.

protected virtual IQueryable<TEntity> ApplySpecification(ISpecification<TEntity> specification,
        bool isTracked = true,
        bool evaluateCriteriaOnly = false) where TEntity : class
    {
        var q = SpecificationEvaluator.GetQuery(EntitySet, specification, evaluateCriteriaOnly);
        return isTracked ? q.AsTracking() : q.AsNoTracking();
    }

So, What do we say? I still think that support AsTracking will be nicer :)

[ardalis]

@fiseni thoughts?

[fiseni]

I think we had another request for AsTracking, so let's add it. @codelovercc Do you want to create a PR?

[codelovercc]

Sure, I'd like to.

[codelovercc]

Should we rename class AsNoTrackingEvaluator to TrackingEvaluator? When it support AsTracking and AsNoTracking, this may be more suitable but it's a breaking change.

[fiseni]

No, no need for that. You'll create a separate new file called AsTrackingEvaluator.

[codelovercc]

Completed, PR #338