Project

General

Profile

Actions

Bug #21743

closed

Can't access groups in Reclaim environment

Added by Colin McDonald 17 days ago. Updated 11 days ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
-
Target version:
Start date:
2025-01-11
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

Please see screenshot attached. Having trouble with newly created and existing groups. I see just the first snippet of the group header and nothing else.


Files

groups-truncated.mov (25.5 MB) groups-truncated.mov Colin McDonald, 2025-01-11 12:05 AM
Actions #1

Updated by Boone Gorges 17 days ago

  • Status changed from New to Testing Required

Since the early-Dec snapshot, the production site was modified so that bp-reply-by-email uses 'inbound' rather than 'imap' mode. That means that the Reclaim snapshot was configured to use 'imap'. But the imap PHP extension is not available on that environment. As such, the preflight checks in

bp_rbe_is_required_completed()
were failing. Because of this, the bp-rbe-new-topic plugin wasn't loading its library files https://github.com/cuny-academic-commons/cac/blob/9f127777f40b5fd7385709c715e40653fe57a504/wp-content/plugins/bp-rbe-new-topic/loader.php#L18. Elsewhere in the codebase, we call BP_RBE_New_Topic without checking for its existence https://github.com/cuny-academic-commons/cac/blob/9f127777f40b5fd7385709c715e40653fe57a504/wp-content/plugins/cac-bp-custom-includes/forums.php#L434. That was triggering a fatal error.

I've manually changed the 'Mode' for RBE to 'inbound' and the problem goes away. This should not be necessary post-migration because the GC production database already has this value saved in the settings.

Ray, as an FYI: The function_exists check here https://github.com/cuny-academic-commons/cac/blob/9f127777f40b5fd7385709c715e40653fe57a504/wp-content/plugins/cac-bp-custom-includes/forums.php#L419 is not enough of a safety mechanism for the reasons outlined above. Maybe this is not a big deal - the fatal error caused us to investigate this failure rather than having it fail silently. But I wanted to flag it for you.

Actions #2

Updated by Raymond Hoh 17 days ago

Ugh. Thanks for finding this bug, Boone. I've added some better checks for RBE New Topic before using its code in https://github.com/cuny-academic-commons/cac/commit/a6d5ddeac9b67522276c05e39fa3293a21965264.

Actions #3

Updated by Boone Gorges 11 days ago

  • Status changed from Testing Required to Resolved
Actions

Also available in: Atom PDF