Also, added more UrlChecks to the MVC app so it checks all its dependencies.
Plus minor bugs/missing code in docker-compose.pro.yml that was evolved in the .override.yml
This prevents cookie confusion when applications are hosted at the same domain and path. For example, under default settings, WebMVC may attempt to decrypt Identity's antiforgery cookie rather than its own.
Controllers/AccountController:
Replace single line methods with expression bodied members.
Includes constructor, C# 7 feature.
Controllers/CatalogController:
Remove unnecessary ToString() call.
Replace single line methods with expression bodied members.
Extensions/HttpClientExtensions:
Replace single line methods with expression bodied members.
Extensions/SessionExtensions:
Replace single line methods with expression bodied members.
Services/BasketService:
Remove unnecessary ToString() calls.
Add ?? to simplify conditional initialization
Use TryGetValue and out variable initialization to simplify Quantity
calculation
Services/CatalogService:
Use Value<T> generic method instead of dynamic types.
There is a lot of overhead for dynamic and it seems overkill here.
Services/IdentityParser:
Use the pattern matching is expression.
Refactor the LINQ queries to enumerate the collection (and create an
enumerator)
only once. The previous code had 3 enumerations.
Services/Utilities/HttpApiClient:
Remove the 'async' modifier and 'await' statements from methods where
the only asynchronous statement is the final statement of the method,
and
the task from the underlying method can be returned.
Services/Utilities/HttpApiClientWrapper:
Use expression bodied members where applicable.
Remove the 'async' modifier and 'await' statements from methods where
the only asynchronous statement is the final statement of the method,
and
the task from the underlying method can be returned.
ViewComponents/Cart:
Use expression bodied members where applicable.
ViewComponents/CartList:
Use expression bodied members where applicable.
Remove the 'async' modifier and 'await' statements from methods where
the only asynchronous statement is the final statement of the method,
and
the task from the underlying method can be returned.
ViewModels/Annotations/CardExpiration:
Use the out variable initializer to simplify the validation of the
card expiration date.
ViewModels/Basket:
Use property initializer syntax instead of constructor body
ViewModels/CatalogViewModels/IndexViewModel:
Use expression bodied property to return the calculated 'Disabled'
property
ViewModels/Order:
Use property initializer syntax.
