google-code-export / dataobjectsdotnet

Automatically exported from code.google.com/p/dataobjectsdotnet
0 stars 0 forks source link

Eliminate Query<T> #535

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I propose removing of this type in favor of non-generic Query.
Having two queries is counter-intuitive.
Currently some methods of non-generic Query (e.g. Store, Execute) are
generic, althought we have generic Query. We should have one "entry point"
for query functionality.

Also, new API will look cool :-)

Query.All<Customer>() (Query all customers)
Query.Single<Customer>(id) (Query single customer)

Original issue reported on code.google.com by denis.kr...@gmail.com on 14 Dec 2009 at 1:55

GoogleCodeExporter commented 9 years ago
Looks as good idea. Initially Query<T>.All was actually the only property 
there, but 
now there are lots of them.

Upcoming Query.AllText<T>() would look nice here as well ;)

Original comment by alex.yakunin on 14 Dec 2009 at 2:06

GoogleCodeExporter commented 9 years ago
Query<T> will be obsolete now.

Original comment by alex.yakunin on 15 Dec 2009 at 6:49

GoogleCodeExporter commented 9 years ago

Original comment by alex.yakunin on 21 Dec 2009 at 7:08

GoogleCodeExporter commented 9 years ago
I finally removed Query<T> at all. LINQ translator already does not support it, 
so I 
decided to, well, simplify everything a bit.

Original comment by alex.yakunin on 21 Dec 2009 at 7:42

GoogleCodeExporter commented 9 years ago

Original comment by alex.yakunin on 21 Dec 2009 at 7:52