Closed (duplicate)
Project:
Entity Reference Autocomplete
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Aug 2015 at 10:50 UTC
Updated:
22 Sep 2015 at 13:01 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
feyp commentedAttached is a patch against 7.x-1.x-dev
Comment #3
feyp commentedComment #4
mbrc commentedFeyP's patch fixes the issue for me.
Thank you!
Comment #5
David_Rothstein commentedThe first part of the patch looks good, but I think the second part is unnecessary. The value callback doesn't have anything to do with the Ajax system, and either way I don't think this module would have any reason to set the verification header on its own?
Here's a version that just includes the first part. Works fine for me.
I will add this to the Drupal 7.39 release notes.
Comment #6
feyp commentedDavid and mbrc, thanks for the review and for pointing out the useless second part. Seems that I confused the ajax callback mentioned in the release notes and the value callback... I can confirm, that the revised version of the patch works fine, so setting this back as RTBC.
Comment #7
galooph commentedPatch from #5 works for me. - Thanks!
Comment #8
drubbHere's a quick one if you don't want to patch the module: just implement hook_element_info_alter() in a custom module!
Comment #9
Leeteq commentedComment #10
eric_a commentedI think #5 is going to cause a fatal error on 7.38 so it seems that either a dependency on >=7.39 or a function_exists() is needed.
Comment #11
slv_ commentedThanks everybody for the ideas and suggested patches. As it's been pointed out, there's a need for the dependency in the .info file, or the function_exists check.
I've solved this with merging the 2 patches provided in #2561431: Elements with autocomplete require #autocomplete_input since Drupal core 7.39., which is a duplicate of this (it was created after this one), but used it because the function_exists check was provided already in one of the patches there.
So, fixed now in dev, and rolling out a new release now. Thanks all \o/.
Comment #12
slv_ commented