Browse Source

revised todos.

pull/81/head
Halil İbrahim Kalkan 10 years ago
parent
commit
e463d52b02
  1. 5
      src/Volo.Abp/Volo/Abp/Json/JsonExtensions.cs
  2. 2
      src/Volo.Abp/Volo/Abp/Linq/DefaultAsyncQueryableExecuter.cs

5
src/Volo.Abp/Volo/Abp/Json/JsonExtensions.cs

@ -5,6 +5,8 @@ namespace Volo.Abp.Json
{ {
public static class JsonExtensions public static class JsonExtensions
{ {
//TODO: Remove this extension method, create IJsonSerializer abstraction (if there is not already such an abstraction).
/// <summary> /// <summary>
/// Converts given object to JSON string. /// Converts given object to JSON string.
/// </summary> /// </summary>
@ -23,7 +25,8 @@ namespace Volo.Abp.Json
options.Formatting = Formatting.Indented; options.Formatting = Formatting.Indented;
} }
//options.Converters.Insert(0, new AbpDateTimeConverter()); //TODO: AbpDateTimeConverter? //TODO: AbpDateTimeConverter contains Clock, so it should be injected!
//options.Converters.Insert(0, new AbpDateTimeConverter());
return JsonConvert.SerializeObject(obj, options); return JsonConvert.SerializeObject(obj, options);
} }

2
src/Volo.Abp/Volo/Abp/Linq/DefaultAsyncQueryableExecuter.cs

@ -6,8 +6,6 @@ using Volo.DependencyInjection;
namespace Volo.Abp.Linq namespace Volo.Abp.Linq
{ {
//TODO: DefaultAsyncQueryableExecuter should be able to work with multiple Executer, each will try to execute it!
//TODO: Implement with EF Core as first executer implementation!
public class DefaultAsyncQueryableExecuter : IAsyncQueryableExecuter, ITransientDependency public class DefaultAsyncQueryableExecuter : IAsyncQueryableExecuter, ITransientDependency
{ {
public Task<int> CountAsync<T>(IQueryable<T> queryable) public Task<int> CountAsync<T>(IQueryable<T> queryable)

Loading…
Cancel
Save