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

Retain Regex options from the parsed JsonToken #4807

Closed
wants to merge 2 commits into from
Closed

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented Oct 11, 2024

We now retain BsonRegularExpression.options when resolving bind values from the original BsonRegularExpression.

Closes #4806

We now retain BsonRegularExpression.options when resolving bind values from the original BsonRegularExpression.

Closes #4806
@christophstrobl
Copy link
Member

LGTM 👍

christophstrobl pushed a commit that referenced this pull request Oct 15, 2024
We now retain expression options when resolving bind values from the original BsonRegularExpression.

Closes: #4806
Original Pull Request: #4807
christophstrobl added a commit that referenced this pull request Oct 15, 2024
Add tests to verify expected behaviour of quoted expressions and string that look like expressions.
Remove comments that look like old merge conflict left overs and apply code format.

Original Pull Request: #4807
christophstrobl pushed a commit that referenced this pull request Oct 15, 2024
We now retain expression options when resolving bind values from the original BsonRegularExpression.

Closes: #4806
Original Pull Request: #4807
christophstrobl added a commit that referenced this pull request Oct 15, 2024
Add tests to verify expected behaviour of quoted expressions and string that look like expressions.
Remove comments that look like old merge conflict left overs and apply code format.

Original Pull Request: #4807
christophstrobl pushed a commit that referenced this pull request Oct 15, 2024
We now retain expression options when resolving bind values from the original BsonRegularExpression.

Closes: #4806
Original Pull Request: #4807
christophstrobl added a commit that referenced this pull request Oct 15, 2024
Add tests to verify expected behaviour of quoted expressions and string that look like expressions.
Remove comments that look like old merge conflict left overs and apply code format.

Original Pull Request: #4807
@christophstrobl christophstrobl deleted the issue/4806 branch October 15, 2024 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ParameterBindingJsonReader doesn't consider regex options
2 participants