Inconsistent naming for UpdateBrandCommand Handler

aspnetcorehero / Boilerplate

Clean Architecture Solution Template for ASP.NET Core 5.0. Built with Onion/Hexagonal Architecture and incorporates the most essential Packages your projects will ever need. Includes both WebApi and Web(MVC) Projects.

https://codewithmukesh.com/project/aspnet-core-hero-boilerplate/

MIT License

582 stars 148 forks source link

Inconsistent naming for UpdateBrandCommand Handler #27

Open tanczosm opened 3 years ago

tanczosm commented 3 years ago

Describe the bug This is not so much a bug as a coding mistake.

/Boilerplate/blob/master/AspNetCoreHero.Boilerplate.Application/Features/Brands/Commands/Update/UpdateBrandCommand.cs

This looks like it may have been a copy of UpdateProductCommand.cs that was altered to suit Brand. The handler inside the UpdateBrandCommand is called UpdateProductCommandHandler instead of UpdateBrandCommandHandler.

ravarador commented 3 years ago

Same goes with IBrandCache repository and IProductCacheRepository.