Closed (fixed)
Project:
Drupal Commons
Version:
7.x-3.x-dev
Component:
Miscellaneous
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
9 Apr 2013 at 19:54 UTC
Updated:
25 Jun 2014 at 21:30 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
japerryI tested that patch and it works well. Shalt I add it to the make file ezra?
Comment #2
ezra-g commentedSeems worth bumping the Entityreference issue via a positive review on the patch and pinging amitaibu in IRC to see if we can get it committed first. I suggest doing that, waiting a day, and if there's no progress then adding the patch to the Commons make file.
Comment #3
ezra-g commentedCorrection: Bumping the issue review and pinging Davereid on the Realname patch :P.
Comment #4
ezra-g commentedI spoke to davereid in IRC and he suggested marking the issue as RTBC so he could commit it to Realname. Then seems our work would be instead to use a specific hash of that module :).
Comment #5
ezra-g commentedLooks like the referenced issue has been fixed in Realname. Let's bump the version number (and possibly help rolling a new release of the Realname module) to resolve this issue in Commons :).
Comment #6
devin carlson commentedAttached is a patch to update the real name module to the latest commit hash which includes the fix for entity reference autocompletes.
Unfortunately, the newer code doesn't seem to fix the problem; searching by real name didn't return any results, I had to search by the user name which was then displayed as the real name. It also caused a number of other errors and suggested replacing email tokens on the status report page.
Comment #7
ezra-g commentedBased on the previous comment, looks like we have more work remaining than was expected. Let's push this out of the 3.6 release.
Comment #8
japerryThe root cause here is that Entity Reference has its own callback within the EntityReference_SelectionHandler_Generic.class
Since we want realname to just alter that query, we need to implement a query alter because there is a tag for anything 'entityreference'
The autocomplete for realname isn't at issue here and works properly. Related to this issue: #2261665: Real name changes the entity reference behavior with views
Comment #9
japerryComment #10
japerryMarking needs review, I believe my latest patch #3 in #2225889: Notice: Undefined offset: 5 in _og_field_widget_replace_autocomplete_path() fixes this problem, or at least it has for me :-) Apply the patch and see if this error for event organizer still occurs.
Comment #12
japerryThe 3rd patch in #2261665: Real name changes the entity reference behavior with views fixes this issue. Pushed to the drupal-org.make file.