In forumdisplay.php you have the alibity to select your daysprune as the forum default, all, etc. What I would like to do is add the option to display those messages since you last logged on as an option.

So you have

and then

What I would like to add is for $daysprune to check against a member's last time on and compare it to the time and date now and then only display those posts that meet this criteria.

Is this possible to do or would it be a larger strain on the server?

Parker

actually I think that would be fairly easy - and quite a nice idea, Parker. Give me a few to look at the code, brb :)

Parker try this (I've not tested it yet):

in forumdisplay.php find (around line 311):

Replace it with:

Add <option value='-2'>Since your last visit</option> to the daysprune select.

That's it.

Bira:

Thanks again. I couldn't do this stuff without you. Now in the modifyoptions template you have

would you then use:

<option value="-2" $days-2selected>Show thread from last visit</option>

so that they can select this option in their control panel options?

Parker

no, you can't have - there, in PHP that means $days - 2selected (minus).

You'll need a bit more "serious" hacking in there -- create a new $xxxselected variable and do some if's. Same as $defaultselected, just say $lastvisitselected.

It means adding it to functions.php as well, if I remember correctly.

If I need to do all that for you, I might as well write the hack ;) -- just look around at how $defaultselected behaves and create a new variable

Bira:

Ok, thanks I will look around and figure out what is going on.

In the above I should have given you the rest of the code as just putting in the suggestion you gave me gives me a parsing error.

The whole thing is:

$datecut="";
$stickyids="";
$stickycount=0;
if ($daysprune!=1000) {
$checkdate = time() - ($daysprune*86400);
$datecut='AND lastpost >= '.$checkdate;
if ($pagenumber==1) {
$datecut.=' AND sticky=0';
$stickies=$DB_site->query("SELECT threadid,lastpost FROM thread WHERE forumid='$foruminfo[forumid]' AND visible=1 AND sticky=1 $limitothers");
while($thissticky=$DB_site->fetch_array($stickies)) {
//if ($thissticky['lastpost']<$checkdate) {
$stickycount++;
//}
$stickyids.=",$thissticky[threadid]";
}
}
}

How would I implement your coding suggestion into this?

Thanks again,
Parker

Parker I just tested the code I gave you in post #3 and it works perfectly. It's the only code you need to replace in forumdisplay.php (unless you want to add a user-selectable default option, of course).

Parker what version are you on?!?

The vinyl code for 2.2.1 in forumdisplay.php is (lines 307-314):

Adding the lastvisit option would change it to:

(change hilighted)

Bira:

I am using 2.2.0 and the code is

$datecut="";
$stickyids="";
$stickycount=0;
if ($daysprune!=1000) {
$checkdate = time() - ($daysprune*86400);
$datecut='AND lastpost >= '.$checkdate;
if ($pagenumber==1) {

instead of

$datecut="";
$stickyids="";
$stickycount=0;
if ($daysprune!=1000) {
$checkdate = time() - ($daysprune*86400);
$datecut='AND lastpost >= '.$checkdate;
}
if ($pagenumber==1) {

It must be a code change between the two versions. I will give what you have a try.

Parker

Bira:

I found that the way it is in 2.2.0 has an extra right bracket at the end of the next if statement.

So I had to change

$stickyids.=",$thissticky[threadid]";
}
}
}

to

$stickyids.=",$thissticky[threadid]";
}
}

to get rid of the parse error.

Parker