- Issue created by @ricardofaria
- Status changed to RTBC
over 1 year ago 12:32pm 18 May 2023 - Issue was unassigned.
- Status changed to Needs work
over 1 year ago 10:22am 19 July 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The steps to reproduce are missing. What did you do to get that error message?
- 🇮🇹Italy apaderno Brescia, 🇮🇹
$originalKeys = $query->getOriginalKeys(); - $lowerOriginalKeys = mb_strtolower($originalKeys); + $lowerOriginalKeys = mb_strtolower($originalKeys ?? '');
Drupal core does not implement any
getOriginalKeys()
method.
I do not know which method that code is supposed to call, but the reason for$lowerOriginalKeys
to beNULL
is that it is initialized with a call to a non-existing method. - 🇮🇹Italy apaderno Brescia, 🇮🇹
I found it: It is the
Drupal\search_api\Query\QueryInterface::getOriginalKeys()
method, which is documented to either return an array, a string, orNULL
. The caller should not assume it returns a string.Still, in a bug report, the title and quoting an error/exception are not sufficient. A bug report should also describe the steps necessary to reproduce the bug.
- Assigned to roshni27
- Merge request !8deprecated mb_strtolower and handle getOriginalKeys method return if array ,string or null → (Merged) created by roshni27
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 5:43am 6 September 2023