patchphpGroupWare - Patches: patch #3399, bugs corrections for vfs_dav :...


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

patch #3399: bugs corrections for vfs_dav : Improved ls function as well as file_type : performance improvement as well as behavior

Submitted by:  Caeies <Caeies>
Submitted on:  Wed 29 Sep 2004 07:58:58 AM UTC  
Category: API - phpGWapiPriority: 5 - Normal
Item Group: NoneStatus: None
Privacy: PublicAssigned to: None
Open/Closed: Open

Thu 28 Oct 2004 01:03:26 PM UTC, comment #1:

Be carefull, this ls (Perhaps the problem was present in previous dav ls, I don't know), the branch 3 of ls don't return the same thing that the branch 3 sql ls ...
In fact we only get one level of directory, not the whole tree as in sql ls with checksubdirs...

I'm in the process of reworking the entire dav layer to solve this problem in a "good" way ie like sql, so stay tuned...


Caeies <Caeies>
Project Administrator
Wed 29 Sep 2004 07:58:58 AM UTC, original submission:

Problems :

  • In file manager, when you try to go in / there was in our case some strange behavior of filemanager. In fact, ls was not unseting the right directory in /
  • Instead of using a scope of 1 when doing a ls -ld we use a scoping of 0 to get only the properties of the directory/file we have, and not all the files/or directories in the listed directory. It seems that it was duplicated code from the 3td branche ...

Solution :

  • Use a scope of 0 in case of branch 1 in ls (to return only infos about the asked file/directory)
  • Use a scope of 0 for file_type (we only want to know information about this file )
  • Modify path_parts, so any trailing / in the real_full_path will be removed to avoid problems. THIS PART IS NOT WELL TESTED AGAINST ALL TYPE OF CONFIGURATION !.

So this patch is less important than the 2 previous ones and depends on them.

Hope this help somebody, and thanks to oberger ;)


Caeies <Caeies>
Project Administrator


Attached Files
file #5541:  bugs_vfs_dav_ls_etc.diff added by Caeies (2KiB - text/x-patch - patch for these problems in vfs_dav)


   patch dependencies.

   patch dependencies.


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 4 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Wed 27 Oct 2004 01:26:31 PM UTCCaeiesDependencies-=>patch # is dependent
Wed 29 Sep 2004 08:01:47 AM UTCCaeiesDependencies-=>Depends on patch #3398
Wed 29 Sep 2004 07:58:58 AM UTCCaeiesAttached File-=>Added bugs_vfs_dav_ls_etc.diff, #3714
  Carbon-Copy-=>Added oberger

Back to the top

Powered by Savane 3.1-cleanup1