Ayende @ Rahienhttp://ayende.comAyende @ RahienCopyright (C) Ayende Rahien 2004 - 2021 (c) 202660JonW commented on Porting MVC Music Store to Raven: StoreManagerControllerKeep up the good work! :-) Here's a tip for strongly typed Includes: [blogs.msdn.com/.../...ment-include-strategies.aspx](http://blogs.msdn.com/b/alexj/archive/2009/07/25/tip-28-how-to-implement-include-strategies.aspx) http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment6http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment6Thu, 03 Jun 2010 08:50:32 GMTAyende Rahien commented on Porting MVC Music Store to Raven: StoreManagerControllerPaul, Probably, but I don't like error handling like that. Either it should catch ValidationException or it should use TryUpdateModel http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment5http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment5Sun, 30 May 2010 07:18:53 GMTPaul Cox commented on Porting MVC Music Store to Raven: StoreManagerControllerBut UpdateModel will place the validation errors in ModelState before throwing an exception. Therefore catching the error and redisplaying the view will display those errors to the user. Won't the new code go to the generic error page as written? http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment4http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment4Sat, 29 May 2010 21:14:49 GMTAyende Rahien commented on Porting MVC Music Store to Raven: StoreManagerControllerPaul, No, I am not. try {} catch{} is evil, errors should be handled, not swept under the rug. http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment3http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment3Sat, 29 May 2010 20:50:38 GMTPaul Cox commented on Porting MVC Music Store to Raven: StoreManagerControllerAre you handling form validation errors somewhere else now that you no longer catch any exceptions from UpdateModel? http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment2http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment2Sat, 29 May 2010 18:38:12 GMTRafal commented on Porting MVC Music Store to Raven: StoreManagerControllerDeleting orders together with album is not a RDBMS limitation but a design flaw (at least for an order processing software) http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment1http://ayende.com/4529/porting-mvc-music-store-to-raven-storemanagercontroller#comment1Sat, 29 May 2010 18:31:57 GMT