Hi,
You are correct whitespace is trimmed off the folder name when the rule is saved. If you wanted to stop it doing that then search for the line $folder = $this->_strip_val($folders[$idx]);
. I guess removing it wouldn't make that much difference since the values are all pre filled any way.
Thank you, changing that line to
$folder = $folders[$idx];
fixes the bug in my testing and so far I'm not finding any negative side effects. Think about it this may help me with other future cases as well. I know some other users at my company tend to put spaces at the start of folder names to manually force them to be indented in their folder list so I suspect such a folder name would trigger the same bug if selected as a folder to filter into. (if _strip_val removes whitespace at both ends of the string like I assume?)
Hi,
The advanced options box in the out of office message is unticked by default, its only ticked if the plugin is configued to show them by default or of your rule uses them.
Ahh I never noticed that before, I've got a cyrus server so I've always had
// Force the :addresses line to always be added to new vacation rules
// Some sieve setups require that the :address part of a vacation rule is always present for the message to be sent
// Cyrus setups need this to option set to true
$rcmail_config['sieverules_force_vacto'] = TRUE;
which now I trace though the code I see changes how advanced options are shown. I happen to also have
$rcmail_config['sieverules_limit_vacto'] = TRUE;
though so the address option isn't much use as the users only have one option and we want it shown so now I see how the code controls it I'll write a little patch for myself so those checks always end up with the advanced options set off.
Thanks for the help.
-J