diff options
author | Aleksander Machniak <alec@alec.pl> | 2014-08-05 14:08:17 +0200 |
---|---|---|
committer | Aleksander Machniak <alec@alec.pl> | 2014-08-05 14:09:25 +0200 |
commit | f5f056e7f213666d9e9556092795b07df8e8354d (patch) | |
tree | e0a7edc8638b9907c95ee87e137651beba57765d /plugins/managesieve/lib | |
parent | 8d4b4897df1861c12b033b864f4d7d8fca7e540a (diff) |
Fix default vacation status (#1490019) - improve error message
Conflicts:
plugins/managesieve/Changelog
Diffstat (limited to 'plugins/managesieve/lib')
-rw-r--r-- | plugins/managesieve/lib/Roundcube/rcube_sieve_vacation.php | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/plugins/managesieve/lib/Roundcube/rcube_sieve_vacation.php b/plugins/managesieve/lib/Roundcube/rcube_sieve_vacation.php index aa65966c4..405a8a02c 100644 --- a/plugins/managesieve/lib/Roundcube/rcube_sieve_vacation.php +++ b/plugins/managesieve/lib/Roundcube/rcube_sieve_vacation.php @@ -107,8 +107,9 @@ class rcube_sieve_vacation extends rcube_sieve_engine } if ($vacation_action['reason'] == '') { - $error = 'managesieve.cannotbeempty'; + $error = 'managesieve.emptyvacationbody'; } + if ($vacation_action[$interval_type] && !preg_match('/^[0-9]+$/', $vacation_action[$interval_type])) { $error = 'managesieve.forbiddenchars'; } @@ -276,7 +277,7 @@ class rcube_sieve_vacation extends rcube_sieve_engine } $table->add('title', html::label('vacation_status', $this->plugin->gettext('vacation.status'))); - $table->add(null, $status->show($this->vacation['disabled'] ? 'off' : 'on')); + $table->add(null, $status->show(!isset($this->vacation['disabled']) || $this->vacation['disabled'] ? 'off' : 'on')); $out .= html::tag('fieldset', $class, html::tag('legend', null, $this->plugin->gettext('vacation.reply')) . $table->show($attrib)); |