OK, so I have had some instances where a client submits a ticket with a misspelled email address and never gets the email with the ticket ID. They then submit another ticket saying the ticket system is broken.

To fix this I have created a mod to add a confirm email address field to the open.php page and have created validation rules for it.

/* Confirm Email Address Modification

* Author: James Williams

* osTicket v1.6 RC4

* Date: 06-30-08

Process for Error Checking Email Confirm Field

------- New form field -------

+ ADD Line 41 open.inc.php:

(Copy the row with the Email Address and change the names from email to confirm_email)

Confirm Email Address:

<?if ($thisclient && ($confirm_email=$thisclient->getEmail())) {

"><?=$confirm_email?>

<?}else {?>

<?}?>

* <?=$errors?>

------- New array_element -------

+ ADD Line 30 class.ticket.php

var $confirm_email;

+ ADD Line 820 class.ticket.php:

(This array defines which validation type rule to use when validating, if it's a required field, and what the error message should be. Just copy the email line and change email to confirm_email and edit the error message.)

$fields = array('type'=>'confirm_email', 'required'=>1, 'error'=>'Email addresses must match');

------- Add Validation Rules -------

+ ADD Line 23 class.validator.php

(This var will store the string that was entered in the Email field so that on the next loop it will be compared with the confirm_email field)

var $chk_same;

+ ADD Line 94 class.validator.php

(Copy and comment out the original case'email': All I have added is setting the $chk_same to the value of the email field. Then I created a new case statement for confirm_email that actually does the check and errors.)

case 'email':

if(!$this->is_email($this->input)){ // IF NOT email format then...

$this->errors=$field;

}

$chk_same = $this->input;

break;

case 'confirm_email':

if(strtolower($chk_same) !== strtolower($this->input)){ //Confirm EMAIL CODE & ERROR

$this->errors=$field;

}

break;

+ADD Line 29 include\staff\newticket.inc.php

(Staff Side error fix)

Confirm Email Address:

* <?=$errors?>

There you go, you are all done! This does not do anything with the data base, merely passes along a new field for validation. If the emails match, the Email field is the only field that is recorded in the DB.

;)