Attachment Descriptions

/*================================================= =====================*\
|| Attachment Descriptions for vBulletin 3.0.7
|| Author : Psionic Vision
\*================================================ ======================*/

What is this:
It enchances attachment upload interface with a new 'attachment description' field

Features:

• Conveniently enchances attachment upload screen with the description field (see screenshots)
• Displays attachment descriptions in:
  Attachment Manager, Post Edit (newthread, newreply, editpost) and Attachment List (in posts)

[Lionel]

Originally Posted by JimpsEd

Incompatiable with the Geek Article System 3.5.0 I think, which makes use of the column "description" in the table attatchments.

negative. I am using it there.

[j_86]

Both running together well?

[Lionel]

Originally Posted by JimpsEd

Both running together well?

Geek placed it in template, but never used that column. He probably provisionned for later. I am going to let him know.

[akanevsky]

They even had the unused description field in attachment table. When I questioned them about it, no straight answer could be given.

Are you sure about this? I didn't have that field prior to creating the hack.

Let me bother you with one question, if you are familiar with vbadvanced news.php module. How to display it in front page too?

1. Add it into the template - sorry don't have it installed, cannot help you with this.
2. In news.php, find:

iif ($vba_options['portal_news_showattachments'], ', attachment.filename, attachment.filesize, attachment.visible,

replace with:

iif ($vba_options['portal_news_showattachments'], ', attachment.filename, attachment.filesize, attachment.visible, attachment.description,

3. Done.

Ability to have multiple fields?

I assume it would be a good exercise for your finger muscles to manually type whatever you need, as it is not a lot at all.

Also, if there was one thing the attachment system needed, it's the ability to use a URL

This does not have anything to do with the current hack, however, I will look into it.

Geek placed it in template, but never used that column. He probably provisionned for later. I am going to let him know.

I appreciate that

Thanks everyone for positive comments

[j_86]

I have it installed, but getting this script error after uploading something:

Line: 337
Char: 273
Unterminated string constant

Line 337 is:

	attachlist.innerHTML = "<div style=\"margin:2px\"><img class=\"inlineimg\" src=\"images/attach/rtf.gif\" alt=\"magnetic field questions.rtf\" border=\"0\" /> <a href=\"attachment.php?attachmentid=5&amp;stc=1\" target=\"_blank\">magnetic field questions.rtf</a> (758.2 KB)

Any ideas?

[j_86]

Originally Posted by Lionel

Geek placed it in template, but never used that column. He probably provisionned for later. I am going to let him know.

The coloumn 'description' already exists on my install, though. I have also seen it as part of the table-installer in GASinstall I think.

[akanevsky]

Make sure you did everything right. If you get outside the DIV, you get that error...

[j_86]

Ah, I always put a comment in templates when I use a hack that modifies them, such as;

<!-- attatch descript hack edit above -->

Don't know what template did not like this, but it was one. Removed all comments and working fine

[Lionel]

Originally Posted by JimpsEd

The coloumn 'description' already exists on my install, though. I have also seen it as part of the table-installer in GASinstall I think.

yes you are right. He added that column, but never used it. I simply altered it from varchar to TINYTXT.

I told him about it so he should not be bothered working on it.

[The Geek]

The attachemnt description field in gas was provisioned for future release. If you have installed GAS 3.x than you will already have an description field in your attachment table. I hadnt yet put the UI in place for it (and now I dont! ). I pushed it into the first 3.0 release so I could import v3articles and keep their descrptions.

This hack should be totally compatible. You may get an error doing an alter table as the column already exists - but everything else should slot in nicely.

Good job - it is a seriously missing element of vB