Change filter typehint from string to mixed

Created on 29 July 2025, 14 days ago

Problem/Motivation

In \Drupal\ai_search\AiVdbProviderSearchApiInterface::prepareFilters() we return mixed type. However, in \Drupal\ai\AiVdbProviderInterface::querySearch() and vectorSearch() we require a string type.

The doc blocks state:

   * @param mixed $filters
   *   The filters as prepared by the VDB provider in ::prepareFilters().

Which contradicts the param type.

Steps to reproduce (required for bugs, but not feature requests)

Proposed resolution

Change the string $filters = '' param in querySearch() and vectorSearch() to accept mixed type.

Remaining tasks

Optional: Other details as applicable (e.g., User interface changes, API changes, Data model changes)

🐛 Bug report

Status

Active

Version

1.2

Component

...to be triaged

Created by

🇦🇺Australia kim.pepper 🏄‍♂️🇦🇺Sydney, Australia

Live updates comments and jobs are added and updated live.

Merge Requests

!799Change filter typehint from string to mixed
Open
🇦🇺Australia kim.pepper
updated 14 days ago

Comments & Activities

Issue created by @kim.pepper
Merge request !799[#3538476] Change filter typehint from string to mixed → (Open) created by kim.pepper
Comment 14 days ago →
🇦🇺Australia kim.pepper 🏄‍♂️🇦🇺Sydney, Australia
Comment 14 days ago →
🇦🇺Australia kim.pepper 🏄‍♂️🇦🇺Sydney, Australia
We might want a followup to make a FiltersInterface instead of mixed?
Comment 14 days ago →
🇦🇺Australia kim.pepper 🏄‍♂️🇦🇺Sydney, Australia
Pipeline finished with Success
14 days ago
Total: 349s
#559191
Comment 10 days ago →
🇩🇪Germany stmh
Good idea. To get sth working for qdrant i am serializing and deserializing my filters array for now.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024