Tickets :: [#10584] Horde/Vfs/File.php _connect() doesn't work for existing directories

Summary	Horde/Vfs/File.php _connect() doesn't work for existing directories
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Resolved
Priority	2. Medium
Owners	slusarz (at) horde (dot) org
Requester	michael (at) bigmichi1 (dot) de
Created	09/30/2011 (5149 days ago)
Due
Updated	09/30/2011 (5149 days ago)
Assigned
Resolved	09/30/2011 (5149 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

09/30/2011 06:33:50 PM	Michael Slusarz	Assigned to Michael Slusarz State ⇒ Resolved

09/30/2011 06:33:02 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #10584~~: Fix using File driver if vfsroot directory already exists 3 files changed, 62 insertions(+), 6 deletions(-) http://git.horde.org/horde-git/-/commit/fe54b5b9583e6f1121950fad73697ca423a34a8f

09/30/2011 05:22:15 PM	michael (at) bigmichi1 (dot) de	Comment #1 Priority ⇒ 2. Medium Patch ⇒ No Milestone ⇒ Queue ⇒ Horde Framework Packages Summary ⇒ Horde/Vfs/File.php _connect() doesn't work for existing directories Type ⇒ Bug State ⇒ Unconfirmed	Reply to this comment
if a directory exist and is readable a excption is thrown were it shouldn't be let's have a look: /tmp directory exist and is readable for world if (!(@is_dir($this->_params['vfsroot']) && is_readable($this->_params['vfsroot'])) \|\| !@mkdir($this->_params['vfsroot'])) { then we got if (!(true && true) \|\| !false) { so we got a resulting true and the exception is thrown and thats not right if dir exist