Skip to content
This repository has been archived by the owner on Dec 20, 2023. It is now read-only.

Add a method to dataset/query to append a group of OR conditions to an e... #82

Merged
merged 2 commits into from
Dec 13, 2013

Conversation

jkc
Copy link
Contributor

@jkc jkc commented Dec 11, 2013

...xisting where clause, and added a small set of tests for it.

Changes

Added the andGroupedOr method to the dataset query object to allow adding a group of OR conditions to an existing where clause. For example ... AND ((x = 1) OR (x = 2)).

@jkc
Copy link
Contributor Author

jkc commented Dec 12, 2013

Tests failed due to a method name change, which I forgot to change in one test. This will be corrected when I amend this pull request to include the other permutations of and/or combinations.

@jkc
Copy link
Contributor Author

jkc commented Dec 12, 2013

Doug requested andGroupedAnd and orGroupedOr methods but the output sql isn't grouping with parens in the same way as the others.

doug-martin added a commit that referenced this pull request Dec 13, 2013
Add a method to dataset/query to append a group of OR conditions to an e...
@doug-martin doug-martin merged commit 40addd5 into C2FO:master Dec 13, 2013
doug-martin added a commit to doug-martin/patio that referenced this pull request Dec 13, 2013
Updated model to store transformed values in the changed hash
Update docs for pr C2FO#82
@doug-martin doug-martin mentioned this pull request Dec 13, 2013
doug-martin added a commit to doug-martin/patio that referenced this pull request Dec 13, 2013
Updated model to store transformed values in the changed hash
Update docs for pr C2FO#82
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants