Here is a patch for VacationConfig.class.php which I think addresses the just mentioned bug:
*** plugins/vacation/lib/VacationConfig.class.php~ Tue Mar 9 15:13:56 2010
--- plugins/vacation/lib/VacationConfig.class.php Fri Jun 11 17:23:03 2010
***************
*** 168,180 ****
{
$keys = $this->allowedOptions[$this->currentArr['driver']];
! $diff = array_diff_key($this->currentArr,array_flip(array_keys($keys)));
! if (! empty($diff) && !in_array(key($diff),array('driver','body','subject')))
{
// Invalid options found
! $this->hasError = sprintf("Invalid option found in config.ini for %s driver and section [%s]: %s is not supported",
$this->currentArr['driver'],$this->currentHost,key($diff));
}
return (empty($diff));
--- 168,184 ----
{
$keys = $this->allowedOptions[$this->currentArr['driver']];
! $diff = array_diff_key($this->currentArr,
! array_flip(array_keys($keys)),
! array('driver' => 1,
! 'body' => 1,
! 'subject' => 1));
! if (! empty($diff))
{
// Invalid options found
! $this->hasError = sprintf("Invalid option found in config.ini for %s driver and section [%s]: '%s' is not supported",
This also makes it apparent that the values provided in the example config.ini are bogus: defaultbody and defaultsubject don't appear to be valid parameters.