Skip to content
Snippets Groups Projects
Commit ef98afc6 authored by Peter Rotich's avatar Peter Rotich
Browse files

Ticket Source

Unify ticket source and preserve original (e.g Wed) source on ticket edit.
parent a1bae6ec
No related branches found
No related tags found
No related merge requests found
......@@ -145,6 +145,11 @@ class TicketModel extends VerySimpleModel {
/* @trans */ 'Phone',
'Email' =>
/* @trans */ 'Email',
'Web' =>
/* @trans */ 'Web',
'API' =>
/* @trans */ 'API',
'Other' =>
/* @trans */ 'Other',
);
......@@ -2746,6 +2751,11 @@ implements RestrictedAccess, Threadable {
$errors['duedate']=__('Due date must be in the future');
}
if (isset($vars['source']) // Check ticket source if provided
&& !array_key_exists($vars['source'], Ticket::getSources()))
$errors['source'] = sprintf( __('Invalid source given - %s'),
Format::htmlchars($vars['source']));
// Validate dynamic meta-data
$forms = DynamicFormEntry::forTicket($this->getId());
foreach ($forms as $form) {
......
......@@ -70,12 +70,17 @@ if ($_POST)
</td>
<td>
<select name="source">
<option value="" selected >&mdash; <?php echo __('Select Source');?> &mdash;</option>
<option value="Phone" <?php echo ($info['source']=='Phone')?'selected="selected"':''; ?>><?php echo __('Phone');?></option>
<option value="Email" <?php echo ($info['source']=='Email')?'selected="selected"':''; ?>><?php echo __('Email');?></option>
<option value="Web" <?php echo ($info['source']=='Web')?'selected="selected"':''; ?>><?php echo __('Web');?></option>
<option value="API" <?php echo ($info['source']=='API')?'selected="selected"':''; ?>><?php echo __('API');?></option>
<option value="Other" <?php echo ($info['source']=='Other')?'selected="selected"':''; ?>><?php echo __('Other');?></option>
<option value="" selected >&mdash; <?php
echo __('Select Source');?> &mdash;</option>
<?php
$source = $info['source'] ?: 'Phone';
foreach (Ticket::getSources() as $k => $v) {
echo sprintf('<option value="%s" %s>%s</option>',
$k,
($source == $k ) ? 'selected="selected"' : '',
$v);
}
?>
</select>
&nbsp;<font class="error"><b>*</b>&nbsp;<?php echo $errors['source']; ?></font>
</td>
......
......@@ -131,7 +131,9 @@ if ($_POST)
<select name="source">
<?php
$source = $info['source'] ?: 'Phone';
foreach (Ticket::getSources() as $k => $v)
$sources = Ticket::getSources();
unset($sources['Web'], $sources['API']);
foreach ($sources as $k => $v)
echo sprintf('<option value="%s" %s>%s</option>',
$k,
($source == $k ) ? 'selected="selected"' : '',
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment