- last update
over 1 year ago 782 pass - last update
over 1 year ago Patch Failed to Apply - ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
Just picking up on a small change to this, Drupal 10.1
`$query = DefaultSelection::buildEntityQuery($match, $match_operator);`
$match adds a condition of name
so you end up with:
where
...
name CONTAINS "aaa"
and
(name CONTAINS "aaa" or email CONTAINS "aaa")NULL'ing the match works in my use case.
- Status changed to Needs work
10 months ago 3:53pm 15 January 2024 - ๐บ๐ธUnited States rszrama
@jsacksick I don't think we want to wait on a core improvement to the selection entity. If it really is this simple to resolve, let's do it. That said, while the patch does work for me to search by partial match on email address in addition to username, it doesn't show the email address in the autocomplete suggestions.
I'm going to move this to "Needs work", and we need some way to ensure the autocomplete suggestions include the searched parameters, like:
Username <email.address@example.com> (UID)
- ๐บ๐ฆUkraine marchuk.vitaliy Rivne, UA
vmarchuk โ made their first commit to this issueโs fork.
- Merge request !221Issue #3035268: Support searching by email address on order add / reassign forms. โ (Merged) created by Unnamed author
- last update
10 months ago 790 pass - Status changed to Needs review
10 months ago 3:53pm 16 January 2024 - ๐บ๐ฆUkraine marchuk.vitaliy Rivne, UA
@rszrama
MR is ready for review https://git.drupalcode.org/project/commerce/-/merge_requests/221.
See how it looks on the screenshot:
- Status changed to RTBC
10 months ago 5:29pm 17 January 2024 - ๐บ๐ธUnited States rszrama
Looks great to me! And just as a note, we can't easily include the uid in the autocomplete suggestion, because it would be duplicated on selection. I think it's fine for the suggestions to just be
John Doe <john.doe@example.com>
. Tested as working on the order create and reassign forms. - ๐ฎ๐ฑIsrael jsacksick
One thing I don't really understand from the code is how the following works?
$query = DefaultSelection::buildEntityQuery($match, $match_operator);
The
buildEntityQuery()
is non static so how does this not crash? - ๐ฎ๐ฑIsrael jsacksick
Ok so although it may look like a static call, it isn't, and the tests are passing... Merging this! Thanks Vitaliy.
-
jsacksick โ
committed ccb1b450 on 8.x-2.x
Issue #3035268: Support searching by email address on order add /...
-
jsacksick โ
committed ccb1b450 on 8.x-2.x
- Status changed to Fixed
10 months ago 8:17am 22 January 2024 -
jsacksick โ
committed d3d2e649 on 3.0.x
Issue #3035268: Support searching by email address on order add /...
-
jsacksick โ
committed d3d2e649 on 3.0.x
Automatically closed - issue fixed for 2 weeks with no activity.