Release 0.1 Mark Forum Read/Mark All Read from database hack

Here's the hack I worked on this week, in rough and not too pretty form. I'll be beautifying it soon.

You can download an rtf with all the code changes listed at http://murmurs.com/codechanges.rtf. Also in that file is the table creation statement.

Attached is a zip of all the changed files. Comments are included, but not consistant yet.

You can see this in action at http://tony.murmurs.com. Only two forums, but still.

I'll pretty up the code this weekend and make better install instructions.

This is way beta, so if there are any problems just post. Don't put this on a production board yet

Ethan

[ethank]

Originally posted by tubedogg
Don't be so sure...a database version of this may be just what I need (this cookie thing hasn't been working too well for me thus far).

Ahh...so it is still using cookies!

Well then...ONWARD WITH THE PROGRAM! I shall merge the other version of this code with mine, polish it up, format a very nice document on how to install it and release it this weekend.

2.? looks might cool though, outside of forum read

Ethan

[SWFans.net]

I'd like to say that one of the parts of ethank's hack that I appreciate, is the ability to have the mark individual forum read link on the forum index as well as within the specific forum.

I have always been a fan of redundancy and having options in more than one place.

[Christine]

I too am getting the "mark all forums read" as a result of trying to mark them individually.

I see where SW posted that he was able to redo the session.php mods, but I am a bit confused.

From that file, we have (around line 342)

PHP Code:

 $bbuserinfo['lastvisit']=$bbuserinfo['lastactivity']; 


And from line ~352, there is

PHP Code:

$bbuserinfo['lastvisit']=$bbuserinfo['lastvisit']; 


First, the instructions call to make the mods at the second instance of $bbuserinfo['lastvisit']=$bbuserinfo['lastactivity']; but I can only find the one case.

Secondly, there is an additional piece that is to be made at the $bbuserinfo['lastvisit']=$bbuserinfo['lastvisit']; but doesn't say which one - the one that Ethan added in the first group or the one that was there to begin with?

The remaining code (between 342 and 352) before modifications is

PHP Code:

 } else {
    if (!isset($bypass)) {
      if ($noshutdownfunc) {
        $DB_site->query("UPDATE user SET lastactivity=".time()." WHERE userid='$bbuserinfo[userid]'");
      } else {
        $shutdownqueries[]="UPDATE user SET lastactivity=".time()." WHERE userid='$bbuserinfo[userid]'";
        // This update will be done in the doshutdownfunction automatically, but the old method was doing screwy things!!
      } 


Where do these additions go in relation to that?

[Christine]

Nevermind - I figured it out. Ethan - you need to update the instructions and clear up the sessions.php instructions to reflect one change instead of two.

We are testing this now - it appears to be working like a champ!

[FWC]

Originally posted by tubedogg
Don't be so sure...a database version of this may be just what I need (this cookie thing hasn't been working too well for me thus far).

The cookie version has been a little flaky for me too, Tube. I think I'll go with Ethan's version once the new vB is out and the hack is done.

[Trillian]

Weeeeeee! Can't wait to install this!

Thanks for taking the time to document it

[ethank]

I updated the file to clarify sessions.php instructions and removed the updateLocation() code.

If anyone has any code/query improvements, just holler!

I tried to merge in the other one, but I figured this one works and why do more hacking than necessary? So anyhow....

Ethan

[Christine]

Ethan - my hat is off to ya! I will be moving this into production today.

Here are some additional notes for the optional template section - changing the column header and the wording of the redirect screen.

in forumdisplay/forumdisplay_forumslist and
Forumhome/forumhome

Replace Moderator with Functions (or whatever tag you choose)

in redirection/redirect_markread

Replace with:
The forum(s) you have selected have been marked as read and the new post indicators will now be off.

[ethank]

Originally posted by Christine
Ethan - my hat is off to ya! I will be moving this into production today.

Here are some additional notes for the optional template section - changing the column header and the wording of the redirect screen.

in forumdisplay/forumdisplay_forumslist and
Forumhome/forumhome

Replace Moderator with Functions (or whatever tag you choose)

in redirection/redirect_markread

Replace with:
The forum(s) you have selected have been marked as read and the new post indicators will now be off.

Thanks. I'll add those and HTML-ize the instructions, as well as put an install script together. Anyone have the source for a good install script, like the one Kier uses?

Do tell me how it works in production and if you see any code tweaks that can be done. I'm working on a blogger hack right now, so my coding time is all difused

Ethan

[Christine]

Yes, it should be interesting. I just checked and I have just under 400 users on line right now so I will wait for it to slow down a bit and load it up. I will let you know.