View All Attachments in Thread

[TWTCommish]

This hack is simple and pretty easy to install.

What's it do?
Ya' know that little paperclip icon you see next to thread links containing attachments? Well, this hack makes that icon clickable. Clicking on it will give you the normal thread view and page navigation and all that, except you'll only see the posts with attachments in them. I find this particularly useful if you have a few threads where, say, members post their pictures. It's nice to cut through a lot of the comments and get straight to the files.


What's it involve?
7 code changes in 2 files. No template work of any sort.


Got a demo?
Surely: Demo...yay (http://www.movieforums.net/showthread.php?threadid=1315&view=attachments).


The attached instructions.txt file will walk you through things. Enjoy; and let me know if you find any bugs, as I haven't tested this on any board but my own yet.

[Jawelin]

Very useful !!!
I'll use ASAP.... but FireFly, please install it here too so everybody could get rid of those long and rich-of-attachments thread in this hack forum !

Thanks a lot

[Admin]

Excellent idea Chris. :) Will install this right away...

[Admin]

Works great, good job.

[Jawelin]

Very very useful...

Just a NON-problem, better a constatation, with another hack:
when I use the 'attach' ikon and only posts with attachments are displayed, post shown are numbered (Post #X hack) in a relative way, of course... I.e. they don't refer to the original post sequence in thread.
As said, just a point out...
;)

Thanks again, to both (now!)

[TWTCommish]

Oh, do you mean with the post counter/direct link hack? Hmmm, I haven't looked at that one in any sort of significant detail...but I imagine, to keep their original numbers and such, I'd have to either give up the page navigation the way it is (so that it'd refer to the original thread including non-attachment posts), or I'd have to stick another query in there somewhere. I'm not sure.

Anyway, glad you like it, guys. :) I remember thinking of it and then wondering why I hadn't before. Hand it to my members, though, for asking for an easy way to access the attachments. :)

[Jawelin]

Of course.... I was first liked this hack ! ;)
Just a think I noticed... that's all.
Thanks

[TWTCommish]

Small update to this: the original hack doesn't turn the paperclip image on the search results page into a clickable link the way it does for the one on the forumdisplay. This is easily corrected. Find this around line 970-something in you search.php file:

$paperclip="<img src=\"{*imagesfolder*}/paperclip.gif\" alt=\"$searchresult[attach] Attachment(s)\" border=\"0\">";
...and replace it with this...
$paperclip="<a href=\"showthread.php?s=$session[sessionhash]&threadid=$searchresult[threadid]&view=attachments\"><img src=\"{*imagesfolder*}/paperclip.gif\" width=\"10\" height=\"13\" alt=\"$searchresult[attach] Attachment(s)\" border=\"0\" align=\"absmiddle\"></a>";
That should do it. :) Take the asterisks out, however, from the {*imagesfolder*} part -- I had to add them in there, or this forum would process them as vB.org's image folder location.

[skyline2002]

how can make this hack work with v2.01.

i couldnt find the code below (in showtheard.php):
-------------------------------------------------
$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage");
----------------------------------------------------

[Admin]

If I were I would forget about this hack and upgrade to 2.2.5 ASAP. I don't know how to say this so you will listen, this is very important and 2.0.1 is highly exploitable.

[sybaxi]

I can't seem to install this. Does it works with 2.2.5 ? Cos i can't seem to find that line to replace with. This is installed in this forums though, and it's running 2.2.5. And oh, i tried installing leaving the last bit out. Everything is fine but when i clicked on the paperclip it displays the whole thread itself, not just the attachments.
Please help. Thanks alot.


$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage");

Replace it with this:


if ($view == 'attachments') {
$attachment_querystring = '&view=attachments';
} else {
$attachment_querystring = '';
}

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage$attachment_querystring");

Save and upload.

[Tarion]

I had the same Problem only when i have installed FireFlys Number of Hacks Hack, without this hack it does work but with it the hole thread is displayed not only the messages with attachments, pleas help

[TWTCommish]

Well, look at FireFly's hack and see if it changes that line. If it does, follow the modified instructions accordingly. :)

