>>I've just changed it to IQueryable in the base classes. Now I'll have to fix all application errors due to this change.
>
>OK, I know I've asked this before. Why go through all the trouble on this, when you could just use stored procs or even straight sql pass through? This is low-hanging fruit functionality, IMO being made overly complicated. Not trying to be a pain - would truly like to understand why a development team would take this approach.

Hmm. In this case all of Naomi's problems stem from the use of the added GetAll() method. I've found that DBs (and the EF model) can get awfully cluttered with SPs that may be only needed by one developer in one specific part of the app.