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

Submitter:  Caeies <Caeies>
Submitted:  Wed 29 Sep 2004 07:58:58 AM UTC
   
 
Category:  API - phpGWapi Priority:  5 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned 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 <Caeies>
Group 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 ;)

Regards,
Caeies

Caeies <Caeies>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

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

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 4 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2004-10-27 Caeies Dependencies- patch # is dependent
2004-09-29 Caeies Dependencies- Depends on patch #3398
2004-09-29 Caeies Attached File- Added bugs_vfs_dav_ls_etc.diff, #3714
    Carbon-Copy- Added oberger

Back to the top

Powered by Savane 3.13-758e.
Corresponding source code