[Tarion]

i do follow FireFlys instructions, but your $attachment_clause has no effect.

This are FireFlys line:

$getpostids=$DB_site->query("
SELECT post.postid,post.userid AS postuserid FROM post
WHERE post.threadid='$threadid' AND post.visible=1
ORDER BY dateline $postorder LIMIT ".($limitlower-1).",$perpage
");


and this is my line:

$getpostids=$DB_site->query("
SELECT post.postid,post.userid AS postuserid FROM post
WHERE post.threadid='$threadid' AND post.visible=1$attachment_clause
ORDER BY dateline $postorder LIMIT ".($limitlower-1).",$perpage
");

[TWTCommish]

You're talking about different things. First, you were talking about another block of code, and now this one. Which is the problem? I don't see what it is you're saying.

[Tarion]

sorry TWT for my bad explaining.

Wath I mean is, wenn i install Fireflys Hack the above namde blocks are effekted by changes of both Hacks, but wenn i do Fireflys instructions and then yours (and the $attachment_clouse behinde the post.visibile) it doesn't work it alway shows up the complet Thread nut only the attachment messages, i tried to do it by my self but it doesn't work.

Sorry also for my bad english i don't often speak or write in english (i'am german :) )

I hope you know an answer

[TWTCommish]

Can you give me the link to FireFly's hack?

[Boofo]

In your instructions you said to replace this line of code:
$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage");

With this:
if ($view == 'attachments') {
$attachment_querystring = '&view=attachments';
} else {
$attachment_querystring = '';
}

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage$attachment_querystring");


The code I found was:
$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display".iif(isset($highlight), "&highlight=$highlight", ""));


Here's what I did, will it work ok?
$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display".iif(isset($highlight), "&highlight=$highlight", "$attachment_querystring"));

[Tarion]

here ist the Link to Firefly's hack: https://vborg.vbsupport.ru/showthread.php?s=&threadid=33811

THX for help

[TWTCommish]

No, I don't think so. I'm not sure, but I think this ought to work, though:

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display$attachment_querystring".iif(isset($highlight), "&highlight=$highlight"));

[Tarion]

Originally posted by Boofo
In your instructions you said to replace this line of code:
$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage");

With this:
if ($view == 'attachments') {
$attachment_querystring = '&view=attachments';
} else {
$attachment_querystring = '';
}

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage$attachment_querystring");


The code I found was:
$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display".iif(isset($highlight), "&highlight=$highlight", ""));


Here's what I did, will it work ok?
$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display".iif(isset($highlight), "&highlight=$highlight", "$attachment_querystring"));


no

put it after this

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&$attachment_querystring&display=$display".iif(isset($highlight), "&highlight=$highlight"));

[Boofo]

Here's what I have for that piece of code right now.
if ($view == 'attachments') {
$attachment_querystring = '&view=attachments';
} else {
$attachment_querystring = '';
}

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display".iif(isset($highlight), "&highlight=$highlight", "$attachment_querystring"));


What should it actually be? :)

[TWTCommish]

It should be this, I think:

if ($view == 'attachments') {
$attachment_querystring = '&view=attachments';
} else {
$attachment_querystring = '';
}

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display$attachment_querystring".iif(isset($highlight), "&highlight=$highlight", ""));

[Boofo]

Tarion? You agree? I just want to make sure on this. If you don't agree, can you tell me why, please?

[Boofo]

TWTCommish ,

How did you put the attachment box in the message and what is the hack for who downloaded it? And, lastly, how did you put the link in the box under the attachment? I'd like to try it, if I can. :)

[Martin64]

Excellent job. Works like a charm. :D

[Smoothie]

Yes, installed. Marvelous!

[Boofo]

This must have been the right way to do it because it works great! :) thanks for all of your help and an excellent hack! :)

Originally posted by TWTCommish
It should be this, I think:

if ($view == 'attachments') {
$attachment_querystring = '&view=attachments';
} else {
$attachment_querystring = '';
}

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage&display=$display$attachment_querystring".iif(isset($highlight), "&highlight=$highlight", ""));