Help Desk Software & Beyond
October 21, 2017, 11:16:51 PM *
Welcome, Guest. Please login or register.
To post messages you need to register. We apologize for inconvenience, but this is to prevent spam.
Registration is instant (no email verification) and we do not ask for any personal information.

Login with username, password and session length
News: Welcome to Help Desk Software forum!
 
   Home   Help Search Login Register  
Pages: [1]
  Print  
Author Topic: BUG FIX - Optional File Attachment Module  (Read 9511 times)
Sparky
Moderator
Hero Member
*****

Karma: 85
Posts: 2,275


stop pushing all those buttons


View Profile
« on: January 07, 2009, 03:13:20 PM »

I've notified Alex of a small bug I found in the optional File Attachment/Upload Module.

This bug prevents the error message from printing when you exceed the allocated file size of 4 MB.

Instead of printing the message, "file too big, not saved (max size = 4 MB)"the addfile subroutine aborts and the remainder of the ticket message is left blank.  Edit:  Ticket header is updated but Ticket message is left blank because the Ticket subroutine can't properly read it when generating the display.

The fix is simple:

In the latest version of the module (2.23) find line 109...

Quote
return '[%File%] '."$fno: $fname ".'[%too big, not saved%] ([%max size%] = '."$mz)\n";

Replace it with this...

Quote
return "File$fno: $fname ".'[%too big, not saved%] ([%max size%] = '."$mz)\n";
« Last Edit: January 12, 2009, 11:54:07 AM by Sparky » Logged

Did you update the paths in ttxcfg.cgi after moving TTX to your new location?   Undecided
To those seeking help.... please report back when you figure it out.  Cheesy
Alex
Administrator
Hero Member
*****

Karma: 22
Posts: 606



View Profile WWW
« Reply #1 on: January 12, 2009, 11:49:13 AM »

Thanks for the info, Sparky. Actually the addfile() completes just fine, but generates wrong message header. Nevertheless, the patch is exactly as described. You may also download an updated File Module from our web site.
Logged

Follow me on: LinkedIn
Sparky
Moderator
Hero Member
*****

Karma: 85
Posts: 2,275


stop pushing all those buttons


View Profile
« Reply #2 on: January 12, 2009, 11:51:40 AM »

Actually the addfile() completes just fine, but generates wrong message header.

Yes, I discovered this later.  Correcting my posting.
Logged

Did you update the paths in ttxcfg.cgi after moving TTX to your new location?   Undecided
To those seeking help.... please report back when you figure it out.  Cheesy
Pages: [1]
  Print  
 
Jump to:  

Powered by MySQL Powered by PHP Powered by SMF 1.1 RC3 | SMF © 2001-2006, Lewis Media Valid XHTML 1.0! Valid CSS!
Page created in 0.016 seconds with 20 queries.