6.0.0-alpha12
6/12/25

[#10494] Saving Maildrop Script with empty and disabeld vacation message fails
Summary Saving Maildrop Script with empty and disabeld vacation message fails
Queue Ingo
Queue Version 2.0.5
Type Bug
State Resolved
Priority 2. Medium
Owners jan (at) horde (dot) org
Requester michael-dev (at) fami-braun (dot) de
Created 09/08/2011 (5026 days ago)
Due
Updated 09/14/2011 (5020 days ago)
Assigned
Resolved 09/14/2011 (5020 days ago)
Github Issue Link
Github Pull Request
Milestone 2.0.6
Patch Yes

History
09/14/2011 09:03:55 AM Jan Schneider Assigned to Jan Schneider
State ⇒ Resolved
Milestone ⇒ 2.0.6
 
09/14/2011 09:03:38 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git for this ticket:

[jan] Catch if files don't exist while deleting with VFS transport 
(michael-dev@fami-braun.de, Bug #10494).

  3 files changed, 8 insertions(+), 4 deletions(-)
http://git.horde.org/horde-git/-/commit/2c1ac7e617cc2a53157674dbacc13a0828389291
09/08/2011 03:48:14 PM michael-dev (at) fami-braun (dot) de New Attachment: Vfs.diff Download
 
09/08/2011 03:47:35 PM michael-dev (at) fami-braun (dot) de Comment #1
Priority ⇒ 2. Medium
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Ingo
Summary ⇒ Saving Maildrop Script with empty and disabeld vacation message fails
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
I'm using INGO with  Maildrop and VFS/SSH2, no vacation message and 
vacation disabled. When trying to activate the maildrop script, this 
fails as vacation.msg cannot be deleted, as it does not exist.
Though one could argue that vacation.msg should not be listed as extra 
files in the Maildrop backend if vacations are disabled, I feel like 
the real mistake here is the vfs transport class trying to delete a 
non-existend file without catching this case.

I therefore propose to patch ingo/lib/Transport/Vfs.php with the 
attched patch.

Saved Queries