Hereby confirmed that everything works as expected :)

Fixed.
That's what I did. It doesn't make sense to use different keys for 
these values.

Hi,



You broke 2 things:



You renamed function substVars to _substVars, but the calls to this 
function are still '$this->substVars()'. Also, you forgot to fill 
$this->_params['username'] and ['password'] with the values of 
['vfs_user'] and ['vfs_pass'], since those variables are used by the 
VFS driver.



Perhaps it would be cleaner to not use 'vfs_user' & 'vfs_pass', but to 
use 'username' and 'password'. But that's up to you :)



I've added a fix for the 2 problems above.

Cleaned up and committed. I didn't test it, so please try if my 
changes still work.

W00ps,



I messed up the previous patch. Here 's the final one :)

The vfs_user and vfs_pass were already 'optional', but I've changed it 
as you requested :)



Also, you were right about the broken writeData().

I missed the line when merging the patch with the 'clean' source :)



I've adjusted this in the new patch, making all the variables optional.



Hopefully you will be satisfied with this one :)

Please change the patch to make the vfs_user and vfs_pass optional, 
i.e. commented out in backends.php.dist by default, and checked with 
empty() in vfs.php.

And you break existing system with writeData() change in vfs.php. I 
see where you're heading at, but this should solved more elegantly, 
and most important, backward compatible.

I've made a small patch against ingo-HEAD-2005-07-22, that allows an 
override of the username & password used for VFS.



It still defaults to '$username' and '$password' as it currently does.



I needed this because all my mail-accounts are virtual and under the 
same uid+gid.