It passes that as-is to the database query, but it needs to be transformed to LIKE conditions.

CommentFileSizeAuthor
#4 3046496-interdiff-2-4.txt1.04 KBarpad.rozsa
#4 3046496-4.patch4.02 KBarpad.rozsa
#2 3046496-2.patch2.97 KBarpad.rozsa

Comments

Berdir created an issue. See original summary.

arpad.rozsa’s picture

arpad.rozsa commented
Status: Active » Needs review
StatusFileSize
new 3046496-2.patch2.97 KB
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Needs work

Nice trick with reusing the entity query API, didn't think of that.

Can we test this a bit in \Drupal\Tests\monitoring\Kernel\MonitoringCoreKernelTest::testDatabaseAggregator(), we should be able to add some alternative conditions with those operators quite easily.

arpad.rozsa’s picture

arpad.rozsa commented
Status: Needs work » Needs review
StatusFileSize
new 3046496-4.patch4.02 KB
new 3046496-interdiff-2-4.txt1.04 KB

Added tests for the two operators.

  • Berdir committed df78bc7 on 8.x-1.x authored by arpad.rozsa 
    Issue #3046496 by arpad.rozsa: STARTS_WITH and CONTAINS conditions for...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Nice!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.