Skip to content

Adding support for case insensitive like constraint#297

Open
abitoprakash wants to merge 4 commits into
balderdashy:masterfrom
abitoprakash:feat/make-like-modifier-case-insensitive
Open

Adding support for case insensitive like constraint#297
abitoprakash wants to merge 4 commits into
balderdashy:masterfrom
abitoprakash:feat/make-like-modifier-case-insensitive

Conversation

@abitoprakash

Copy link
Copy Markdown

Issue - balderdashy/sails#4628

Change log

  • Passing meta from query to compileStatement from following methods
    • avg
    • count
    • destroy
    • select
    • sum
    • update
  • Adding meta as parameter to compileStatement
  • Adding modify-where-clause which checks for makeLikeModifierCaseInsensitive in meta and replaces LIKE with ILIKE if present
  • Adding test cases for insensitive queries
  • Minor eslint fix in lib/private/redact-passwords.js (Missing semicolon)
Screenshot 2024-09-08 at 1 06 47 AM

@abitoprakash

Copy link
Copy Markdown
Author

@eashaw, @DominusKelvin Let me know what you guys think

@DominusKelvin

Copy link
Copy Markdown

Thanks for working on this @abitoprakash, I'll have a look shortly.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants