Ayende @ Rahien

Peter Willis commented on Porting MVC Music Store to Raven: StoreController

Sun, 13 Jun 2010 10:32:47 GMT

"There are one this in this code that bothers me, and that is that this code is going to perform two DB queries. But that is beside the point, since we are going to modify the whole thing." "One thing that bothers me is that this code is going to perform two DB queries. This is beside the point however, since we are going to modify the whole approach."

Ayende Rahien commented on Porting MVC Music Store to Raven: StoreController

Tue, 25 May 2010 21:50:57 GMT

configurator, a) the key I used was the one the original application used. b) yes, jazz is easier than 17, but I followed the path of the app. c) all keys in Raven are strings

configurator commented on Porting MVC Music Store to Raven: StoreController

Tue, 25 May 2010 21:47:05 GMT

A few questions. (I haven't looked at the actual example but I believe I understand what's going on) First, isn't the parameter to Browse passed in the url? In that case I'd much rather see /browse/jazz than /browse/17, wouldn't you? Second, what is the static type of session.Query

tobi commented on Porting MVC Music Store to Raven: StoreController

Tue, 25 May 2010 16:07:11 GMT

I think the index system is very nice. Simple, maintainable and fast.

Rob Ashton commented on Porting MVC Music Store to Raven: StoreController

Tue, 25 May 2010 13:54:43 GMT

Note to the uninitiated, this is now out of date and it is now possible to do strongly typed queries .Where(x=>x.Genre == genre); :)