Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I upgraded to OG 6.x-2.2 and am now getting the following error...
Fatal error: Cannot access empty property in /home/myaccount/public_html/sites/mysite.com/modules/og/modules/og_views/includes/og_views_handler_field_og_invite.inc on line 35
Comment | File | Size | Author |
---|---|---|---|
#7 | og.1413312-7.patch | 622 bytes | Grayside |
#6 | og.1413312-4.patch | 773 bytes | Grayside |
Comments
Comment #1
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedi have this issue too.
Comment #2
websitetechnician CreditAttribution: websitetechnician commentedsame on the latest version of open atrium (1.2)
Comment #3
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedi had to go through my views and delete og: invite links
Comment #4
Grayside CreditAttribution: Grayside commentedThis seems quite likely to be a regression related to backing out the upgrade of Notifications 4.x, or otherwise a mysterious bug.
@websitetechnician the error comes from OG 6.x-2.2, which was included in the latest release of Atrium.
Comment #5
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedI upgraded to the latest dev version of og to test and after clearing cache, apc, etc, I still got the WSOD on the pages where the link was
Comment #6
Grayside CreditAttribution: Grayside commentedAttached patch fixes the problem. It is not, however, a proper solution. I only had a few minutes for now to debug why the alias for nid isn't working, so reverted to what was already working in existing views.
Comment #7
Grayside CreditAttribution: Grayside commentedThe attached patch is the correct fix. Just want someone actually seeing this problem to confirm the fix.
The problem was introduced through an under-tested change to the OG Views integration. On the plus side, now that this problem was debugged, you can create invitation links by way of OG Relationships, where before those would have quietly failed.
Comment #9
Grayside CreditAttribution: Grayside commented#7: og.1413312-7.patch queued for re-testing.
Comment #11
Ainur CreditAttribution: Ainur commentedSubscribe
Comment #12
Grayside CreditAttribution: Grayside commented@Ainur, you know longer need to post to subscribe. Just hit the "Follow" button up near the top of the page.
Comment #13
Grayside CreditAttribution: Grayside commentedConfidant it works, getting bug out of the way.
http://drupalcode.org/project/og.git/commit/c643080
Comment #14
rfayThe problem with testing this one was that the branch was set to 6.x-2.2, I would imagine. See http://qa.drupal.org/pifr/test/218758
Comment #15
Grayside CreditAttribution: Grayside commented@rfay why would that be a factor? The tests haven't been changed since 2.2 and bugs should be set to the version in which they appeared...
Comment #16
rfayWell, the dependencies are quite different, at least as listed. But regardless... patches should always tested against dev in every project. That's what they'll be applied to.
Comment #17
Grayside CreditAttribution: Grayside commentedDependencies did not change at all. That was a puzzlement on it's own. I'll take this up in #1419530: OG6 Language Tests Failing
Comment #19
pfournier CreditAttribution: pfournier commentedA similar patch should be applied to og_views_handler_field_og_is_admin.
All views handlers using
$this->aliases['nid']
inrender()
should also have something like:Comment #20
Grayside CreditAttribution: Grayside commentedPlease create a new bug report. Referencing this issue would be a good idea :)
Comment #21
memcinto CreditAttribution: memcinto commentedSame issue, og_views_handler_field_og_is_admin, 6.x-2.4. Took a peek at the latest dev and the code in this include has not changed.
Error occurs when you try to add the field "Organic groups: OG: Is member an admin in a group" to a View.
Fatal error: Cannot access empty property in [site file path]/modules/og/modules/og_views/includes/og_views_handler_field_og_is_admin.inc on line 19
Comment #22
memcinto CreditAttribution: memcinto commentedSorry, inadvertently created duplicate comment.
Comment #23
pfournier CreditAttribution: pfournier commentedDone. See #1671326: Some views handler are broken.