Skip to content
Snippets Groups Projects
Commit 81076638 authored by Jared Hancock's avatar Jared Hancock
Browse files

Forbid empty reply separators

If it only contains whitespace, it is also invalid
parent a3dd33d7
Branches
Tags
No related merge requests found
......@@ -883,7 +883,7 @@ class OsticketConfig extends Config {
$f['alert_email_id']=array('type'=>'int', 'required'=>1, 'error'=>'Selection required');
$f['admin_email']=array('type'=>'email', 'required'=>1, 'error'=>'System admin email required');
if($vars['strip_quoted_reply'] && !$vars['reply_separator'])
if($vars['strip_quoted_reply'] && !trim($vars['reply_separator']))
$errors['reply_separator']='Reply separator required to strip quoted reply.';
if($vars['admin_email'] && Email::getIdByEmail($vars['admin_email'])) //Make sure admin email is not also a system email.
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment