Bug #14716
closedMy Commons activity blank
0%
Description
Hello, it seems that the My Commons streams may be acting up. When I first go there I can see All Activity, but toggling back and forth to Groups / Sites / etc I don't see anything (see attached), and I can't get back to All Activity being populated either. I just see the yellow error message. Sonja first noticed this, so it seems at to be at least somewhat widespread.
Files
Related issues
Updated by Raymond Hoh over 4 years ago
- Category name set to My Commons
- Status changed from New to Assigned
- Assignee changed from Boone Gorges to Raymond Hoh
- Target version set to 1.18.17
I'm looking into this. Might be related to a recent change to BuddyPress.
Updated by Raymond Hoh over 4 years ago
- Related to Bug #13352: jQuery update for bp-nelo added
Updated by Raymond Hoh over 4 years ago
Fixed in https://github.com/cuny-academic-commons/cac/commit/97da394f23571626e0cbe637c2cfa8fbbebb2322.
Issue is due in part to #13352. If you're interested in the cause, continue reading :)
When we switched out the older jQuery Cookie library for js-cookie, the bp_get_cookies() function needed to be rewritten to use js-cookie's logic. I made a previous attempt at addressing this in https://redmine.gc.cuny.edu/issues/13352#note-9, however this did not account for an edge-case where a non-existing value could be passed.
Specifically, the filter value here: https://github.com/cuny-academic-commons/cac/blob/ea215fb837088e4ad90f31e6422ade7446617e55/wp-content/themes/bp-nelo/_inc/global.js#L216. On the "My Commons" page, the activity dropdown filter is not rendered, so this value is undefined. The previous cookie logic already took this into account, but not the rewritten version.
Should be fixed now. This is added as a hotfix on production. You might need to refresh the "My Commons" page or purge your browser cache if the fix is not working.
Updated by Raymond Hoh over 4 years ago
- Status changed from Assigned to Resolved
Updated by Colin McDonald over 4 years ago
Looks good Ray, thanks for the fast work on this!
Updated by Boone Gorges over 4 years ago
Ack, thank you for the quick diagnosis and fix, Ray!