6.0.0-beta1
7/5/25

[#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 (5027 days ago)
Due
Updated 09/30/2011 (5027 days ago)
Assigned
Resolved 09/30/2011 (5027 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
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

Saved Queries