- Issue created by @loze
- πΊπΈUnited States loze Los Angeles
Not exactly sure why, but apparently inside the
.each()
functions$(this)
was undefined. so instead I changed it to use the element parameter in the each function. This MR seems to resolve the issue for me. - Status changed to Needs review
11 months ago 4:43am 15 June 2024 - πΊπΈUnited States loze Los Angeles
Apparently due to π Allow for deletion of a single value of a multiple value field Fixed adding a remove button to multi value reference fields, this module's js was triggering the 'remove' button instead of the 'add more' button of the hidden member reference field, resulting in js errors when attempting to add members to a thread.
My MR attempts to clear this up.
- π΅π±Poland alorenc Wolsztyn, π΅π±
I am not able to replicate the issue.
* I pulled the latest 3.0.x version,
* visited /private-message/create
* used autocompletion
* I didn't notice any errorsCan you verify the issue on the latest version?
- π΅π±Poland alorenc Wolsztyn, π΅π±
Issue with $config initialization is covered by https://www.drupal.org/project/private_message/issues/3485607 π Error when entering comma-separated usernames Active , thanks for help!
- π·π΄Romania claudiu.cristea Arad π·π΄
Postponed to get more feedback from @loze
- πΊπΈUnited States loze Los Angeles
I believe this is the same issue being described in π Sending private message to more than one recipient does not work Needs work and I believe started with Drupal 10.3
The issue appears when adding more than one user/recipient to a private message.
When adding the 2nd recipient, this modules js is targeting the "add more" button incorrectly. The js attempts to simulate a click on the add more button, but instead targets the first "remove" button of the hidden autocomplete field.
when this was first written core did not have a remove button on the autocomplete field.
- πΊπΈUnited States loze Los Angeles
But this may be fixed with π Refactor private_message_thread_member_widget Active