Merge branch 'dev' of https://github.com/dotnet/eShopOnContainers into dev

Author: ramon-tomas-c

src/Services/Catalog/Catalog.API/Controllers/CatalogController.cs

@@ -130,6 +130,7 @@ public async Task<IActionResult> CatalogBrands()
             return Ok(items);
         }
 
+        //POST api/v1/[controller]/edit
         [Route("edit")]
         [HttpPost]
         public async Task<IActionResult> EditProduct([FromBody]CatalogItem product)
@@ -164,6 +165,7 @@ public async Task<IActionResult> EditProduct([FromBody]CatalogItem product)
             return Ok();
         }
 
+        //POST api/v1/[controller]/create
         [Route("create")]
         [HttpPost]
         public async Task<IActionResult> CreateProduct([FromBody]CatalogItem product)
@@ -184,6 +186,7 @@ public async Task<IActionResult> CreateProduct([FromBody]CatalogItem product)
             return Ok();
         }
 
+        //DELETE api/v1/[controller]/id
         [Route("{id}")]
         [HttpDelete]
         public async Task<IActionResult> DeleteProduct(int id)

src/Services/Ordering/Ordering.Infrastructure/OrderingContext.cs

@@ -237,10 +237,18 @@ void ConfigureCardTypes(EntityTypeBuilder<CardType> cardTypesConfiguration)
 
         public async Task<int> SaveEntitiesAsync(CancellationToken cancellationToken = default(CancellationToken))
         {
+            // Dispatch Domain Events collection. 
+            // Choices:
+            // A) Right BEFORE committing data (EF SaveChanges) into the DB will make a single transaction including  
+            // side effects from the domain event handlers which are using the same DbContext with "InstancePerLifetimeScope" or "scoped" lifetime
+            // B) Right AFTER committing data (EF SaveChanges) into the DB will make multiple transactions. 
+            // You will need to handle eventual consistency and compensatory actions in case of failures in any of the Handlers. 
+            await _mediator.DispatchDomainEventsAsync(this);
+
+
+            // After executing this line all the changes performed thought the DbContext will be commited
             var result = await base.SaveChangesAsync();
 
-            // Dispatch the Domain Events collection right after saving/committing data into the database
-            await _mediator.DispatchDomainEventsAsync(this);
             return result;
         }        
     }

test/Services/FunctionalTests/Services/Catalog/CatalogScenariosBase.cs

@@ -34,7 +34,7 @@ public static string ProductByName(string name)
 
         public static class Post
         {
-            public static string UpdateCatalogProduct = "api/v1/catalog";
+            public static string UpdateCatalogProduct = "api/v1/catalog/edit";
         }
     }
 }