From d2f7d6a5d6d9cc90caf9f7c6503bec459115c7af Mon Sep 17 00:00:00 2001 From: Cesar De la Torre Date: Tue, 7 Nov 2017 16:53:41 -0800 Subject: [PATCH] Minor clean-up on Entity, Domain Events. --- .../AggregatesModel/OrderAggregate/Order.cs | 8 ++++---- .../Ordering.Domain/Events/OrderStartedDomainEvent.cs | 9 ++++----- src/Services/Ordering/Ordering.Domain/SeedWork/Entity.cs | 3 ++- .../Ordering/Ordering.Infrastructure/OrderingContext.cs | 2 +- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/Services/Ordering/Ordering.Domain/AggregatesModel/OrderAggregate/Order.cs b/src/Services/Ordering/Ordering.Domain/AggregatesModel/OrderAggregate/Order.cs index 9b5216392..661965c5c 100644 --- a/src/Services/Ordering/Ordering.Domain/AggregatesModel/OrderAggregate/Order.cs +++ b/src/Services/Ordering/Ordering.Domain/AggregatesModel/OrderAggregate/Order.cs @@ -178,16 +178,16 @@ namespace Microsoft.eShopOnContainers.Services.Ordering.Domain.AggregatesModel.O private void AddOrderStartedDomainEvent(string userId, int cardTypeId, string cardNumber, string cardSecurityNumber, string cardHolderName, DateTime cardExpiration) { - var orderStartedDomainEvent = new OrderStartedDomainEvent( - this, userId, cardTypeId, cardNumber, cardSecurityNumber, - cardHolderName, cardExpiration); + var orderStartedDomainEvent = new OrderStartedDomainEvent(this, userId, cardTypeId, + cardNumber, cardSecurityNumber, + cardHolderName, cardExpiration); this.AddDomainEvent(orderStartedDomainEvent); } private void StatusChangeException(OrderStatus orderStatusToChange) { - throw new OrderingDomainException($"Not possible to change order status from {OrderStatus.Name} to {orderStatusToChange.Name}."); + throw new OrderingDomainException($"Is not possible to change the order status from {OrderStatus.Name} to {orderStatusToChange.Name}."); } public decimal GetTotal() diff --git a/src/Services/Ordering/Ordering.Domain/Events/OrderStartedDomainEvent.cs b/src/Services/Ordering/Ordering.Domain/Events/OrderStartedDomainEvent.cs index 38de4601a..a457067f6 100644 --- a/src/Services/Ordering/Ordering.Domain/Events/OrderStartedDomainEvent.cs +++ b/src/Services/Ordering/Ordering.Domain/Events/OrderStartedDomainEvent.cs @@ -9,8 +9,7 @@ namespace Ordering.Domain.Events /// /// Event used when an order is created /// - public class OrderStartedDomainEvent - : INotification + public class OrderStartedDomainEvent : INotification { public string UserId { get; private set; } public int CardTypeId { get; private set; } @@ -21,9 +20,9 @@ namespace Ordering.Domain.Events public Order Order { get; private set; } public OrderStartedDomainEvent(Order order, string userId, - int cardTypeId, string cardNumber, - string cardSecurityNumber, string cardHolderName, - DateTime cardExpiration) + int cardTypeId, string cardNumber, + string cardSecurityNumber, string cardHolderName, + DateTime cardExpiration) { Order = order; UserId = userId; diff --git a/src/Services/Ordering/Ordering.Domain/SeedWork/Entity.cs b/src/Services/Ordering/Ordering.Domain/SeedWork/Entity.cs index 9b68af850..11b940218 100644 --- a/src/Services/Ordering/Ordering.Domain/SeedWork/Entity.cs +++ b/src/Services/Ordering/Ordering.Domain/SeedWork/Entity.cs @@ -21,7 +21,8 @@ } private List _domainEvents; - public List DomainEvents => _domainEvents; + public List DomainEvents => _domainEvents; + public void AddDomainEvent(INotification eventItem) { _domainEvents = _domainEvents ?? new List(); diff --git a/src/Services/Ordering/Ordering.Infrastructure/OrderingContext.cs b/src/Services/Ordering/Ordering.Infrastructure/OrderingContext.cs index 1855b268a..a2cfb96a5 100644 --- a/src/Services/Ordering/Ordering.Infrastructure/OrderingContext.cs +++ b/src/Services/Ordering/Ordering.Infrastructure/OrderingContext.cs @@ -12,7 +12,7 @@ using System.Threading.Tasks; namespace Microsoft.eShopOnContainers.Services.Ordering.Infrastructure { - public class OrderingContext : DbContext,IUnitOfWork + public class OrderingContext : DbContext, IUnitOfWork { public const string DEFAULT_SCHEMA = "ordering"; public DbSet Orders { get; set; }