Deleting an RSS from from within a group results in a dead white screen.

I'm looking into this issue now and will post a fix shortly.

CommentFileSizeAuthor
#3 og_aggregator-fix-wsod.patch1.52 KBmstef

Comments

mstef’s picture

mstef commented
Status: Active » Needs review

Line 329

SHOULD BE:

$form_state['redirect'] = 'node/'.$form_state['values']['og_id'].'/aggregator';

NOT:

$form_state['redirect'] = 'node/'.$form_state['values']['og_id'].'/feed';

I'm not sure why /feed was used - it doesn't exist.

I also suggest moving menu_rebuild() to the bottom of the function.

mstef’s picture

mstef commented

Line 319 also needs to follow these changes.

mstef’s picture

mstef commented
StatusFileSize
new og_aggregator-fix-wsod.patch1.52 KB

Patch

rootwork’s picture

rootwork
he/him
Primary language English
Location 🇺🇸 US-Pacific 🌎 Chinook (Multnomah, Clackamas) & Cowlitz lands 🌹 Portland, OR
commented

This works for me. Can we get it into a new version, or (if only one review isn't sufficient) at least a new dev?

Taz’s picture

Taz commented
Version: 6.x-1.4 » 6.x-1.x-dev
Assigned: Unassigned » Taz
Status: Needs review » Fixed

It was originally /feed but due to the auto url alias module that just tacks /feed on the end of groups, I changed it to aggregator in the menu - but missed these redirects.

Patched and will be in the next dev release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.