bugphpGroupWare - Bugs: bug #7247, webdav mkdir strange behavior in...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #7247: webdav mkdir strange behavior in some case (was corrected in SQL ).

Submitted by:  Caeies <Caeies>
Submitted on:  Wed 14 Jan 2004 01:42:56 PM UTC  
 
Item Group: 0.9.16RC2Category: API - phpGroupware API
Severity: 3 - NormalPriority: 5 - Normal
Status: FixedPrivacy: Public
Assigned to: Dave Hall <skwashd>Open/Closed: Closed
Component Version: SVNOperating System: GNU/Linux - Debian
Reproducibility: Every TimePlanned Release: None
Fixed Release: 

Wed 14 Jan 2004 05:31:54 PM UTC, comment #1:

Note the removing of '/' too in the fake_full_path which is very important !!

Caeies

Caeies <Caeies>
Project Administrator
Wed 14 Jan 2004 01:42:56 PM UTC, original submission:

(background: )Because I'm making some tests, I need sometimes to create collection (aka dir) in my repository, then go in them with phpgw, and delete them. But for deleting, this is not possible to do this in phpgw filemanager (because it's a group directory). So I need to delete it in using a webdav client (cadaver). All is nice, but if you try to remake the directory mkdir will failed because file_exists() checks related to the current directory (due to mkdir not using a relative directive in his path when asking if the file exists.). and if you where not in the good directory (ie '/'), and that you call mkdir with RELATIVE_NONE, the file_exists will failed.

in class.vfs_dav.inc.php

So I think that the correction could be something like add a relative to the file_exists() call near line2096 (in the if statement) which could be the same as the one used to create the directory.

hehe, after reading the class.vfs_sql.inc.php I see that this is what it is done. (sorry I think in the same time that making this bug report).

So to be short, please sync bugs between vfs_sql and vfs_dav, because they have a lot's of code in common :).
I will try to make some patch about this if I get the time.

regards,
Caeies.

PS: the vfs_sql use this :
if (!$this->file_exists (array(
'string' => $p->fake_full_path,
'relatives' => array ($p->mask)
))
)
and so, the webdav should be turn from :
if (!$this->file_exists (array(
'string' => $p->fake_full_path.'/'
))
)
too :
if ($this->file_exists (array(
'string' => $p->fake_full_path,
'relatives' => array ($p->mask)
))
)

(please not the correction of the previous bug by removing the "!" see bugs #7228)

Caeies <Caeies>
Project Administrator

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 3 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Wed 25 Feb 2004 09:50:39 PM UTCskwashdStatusNone=>Fixed
  Assigned toNone=>skwashd
  Open/ClosedOpen=>Closed

Back to the top


Powered by Savane 3.1-cleanup1