Use exact matching for `IN` clause with ignore case #4412

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.

Sign up for GitHub

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

Jump to bottom

Closed

christophstrobl wants to merge 3 commits into main from issue/4404

Member

christophstrobl commented Jun 13, 2023

Prior to this change the generated pattern would have matched more entries than it should have. The behavior is now aligned to its counterpart not using the IgnoreCase flag.

Resolves: #4404

christophstrobl added 3 commits

June 13, 2023 09:57


          Prepare issue branch.

caf3841


          Use exact matching for IN clause with ignore case.

54d7e12

Prior to this change the generated pattern would have matched more entries than it should have. The behavior is now aligned to its counterpart not using the IgnoreCase flag.


          Polishing.

7977a25

Mark method potentially returning null as such and remove unused imports.

christophstrobl assigned mp911de

mp911de changed the title ~~Use exact matching for IN clause with ignore case.~~ Use exact matching for IN clause with ignore case

mp911de added the type: bug label

mp911de added this to the 3.4.13 (2021.2.13) milestone

mp911de pushed a commit that referenced this pull request


          Use exact matching for IN clause with ignore case.

d6227e5

Prior to this change the generated pattern would have matched more entries than it should have. The behavior is now aligned to its counterpart not using the IgnoreCase flag.

Closes #4404
Original pull request: #4412

mp911de pushed a commit that referenced this pull request


          Polishing.

57688d8

Mark method potentially returning null as such and remove unused imports.

See #4404
Original pull request: #4412

mp911de added a commit that referenced this pull request


          Polishing.

5d57100

Use extended switch syntax.

See #4404
Original pull request: #4412

mp911de pushed a commit that referenced this pull request


          Use exact matching for IN clause with ignore case.

aecfd45

Prior to this change the generated pattern would have matched more entries than it should have. The behavior is now aligned to its counterpart not using the IgnoreCase flag.

Closes #4404
Original pull request: #4412

mp911de pushed a commit that referenced this pull request


          Polishing.

e2dc76e

Mark method potentially returning null as such and remove unused imports.

See #4404
Original pull request: #4412

mp911de added a commit that referenced this pull request


          Polishing.

4d51d27

Use extended switch syntax.

See #4404
Original pull request: #4412

mp911de pushed a commit that referenced this pull request


          Use exact matching for IN clause with ignore case.

c48b2c6

Prior to this change the generated pattern would have matched more entries than it should have. The behavior is now aligned to its counterpart not using the IgnoreCase flag.

Closes #4404
Original pull request: #4412

mp911de pushed a commit that referenced this pull request


          Polishing.

9038d7c

Mark method potentially returning null as such and remove unused imports.

See #4404
Original pull request: #4412

mp911de added a commit that referenced this pull request


          Polishing.

40567df

Use extended switch syntax.

See #4404
Original pull request: #4412

mp911de pushed a commit that referenced this pull request


          Use exact matching for IN clause with ignore case.

746199f

Prior to this change the generated pattern would have matched more entries than it should have. The behavior is now aligned to its counterpart not using the IgnoreCase flag.

Closes #4404
Original pull request: #4412

mp911de pushed a commit that referenced this pull request


          Polishing.

37fb6f5

Mark method potentially returning null as such and remove unused imports.

See #4404
Original pull request: #4412

Member

mp911de commented Jun 14, 2023

That's merged and backported now.

mp911de closed this

mp911de deleted the issue/4404 branch

June 14, 2023 08:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels