Quick Reply Box (with Close/Open thread switch)

This hack will let your users post a Quick Reply, instead of going through the New Reply steps.
If you have permissions, it will show you a Close/Open Thread check box beside the Quick Reply. Also, the check box will show in New Thread, New Reply, New Post and Edit Post.

ESTIMATED INSTALL TIME: 10 minutes
REQUESTED BY: hellsatan
TESTED IN VERSION: 2.2.7

CREDITS: The hack includes one small code segment of FireFly's Quick Reply Box related to user permissions.
(9 lines of code)

[high]RELATED HACKS[/high]
Quick Delete Post
Quick Close/Open Thread (with reason)

You like this hack? Please click on the [high]Install[/high] button.
To download the latest version of this hack, click below:

[TECK]

You can probably imagine that I want this confirmed before I go digging deep into the code again.

you will have to do it. you made a mistake. the hack works perfectly for everyone... well at least for 100 of them.

[crazza]

After checking the code four times, installing the hack on a brand spanking new installation of VBulletin 2.2.6 I discovered that I did nothing wrong in the first place.

Route 1.

When entering a thread by clicking the thread's title (e.g. ..../forums/showthread.php?s=&threadid=6055) the thread is only called by threadid.

After entering something in the QuickReply box AND clicking either close or open.. the exact same link will be called as stated above, thus showing you an 'old copy' of the page, without your new post.

Route 2.

When entering a thread by clicking the 'last post' button for this thread (e.g. ..../forums/showthread.php?s=&goto=lastpost&threadid=6055) a 'forced query' is run to determine what the actual lastpost is.

After entering something in the QRB and doing an open or close.. the exact same link will be called.. the same query will be run.. and voila.. there's your post.

Conclusion

Reloading the same page without running the extra query doesn't update correctly when closing or opening the thread. This can be fixed by forcing the reload to run the query (i.e. adding a 'lastpost' statement).

Code change

Only the php code for newreply.php needs to be altered in two places [around lines 308 and 450]

Old code:

$threadinfo[notes]="Thread $action by $bbuserinfo[username] on ".vbdate($dateformat." ".$timeformat,time()).". $threadinfo[notes]";
$DB_site->query("UPDATE thread SET open=$threadinfo[open],notes='".addslashes($threadinfo[notes])."' WHERE threadid='$threadid'");

eval("standardredirect(\"".gettemplate("redirect_o penclose")."\",\"showthread.php?s=$session[sessionhash]&threadid=$threadid\");");

} else {
eval("standardredirect(\"".gettemplate("redirect_p ostthanks")."\",\"$goto\");");
}

New Code:

$threadinfo[notes]="Thread $action by $bbuserinfo[username] on ".vbdate($dateformat." ".$timeformat,time()).". $threadinfo[notes]";
$DB_site->query("UPDATE thread SET open=$threadinfo[open],notes='".addslashes($threadinfo[notes])."' WHERE threadid='$threadid'");

$goto="showthread.php?s=&goto=lastpost&threadid=$t hreadid";
eval("standardredirect(\"".gettemplate("redirect_o penclose")."\",\"$goto\");");

} else {
eval("standardredirect(\"".gettemplate("redirect_p ostthanks")."\",\"$goto\");");
}

[aLuCaRD]

Any ideas why my avatars arent showing in the thread now..? There still in the PMs though...

[aLuCaRD]

My bad, was a hack I done before but didnt realise the avatars missing, lmao. Great hack mate, I perfer the QR box without smilies, less load on the server.

**EDIT** I mean without not with

[TECK]

crazza, do you have it showing:

eval("standardredirect(\"".gettemplate("redirect_openclose")."\",\"showthread.php?s=$session[sessionhash]&threadid=$threadid\");");

or:

eval("standardredirect(\"".gettemplate("redirect_openclose")."\",\"showthread.php?s=$session
[sessionhash]&threadid=$threadid\");");

??

it must be in one line. if separated, that's why is not redirecting you properly.
in the hack, it is listed in one line.

[crazza]

Tech,

Sorry for the inconvenient wrapping in my earlier post.. but yes, the statement was on one line (it wouldn't even end up in the right thread if it wasn't).

But the "fix" I put in works a charm.. so I'm a happy camper after all..

[TECK]

np, the important thing is that it's working great for you now. congrats.

[crazza]

And as a bonus I am actually starting to understand all this mumbly jumbly lame excuse for a programming language vbportal vbulletin myseekwall thingamajig

It makes me happy I might add..

[TECK]

good to hear this.. and welcome to our community.

[yzztik]

I notice that when you reply using the quick reply box, it does not register that you have posted to subscribe you to the thread if you have that as a default ... is there anything I can add to the code to make it do that? even if I have to add another checkbox below it to do it that would be great!

Any ideas anyone?