Theme support

[Maxx1]

Sorry Guys had many personal troubles> I'm back if you let me stay? I'll help support Bloc's or anybodies themes, if possible! I don't need to be on any team or such just here to help out like I promised!

Kind regards,.
maxx

[IchBin]

Sounds good Maxx. You are free to help out anywhere you want around here. Hope the personal troubles subside for you.

[Maxx1]

Thanks Brad!!!

regards,
Maxx

[Rus]

An old thread but it's right in line with what I'm looking for.  I'm using the Pandora theme from once upon a long time ago.  I know it was written for RC5 but I like it and it still works.

It does put an error in my error log when anyone clicks on unread posts and there are unread posts to see:


http://www.tallingroth.ca/index.php?action=unread
Apply Filter: Only show the errors with the same message
8: Undefined index:

File: /home/cy4410/public_html/tallingroth/Themes/pandora_smf20rc5b/Recent.template.php
Line: 209

The line in question is this one:

            $mootip = $topic[$which_post]['preview'];

from this area of code:

Code Select Expand


if(!empty($settings['showmootips']))
{
$which = !empty($settings['firstpreview']) ? 'first' : 'last';
$which_post = $which.'_post';
$mootip = '
<img src=' . (!empty($avvy[$topic[$which_post]['member']['id']]) ? $avvy[$topic[$which_post]['member']['id']] : $settings['images_url'].'/TPguest.png') . '  class=mooavy />
<span class=greysmalltext>' . $which .' post by <b>'.$topic[$which_post]['member']['name'].'</b><br>' . $topic[$which_post]['time'] . '</span>
<hr><div> '. $topic[$which_post]['preview'].'</div>';
}
else
$mootip = $topic[$which_post]['preview'];


I've tried declaring those variables up top in a multitude of ways but can't seem to get it right.  Any help would be great.  It's not killing the forum by any means but sometimes it does bury other errors I'm trying to find when I do tweaks.

Thanks guys.

[IchBin]

Move these two lines above the if statement.

$which = !empty($settings['firstpreview']) ? 'first' : 'last';
$which_post = $which.'_post';

Basically what is happening is that those two variables (in the if statement) are not getting set if you hit the (else statement) part of that code.

[Maxx1]

Thanks Brad!

I just got done installing the Pandora rc5b on my smf 2.0.4. test site and do not have any issues at all with it so far, no errors in the log. FYI

regards,
maxx

[Rus]

Thanks!  That has indeed eliminated those errors.  I had tried to declare them at the top of the function in several ways but nothing seemed to completely fix it, I just kept making new errors.