Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FindAll() with query expressions not working #17

Closed
sstoenner opened this issue Jan 7, 2017 · 7 comments
Closed

FindAll() with query expressions not working #17

sstoenner opened this issue Jan 7, 2017 · 7 comments
Labels

Comments

@sstoenner
Copy link

Using the following prerequisite:
var _ctx = new DapperRepository(new SqlConnection(connectStr), ESqlConnector.MSSQL);

Please consider the following 2 statements:

  1. var result = _ctx.FindAll(x => x.MachineLinkID == 1 && x.DateStampUp == null && x.OperatorID == null);

  2. var result = _ctx.FindAll().Where(x => x.MachineLinkID == 1 && x.DateStampUp == null && x.OperatorID == null);

The first one returns a result count of 0 while the second one returns the expected results...
What am I missing here?
I am using your library with Sql Server 12.

@phnx47 phnx47 added the bug label Jan 7, 2017
@digitalmedia34
Copy link

Question, was this issue resolved? I am trying to replicate it but cant.

@phnx47
Copy link
Owner

phnx47 commented Feb 3, 2017

I haven't finished with tests

@sstoenner
Copy link
Author

Don't know what was done, but actually this is working correctly in the latest 1.6.0-Beta4 version, but not in 1.5.

@phnx47
Copy link
Owner

phnx47 commented Feb 3, 2017

Yes, that's right. I will make the release next week.

@digitalmedia34
Copy link

Phnx47, let me know if you help. I'm a programmer and really love dapper. This tool it's really nice.

@phnx47
Copy link
Owner

phnx47 commented Feb 4, 2017

@digitalmedia34 Write me to Email or Telegram. I have incomplete tasks

@phnx47
Copy link
Owner

phnx47 commented Feb 19, 2017

I closed it. This is working correctly in pre-release version.

@phnx47 phnx47 closed this as completed Feb 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants