Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Hi,
Thank you for your work on this project.
Trying to use the module with an entity browser configured with a view as selection manager, but unfortunatly, the multi-selection is not working after appying ajax filters.
Notice that the double click on single item works perfectly.
| Comment | File | Size | Author |
|---|---|---|---|
| #22 | entity_browser_enhanced_8_1_beta6_fix_handler_attachment_02.patch | 1.6 KB | brtamas |
| #21 | entity_browser_enhanced_8_1_beta6_fix_handler_attachment_01.patch | 6.12 KB | brtamas |
| #19 | 2894829-click-error-3.patch | 1.79 KB | k3vin_nl |
| #15 | 2894829-click-error-2.patch | 1.87 KB | k3vin_nl |
| #6 | 2894829-click-error.patch | 961 bytes | k3vin_nl |
Comments
Comment #2
rich.yumul commentedI've observed this behavior too.
Comment #3
rajab natshahThanks for reporting :)
Troubleshooting the wrong behaviour.
Comment #4
k3vin_nl commentedLooks like the javascript doesn't respect the context properly, causing the event handler to be attached multiple times.
I have added a patch that should fix this.
Comment #5
k3vin_nl commentedComment #6
k3vin_nl commented(Updated the patch to use the correct file structure.)
Comment #7
rajab natshahComment #8
rajab natshahComment #10
rajab natshahCommitted ... Thank you :)
Comment #11
rajab natshahComment #12
rajab natshahComment #13
k3vin_nl commentedHi, thanks for accepting my patch.
I just saw however that when I created my patch, I missed one line.
The context also needs to be added on line 71:
$('form.entity-browser-enhanced.multiselect .view .views-col', context).on('dblclick', function () {Comment #14
extect commentedThe patch committed in #9 is causing problems and makes image selection stop working as soon as I click on apply in exposed filters in the view (using inline entity form, entity browser and entity browser enhanced). Double-click is working, though. Removing the changes from #9 makes everything work again.
The changes suggested in #13 make even double-click stop working.
Comment #15
k3vin_nl commentedI saw that the patch indeed did not fix the issue completely, so I did some more debugging.
After an ajax refresh the 'context' that would get passed to the entityBrowserMultiselect function was the complete form. This caused the $('form.entity-browser-enhanced.multiselect .view .views-col', context) selector to not select anything after an ajax refresh.
I changed this for a smarter way of selecting the columns, so this patch should hopefully fix the issue.
Comment #16
k3vin_nl commentedComment #17
merilainen commentedTested the functionality and works nicely. Maybe the console debug messages should be removed from code?
Comment #18
k3vin_nl commentedAh, yes, the debug messages shouldn't have been in there. I'll update the patch!
Comment #19
k3vin_nl commentedComment #20
rajab natshahThank you Kevin,
Testing the issue
console.log should be removed from your patch as you remarked.
Comment #21
brtamas commentedHi All,
I still had some error with the working. I attach another patch, maybe it solves more case.
Comment #22
brtamas commentedHere is a better one
Comment #24
rajab natshahCommitted ... Thank you :)
Comment #25
rajab natshah