Complex sections often don't work

Perhaps someone else out there can provide a better title for this bug report.

I have 3 sections.

1st one for logged-in users who are editing their own account, own content, etc. This one uses PHP settings for visibility.
2nd one is for specific pages and views. This one simply uses paths for visibility.
3rd one is the default one. When the above 2 don't match, this one kicks in.

The trouble is that the 1st one simply wasn't working - ever. I messed with weighting & the PHP and never came right. That's when I decided to investigate the module code. Something wasn't right.

What I found was that the $output variable in the _sections_in_section() function was being overwritten within the module, making it impossible for more complex section setups to work. The fix was very simple and now my site's working 100%. Here it is...

Line 159; Replace this:

$output = $row;

With this:

return $row;

Do the EXACT same thing at line 164.

What this does is it returns the correct section as soon as it finds one that works. The rest is handled by your paths and weights. It works very, very well. I'm hoping this helps someone else out there!

PS: If someone could roll a patch and post it in a comment - that would be great. I'm a bit useless with patches.