Merge pull request #38 from codepb/develop

Changes to API Controllers

Author: codepb

src/DDDToolkit.API/AggregateController.cs

@@ -19,10 +19,10 @@ protected AggregateController(IApplicationService<T, TId> applicationService)
 
         [HttpGet]
         [Route("{id}")]
-        public virtual Task<T> GetById(TId id) => _readableAggregateController.GetById(id);
+        public virtual Task<IActionResult> GetById(TId id) => _readableAggregateController.GetById(id);
 
         [HttpGet]
-        public virtual Task<IReadOnlyCollection<T>> GetAll() => _readableAggregateController.GetAll();
+        public virtual Task<IActionResult> GetAll() => _readableAggregateController.GetAll();
 
         [HttpPost]
         public virtual Task<IActionResult> Create([FromBody] T aggregate) => _writableAggregateController.Create(aggregate);

src/DDDToolkit.API/IReadableAggregateController.cs

@@ -1,12 +1,13 @@
 using System.Collections.Generic;
 using System.Threading.Tasks;
 using DDDToolkit.Core.Interfaces;
+using Microsoft.AspNetCore.Mvc;
 
 namespace DDDToolkit.API
 {
     public interface IReadableAggregateController<T, TId> where T : class, IAggregateRoot<TId>
     {
-        Task<IReadOnlyCollection<T>> GetAll();
-        Task<T> GetById(TId id);
+        Task<IActionResult> GetAll();
+        Task<IActionResult> GetById(TId id);
     }
 }

src/DDDToolkit.API/ReadableAggregateController.cs

@@ -19,15 +19,15 @@ protected ReadableAggregateController(IReadableApplicationService<T, TId> applic
 
         [HttpGet]
         [Route("{id}")]
-        public async Task<T> GetById(TId id)
+        public async Task<IActionResult> GetById(TId id)
         {
-            return await _applicationService.GetById(id);
+            return Ok(await _applicationService.GetById(id));
         }
 
         [HttpGet]
-        public async Task<IReadOnlyCollection<T>> GetAll()
+        public async Task<IActionResult> GetAll()
         {
-            return await _applicationService.GetAll();
+            return Ok(await _applicationService.GetAll());
         }
     }
 }