[REQ] File Attachment Indicator

[arvin]

Just wondering ....

is there a way....to indicate that a customer has attached a file in the ticket browser page .... I guess have its own column and paper clip gif or something .... I sure there is a way to do this .... but how much time and code is required .....

also ... can I restict the file types accepted? to only *.doc,*.xls, *.pdf


Thank you in advance

[Sparky]

This is an excellent idea and should be very do-able.

Roughly:

You'd need the Layout Designer module.

From within Setup, specify a new field for the paperclip and also make it a column in the ticket list.

Then create a hidden field for the paperclip in the newticket.html template.

Then creatively using some Javascript, upon form submission, look to see if there is a document attached and then set the paperclip value to one of two things.... full HTML of a blank image or a paperclip.

I foresee plenty of kinks, otherwise...

Easy.   


"... can I restict the file types accepted? to only *.doc,*.xls, *.pdf"

Normally, that's something you should be able to do with any file attachment program.  I don't have the module so I can't say how or where you'd specify that.

[Sparky]

You could probably do some Javascript "form validation".

Basically, it interrupts the submission of the form to check the form entries.  If the entries pass validation, then the form submits normally.  Otherwise an error message.

Since TTX does form validation (required fields) in the PERL, you don't need that part.

But you could modify a Javascript to validate just the file attachment field.  If you see a file extension that doesn't meet your criteria, then fail the validation with an error message.

[arvin]

Awesome ....

This is a project for tomorrow 

I will keep you posted ....


Thank you for you suggestions ...they come in very handy

[arvin]

Hello,

How or where can I specify the file types allowed to be uploaded via the File Attachment MOD.

I want only .doc, .xls, .pdf allowed

sorry if this question has already been asked ..and Thank you in adv

Arv

[Sparky]

It was asked recently.

I don't think you can without modification to the module.

However I mentioned a Javascript solution involving a form validation routine.

TTX already does form validation.  It looks at which fields are required and prevents submission while giving you an error.  This is done on the CGI/server level.

My suggestion was to find and install a free Javascript form validation script.  This would prevent submission of the form and come into play before anything else.  Instead of having it look at blank fields, you'd have it validate your file types.

[Sparky]

It was you!   

http://forum.unitedwebcoders.com/index.php/topic,618.0.html


Topics Merged

[arvin]

LMAO ....OOPS !

Thank you Sparky!

[Sparky]

Not sure how good you are at Perl but this is a start.

I've successfully filtered the file upload to only PDF type.  When you select any other type, the upload is just ignored and nothing is uploaded.  It's very crude (case-sensitive and no error messages) but it's a simple start and we can spruce it up later.

For existing tickets...

In TTXTicket.pm in the ticket subroutine...

The red is new...

my $attachment;
    if (TTXData::get('ISPRO')) {
      for (my $i = 1; $i < 4; ++$i) {
        next if $query->param("f$i") eq undef;
        if ($query->param("fname$i") =~ /.pdf/) {
        $attachment .= TTXFile::addfile($query, "f$i", "fname$i", $t->{id});
        }
      }
    }

The newticket subroutine would have similar code...

if (TTXData::get('ISPRO')) {
    for (my $i = 1; $i < 4; ++$i) {
      next if $query->param("f$i") eq undef;
      if ($query->param("fname$i") =~ /.pdf/) {
      $messages[0] .= TTXFile::addfile($query, "f$i", "fname$i", $ticket->{id});
      }
    }
  }

[arvin]

Good Morning Sparky!

Still learning perl ...but i understand enough of it to get by LOL ....

okay ... so ..is it safe to assume that......if I take this

 if ($query->param("fname$i") =~ /.pdf/) {

and change it to

 if ($query->param("fname$i") =~ /.pdf/.doc/.xls) {

I should be safe .....

Thank you in advance
 

[Sparky]

if ($query->param("fname$i") =~ /.pdf|.PDF|.doc|.DOC|.xls|.XLS/) {

I think there's a way with Perl to ignore the case-sensitivity but I don't have the time today to clean that up.

You could try this...

if ($query->param("fname$i") =~ /.pdf|.doc|.xls/i) {

[Sparky]

What happened with this?   

[arvin]

LOL .... Sorry Sparky ...

I was on another vacation .... felt so good !

I'm back now ...and will start working on this

[ihiebert]

I'm looking for the same thing, restrict attachments to .doc, .xls &.pdf's

Did anyone come up with a working mod ?

[Sparky]

I think it's working as-is.  Try it.