- Oct 31, 2013
-
-
Jared Hancock authored
PHP can't discern the difference between d/m/Y and m/d/Y when just the date is submitted to strtotime(). Unfortunately, strptime() is not available until PHP 5.1.0. This patch forces datepickers to change their values to YYYY-MM-DD upon submission to disambiguate parsing issues. Fixes #832
-
- Oct 30, 2013
-
-
Jared Hancock authored
Previously the American date format (m/d/Y) was used exclusively for formatting dates in the datepicker widget. This patch enables the widget to be formatted based on the format set in the admin panel. Fixes #829
-
- Oct 23, 2013
-
-
Peter Rotich authored
-
Peter Rotich authored
-
Jared Hancock authored
* %{ticket.thread.original} represents the original message submitted when creating the ticket * %{ticket.thread.lastmessage} represents the most recent message submitted by the end user
-
- Oct 22, 2013
-
-
Jared Hancock authored
The XKCD index page was accidentally added when testing a new feature for osTicket-1.8
-
- Oct 17, 2013
-
-
Peter Rotich authored
-
Jared Hancock authored
-
Jared Hancock authored
Validate port during installation Add help information about the non-standard port number to the popup tips in the install screen. Support MySQL 'localhost' connections using a non-standard port number. Ordinarily, MySQL will ignore the port setting on *nix systems if the host is specified as 'localhost' Implements/Fixes #769
-
Jared Hancock authored
This patch makes openssl the preferred source of random data on Windows and will only use mcrypt_create_iv on PHP 5.3.7 and newer. Fixes #771
-
Jared Hancock authored
Fixes #772
-
Jared Hancock authored
Previous explain included a nested sub-select, which exponentially increased the count of the rows to be examined. This patch eliminates one layer of nesting on the sub-select and dramatically increases the performance finding orphaned files. Fixes #773 References: http://www.osticket.com/forums/forum/osticket-1-7-latest-release/troubleshooting-and-problems-aa/9446-version-1-7-much-slower-than-1-6
-
- Oct 11, 2013
-
-
Jared Hancock authored
Some email clients (names omitted to protect the innocent) advertise the encoding as us-ascii when iso-8859-1 was really implied. This patch allows the two charsets to be interchangable.
-
- Oct 09, 2013
-
-
Jared Hancock authored
If a ticket has a due date and does not have an SLA set, the ticket would never transition to overdue. This patch addresses an SQL flaw preventing the ticket from being transitioned. Fixes #757
-
- Oct 07, 2013
-
-
Jared Hancock authored
-
Jared Hancock authored
If the GD extension is not available for the PHP installation, then use a copy of the default client-site logo with the alpha channel removed.
-
Jared Hancock authored
Assume that text in the database is encoded in UTF-8 and assume that it is HTML text and the entities should be decoded prior to display in the PDF. Fixes #756
-
- Oct 06, 2013
-
-
Jared Hancock authored
When scanning the file_chunk table for orphaned file chunks that can be deleted, apparently, MySQL will read (at least part of) the blob data from the disk. For databases with lots of large attachments, this can take considerable time. Considering that it is triggered from the autocron and will run everytime the cron is run, the database will spend considerable time scanning for rows to be cleaned. This patch changes the orphan cleanup into two phases. The first will search just for the pk's of file chunks to be deleted. If any are found, then the chunks are deleted by the file_id and chunk_id, which is the primary key of the table. The SELECT query seems to run at least 20 times faster than the delete statement, and DELETEing against the primary key of the blob table should be the fastest possible operation. Somehow, both queries required a full table scan; however, because the SELECT statement is explictly only interested in two fields, it is more clear to the query optimizer that the blob data should not be scanned. References: http://stackoverflow.com/q/9511476
-
- Oct 04, 2013
-
-
Jared Hancock authored
Some mail clients and mail brokers do not properly handle quoted-printable encoding, used in osTicket outgoing emails. Oddly, the end mail clients render the email with trailing equal signs (=) at the end of every line, where it was added due to QP encoding. References: http://www.osticket.com/forums/forum/osticket-1-7-latest-release/troubleshooting-and-problems-aa/10708-solved-microsoft-exchange-internet-mail-lines-end-with RFC 2045 section 6.7, http://www.ietf.org/rfc/rfc2045.txt
-
Jared Hancock authored
If the body is declared with inline disposition and no filename, the parser engine would create an incorrect attachments list.
-
- Sep 30, 2013
-
-
Peter Rotich authored
-
Jared Hancock authored
If it only contains whitespace, it is also invalid
-
- Sep 29, 2013
-
-
Peter Rotich authored
-
Peter Rotich authored
line is used.
-
Peter Rotich authored
id/number. * Ticket number will be customization in the future * Ticket thread only link to Ticket should be internal ticket id.
-
- Sep 27, 2013
-
-
Jared Hancock authored
if the References or In-Reply-To header matches in a way that continues a ticket's thread, include the attachments in the thread if allowed by the system settings.
-
Jared Hancock authored
In the event that the system is set to do-nothing for POP fetched emails, and eventually, one of the emails is rejected, ensure that the message-id is logged so that the message will not be re-fetched, even if it is undeleted.
-
Jared Hancock authored
Fixes #742
-
- Sep 26, 2013
-
-
Jared Hancock authored
On some configurations of PHP and remote mail servers, Kerberos and NTLM challenge response authentication could be possibly attempted. This is essentially futile and potentially fatal for osTicket mail fetching, as the system is currently designed for username and password authentication only. This patch disables challenge and response authentication for PHP versions 5.3.2 and newer, which support the fix. This patch also consistently encodes mailbox names according to the rfc 2060 for IMAP.
-
- Sep 24, 2013
-
-
Jared Hancock authored
Previously, filenames specified in email headers fetched using the PHP imap extension were not decoded properly. Specifically, if RFC5987 was used to encode the filenames, no decoding was performed at all. This patch properly detects the filename attribute, using a decoding scheme for RFC5987 if the attribute was encoded properly. References: http://osticket.com/forums/showthread.php?t=6129
-
- Sep 23, 2013
-
-
Jared Hancock authored
RFC2045, section 5.1 seems to indicate that arbitrary parameters can be appended to a Content-Type header specification. Some email clients seem to use the Content-Type header to specify an attachment without giving a separate Content-Disposition header normally used to signify attachments. This patch corrects attachment detection for piped emails to detect such emails. The patch also correctly decodes filenames specified either in the Content-Disposition or Content-Type headers using RFC5987, where the filename can be encoded using an arbitrary character set (ie, not us-ascii).
-
Jared Hancock authored
8e72e521 (v1.7.1.2+) introduced a bug where osTicket version 1.6 would not send a cookie (by calling PHP session_start()) for the login page. Therefore, after unpacking the 1.7.1 source code, an upgrade would not be possible, because a login would never be processed correctly.
-
- Sep 20, 2013
-
-
Jared Hancock authored
When comparing the From address of incoming email. If the ticket owner sent an email back into the system and the email address did not match exactly, case-wise, the email would not be considered from the ticket owner.
-
Jared Hancock authored
-
- Sep 18, 2013
-
-
Jared Hancock authored
-
Jared Hancock authored
Also converts ROOT_DIR detection to always use forward slashes. And it fixes the removal of the leading double-backslash on Windows UNC names (fixes #649)
-
- Sep 13, 2013
-
-
Jared Hancock authored
This is safe now, because the title is appropriately encoded in class.thread.php/ThreadEntry::create() Fixes #567, #718
-
Jared Hancock authored
if the domain given in HTTP_HOST variable happens to have a port specification. Technically, the port specification should not be included in the domain spec given in the cookie. (And for the record, that makes no sense to me, seeing as a cookie would otherwise be valid for all servers on any ports at a particular domain).
-
- Sep 12, 2013
-
-
Jared Hancock authored
The previous implementation did not work correctly for symlinked folders. The new approach uses debug_backtrace() and ROOT_DIR to determine the difference between ROOT_DIR and the osTicket installation path. This thing is like a turd that won't flush
-
- Sep 11, 2013
-
-
Jared Hancock authored
-