AnantLabs / codesmith

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

PLINQO EF: Using the generic type 'System.Func<TResult>' requires 1 type arguments for GetByKey query method #572

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Number of columns in the table for the method is more than allowed for the 
Func<T>:

            internal static readonly Func<WebCert.Data.WebCertDataContext, System.Int32, System.String, System.String, System.String, System.String, System.String, System.String, System.Double, System.Double, System.String, System.String, System.String, System.Int32, System.String, System.Int32, System.String, WebCert.Data.vUnitStudiedCaptured> GetByKey =

Original issue reported on code.google.com by kachalkov on 6 Apr 2011 at 1:46

GoogleCodeExporter commented 9 years ago
This issue appears for the views which does not have primary key defined...

Original comment by kachalkov on 6 Apr 2011 at 1:47

GoogleCodeExporter commented 9 years ago
Hello,

We explicitly check to see if a view has a key and if it doesn't we should drop 
it. Are you getting any validation errors on your views that have no key 
defined?

Thanks
-Blake Niemyjski

Original comment by bniemyjski on 8 Apr 2011 at 12:06

GoogleCodeExporter commented 9 years ago
Compilation errors as one of our views has 46 columns, so I believe this check 
when there is no key does not work very well

Original comment by kachalkov on 8 Apr 2011 at 12:22

GoogleCodeExporter commented 9 years ago
Hello,

By default EF defines a view's keys as any non-nullable column. So we also 
replicate this behavior. But, we do it much better in the sense that if you 
modify the key definition for a view we do not modify it. Would it be possible 
to send me your generated edmx file to support so I could take a look into 
these errors.

Thanks
-Blake

Original comment by bniemyjski on 8 Apr 2011 at 5:34

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
sorry, Monday probably is a hard day :)
Here is edmx file

Original comment by kachalkov on 11 Apr 2011 at 7:15

Attachments:

GoogleCodeExporter commented 9 years ago
Hello,

I was not able to reproduce this in the latest nightly build 
(http://community.codesmithtools.com/nightly/). Could you please try the 
following nightly build and let us know if you are still receiving this error.

Original comment by bniemyjski on 14 Apr 2011 at 1:48

GoogleCodeExporter commented 9 years ago
The problem still exists with latest nightly build, for couple of views...

Original comment by kachalkov on 15 Apr 2011 at 1:13

GoogleCodeExporter commented 9 years ago
it looks like now all views and stored procedures are being excluded from 
generation because of missing primary key...

Original comment by kachalkov on 15 Apr 2011 at 3:36

GoogleCodeExporter commented 9 years ago
Hello,

Should this ticket be reopened or has this issue changed and a new ticket is 
needed?

Thanks
-Blake

Original comment by bniemyjski on 15 Apr 2011 at 3:52

GoogleCodeExporter commented 9 years ago
yes, please reopen it again as in latest version it's still exists

Original comment by kachalkov on 28 Apr 2011 at 4:03

GoogleCodeExporter commented 9 years ago

Original comment by bniemyjski on 28 Apr 2011 at 1:11

GoogleCodeExporter commented 9 years ago

Original comment by bniemyjski on 28 Apr 2011 at 11:16

GoogleCodeExporter commented 9 years ago
This issue was updated by revision r2166.

Fixed a bug with View Key Generation (validation between conceptual and storage 
model). 
Fixed a bug where Queries extensions were being generated for views.

Original comment by bniemyjski on 29 Apr 2011 at 7:10

GoogleCodeExporter commented 9 years ago
These bugs were discovered and fixed as a result of your schema.

Original comment by bniemyjski on 29 Apr 2011 at 7:12

GoogleCodeExporter commented 9 years ago

Original comment by bniemyjski on 15 Jun 2011 at 11:27