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

executeAndFetch fallback to executeScalar #46

Merged
merged 8 commits into from
Feb 8, 2014

Conversation

aldenquimby
Copy link
Contributor

This is a second try at #45.

  • executeAndFetch methods fall back to executeScalar methods if:
  • there is only one column returned
  • no property on the given class matches the column name
  • we have a converter for the class
  • executeScalarList now uses registered converters, see executeScalarList should use converters #36
    • WARNING this changes the API because a class now needs to be passed in to use the converter
    • This is good though, the method should have required a class previously

@aldenquimby
Copy link
Contributor Author

I apologize for the strange empty line diffs, IntelliJ automatically cleans up tabbing. I tried to revert some of them, but it became too tedious

aaberg added a commit that referenced this pull request Feb 8, 2014
executeAndFetch fallback to executeScalar
@aaberg aaberg merged commit 3bc102a into aaberg:master Feb 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants