Skip to content

Conversation

@anderly
Copy link
Member

@anderly anderly commented Sep 4, 2025

Add:

  • whereContains
  • orWhereContains
  • whereNotContains
  • orWhereNotContains

Closes #148 and #99

@anderly anderly requested a review from Copilot September 4, 2025 03:55
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements four new query builder methods for string containment filtering: whereContains, orWhereContains, whereNotContains, and orWhereNotContains. These methods provide OData filtering capabilities using the contains() function and indexof() function for negative matching.

Key changes:

  • Added four new query builder methods with comprehensive test coverage
  • Implemented corresponding grammar compilation methods to generate proper OData filter syntax
  • Version bump to 0.10.2

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/Query/Builder.php Implements the four new whereContains methods with proper method chaining
src/Query/Grammar.php Adds grammar compilation methods to generate OData filter syntax
tests/Query/BuilderTest.php Unit tests verifying correct OData query generation
tests/ODataClientTest.php Integration tests with real data validating end-to-end functionality
composer.json Version bump from 0.10.1 to 0.10.2

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@anderly anderly merged commit 3b14131 into master Sep 4, 2025
12 checks passed
@anderly anderly deleted the fix/148-where-not-contains branch September 4, 2025 05:17
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.

where field value does not contain wildcard

2 participants