Points in collapsing forums block.

horrorcode · Posted: Thu Jan 29, 2009 2:27 am

Didnt really want to open a new topic but than saw how many exist for the block so here goes.

I used the Top Users Points block and noted what was needed to apply this, sorry but Im making this simple so I dont drag on, this isnt a future request but it could easily be added.
Live preview(note my other changes):

Only registered users can see links on this board!
Get registered or login to the forums!

EDIT: It changed the code here so you should probably read these instructions too.
Full instructions can be read:

Only registered users can see links on this board!
Get registered or login to the forums!

Notice in the instructions all nbsp's were converted, so that needs to be considered. Very simple edits just add "points," before user_avatar, so the new code would look like this:

Code:

$sql = "SELECT user_id, username, user_posts, points, user_avatar,

and

Code:

while(list($user_id, $username, $user_posts, $points, $user_avatar,

Than I just threw it before Posts: so it wouldnt take forever, so that section(for me) looks like this:

Code:

<br />Points: $points    <br />Posts:

Hopefully you can figure out the "nbsp" and maybe someone here can come up with a better way of doing this, I know I didnt enclose the "Points:" in "" but I didn't see it as needed. So an explanation on that for a non-link would also be great.

Anyways, hope you like Very Happy

Raven · Posted: Thu Feb 05, 2009 12:57 am

I just caught up with this - sorry! I think that's a nice mod. I don't use points but I know there are many who do. I don't know if you added this as an optional setting but that's how I would add it. Something like:

$showTopPostersPoints = true; // true=yes - false=no

Thanks!

horrorcode · Posted: Thu Feb 05, 2009 6:41 am

Thanks, I took what you said and went with it, took me a couple hours but hows this..

I started by adding exactly what you gave me, in between Ranks and Num so it looks like this:

Code:

$showTopPostersRanks = 4; // 0=None, 1=Admin only, 2=Moderator only, 3=Admin and Moderator, 4=All
$showTopPostersPoints = TRUE; // true=yes - false=no
$showTopPostersNum = 5; // Total number of top posters to show

Than you still need to add the "points" so it looks like this:

Code:

$sql = "SELECT user_id, username, user_posts, points, user_avatar,

and this:

Code:

while(list($user_id, $username, $user_posts, $points, $user_avatar,

Next what took me the longest is figuring out to add the if statement and I'm not sure if its proper but heres what I did

In the end of this line:

Code:

$content .= " $user_posts <br />";

the only way I could get it to display correctly is to change the "</td>" to the "<br />"...problem here is where else can I add the td, I tried an if statement and it wont work here, I added this on the next line so it looks like:

Code:

$content .= "<td align=\"center\"> <a href=\"profile-.html$user_id\" $staffTitle><b>$username</b></a> <br /> Posts:";
$content .= " $user_posts <br />";
if ($showTopPostersPoints) {
$content .= " Points: $points </td>";
}
if (!($cycle%$showTopPostersPerRow)) $content .= "</tr>";
$cycle++;

And you can see it here:

Only registered users can see links on this board!
Get registered or login to the forums!

After changing it to false it still works, but is it a problem that the td tag isnt closed in that line or is it closing anyways? Also, this is the original block if you want me to post it let me know, the way I had it before was totally different.. I couldnt figure out how to get the avatars and everything in one row without making signifigant changes.

trunks · Posted: Thu Feb 05, 2009 8:41 am

Great development of such a useful block created by raven! I know i personally use it and i also make great use of the points system, so its definatly a +++ for me to have. I think it will also add another level of interaction with both site and users themselves.

Nice work horror,

Trunks

horrorcode · Posted: Thu Feb 26, 2009 8:32 pm

Looks like this got lost lol... Has anyone tested this out? I still havent figured out what to do about that td tag, but maybe it would be ok on the next line after $cycle? Which would be here:

Code:

if ($showTopPostersPoints) {
$content .= " Points: $points </td>";
}

if (!($cycle%$showTopPostersPerRow)) $content .= "</tr>";
$cycle++;
}

$content .= "</table></div><br />";
}

The br would stay where td was changed in the previous post, but the td would be moved to after the tr, which might be a problem. But maybe it could also be in that if statement before the tr.

trunks · Posted: Fri Feb 27, 2009 4:30 pm

What is it your trying to achieve there horror?

horrorcode · Posted: Fri Feb 27, 2009 6:35 pm

Just compliance really, need to make sure it validates properly but need someone to test it.

horrorcode · Posted: Sat Feb 28, 2009 1:22 am

I think it would be best to just add td between the 2 if statements, so it would look like this:

Code:

if ($showTopPostersPoints) {
$content .= " Points: $points ";
}
$content .= "</td>";
if (!($cycle%$showTopPostersPerRow)) $content .= "</tr>";
$cycle++;
}

$content .= "</table></div><br />";
}

Then this is done, just need a tester now.

Edit: I think we can call this finished I went ahead and tested it and it seems fine. Ill compress and post soon, on linux right now so not having the best time navigating through my ntfs clutter. Bang Head

trunks · Posted: Sat Feb 28, 2009 1:48 pm

Well hit me up on msn and we'll get this tested for you... compile it as a seperate block for time's sake for now Smile

-Trunks