Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
After some of the recent updates, it looks like og is throwing errors on the user profile edit pages. See attached screenshot for more.
I've verified this happens on a completely stock new install of commons without any patches, other than what comes with the make file.
Comment | File | Size | Author |
---|---|---|---|
#10 | 2225889-realname-correct-menu-10.patch | 3.63 KB | fenstrat |
| |||
#4 | 2225889-realname-correct-menu-3.patch | 3.65 KB | japerry |
Screenshot_3_25_14_1_03_PM-2.png | 192.89 KB | japerry |
Comments
Comment #1
japerryFurther investigation from the last comment in #1722112: Autocomplete for entityreference fields reveals that the code replaces the functionality of the entityreference module, but doesn't confirm to the API entityreference uses.
This causes other modules that rely on altering entity reference autocomplete to fail. The patch below fixes that by bringing the menu callbacks in line with what entity reference uses.
You can reproduce by enabling OG adding a user group reference. (or just use the latest commons dev snapshot, which includes the newest version of realname)
Comment #2
scottzak CreditAttribution: scottzak commentedApplying this patch breaks og_ui_add_users form. This was working in Commons 3.9 and no longer works in 3.11. When I undo the patch it begins working again. We're not using trusted contacts, so we don't encounter the original issue (if I understand the issue correctly).
Comment #3
hass CreditAttribution: hass commentedComment #4
japerryTry this patch out, it should work better with OG and user reference callbacks.
Comment #5
scottzak CreditAttribution: scottzak commentedThank you very much for working on this. I'll give it a good look.
Comment #6
eacute CreditAttribution: eacute commentedI confirm that the patch #3 fix the problem with commons 3.13
Comment #7
scottzak CreditAttribution: scottzak commentedPatch #3 seems to solve the problem for me. Thanks again!
Comment #8
hass CreditAttribution: hass commentedComment #9
yepaPatch #3 remove notices.
Comment #10
fenstratReroll of #4.