- Issue created by @dpi
- ๐ฆ๐บAustralia dpi Perth, Australia
As discovered by https://git.drupalcode.org/project/pusher_mini/-/merge_requests/1
- ๐ฎ๐ณIndia aditi saraf
Aditi Saraf โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia samir_shukla bareilly
Hi, created the patch for the issue. Please review.
- Status changed to Needs review
over 1 year ago 5:06am 30 January 2024 - last update
over 1 year ago 8 pass - First commit to issue fork.
- Merge request !39Fix: Key::setKeyValue can accept array, documented as only accepting string. โ (Open) created by rajeshreeputra
- ๐บ๐ธUnited States cmlara
This method can in fact accept arrays for use with multi-value keys.
Can it ?
KeyTypeMultivalueInterface calls for a serialize method to be present which allows converting the multiple value to string. To me that would imply that these should only be strings by the time it reaches the storage layer.
This sounds more like the referenced project was actually using the Key module outside the API and PHPstan correctly flagged it.
Needs work for method signature change, though I believe this is more likley works as designed.