HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.

Quick change to popular posts list items MostPopular plugin

Noticed a small bug. In class.mostpopularmodule.php, if the loop that adds new list items (ie popular posts) runs out of items and returns, then the html outputted will create a list that envelopes whatever module or html is below it on the page. I added a small fix around lines 40-43 (I changed other things in the script so I'm not sure where this is in the original)...look for the line:

if($DiscID < 1) return;

And I changed this to:

if($DiscID < 1) { echo "</ul></div>"; return; }

Looks like this effectively closes that loop with the end ul and div tags in the event of the loop running out of new list items to append.

I made a few other changes to the script (added the original user's photo, and added the ability to not include private category posts), but I won't post the changes unless peregrine wants them. Hope this helps someone.

Comments

Sign In or Register to comment.