Hi,

1) My site is running at http://localhost/drupal.
2) I've installed the webdav module for filesystems and created a filesystem called 'dave'.
3) When I try to add http://localhost/drupal/webdav to explorer this nicely works but then it shows me two maps . One is 'webdav' with a url 'http://localhost/webdav', the other is 'dave' with a url 'http://localhost/webdav/dave'.

Note that in both these url's the '/drupal/' is missing!

CommentFileSizeAuthor
#5 webdav-base_path_fix-375115-5.patch2.1 KBSQP

Comments

wvd_vegt’s picture

Hi,

I found at least in one place that generates a wrong link.

In server.inc , function convert_item() the line:

    $file['path'] =  webdav_route_to_path($route);

should read:

    $file['path'] =  base_path().webdav_route_to_path($route);

This nicely results in '/drupal/' being inserted in front of the filename and windows explorer then allows me to navigate and download.

What still doesn't work after this is uploading a file by dropping it onto an explorer window.

Please note I did not test this on a root installation of drupal (do not have one) and tested only with windows explorer as it's the only webdav client available to me.

Yoran’s picture

Can you have a look at the other comment I wrote about the inernal browser ?

Yoran’s picture

feel free to open this again is your issue is not solved.

Yoran’s picture

Status: Active » Closed (fixed)
SQP’s picture

Version: 6.x-1.0-rc7 » 6.x-1.0-rc8
Status: Closed (fixed) » Needs review
StatusFileSize
new2.1 KB

Confirmed both bug and patch, so I reopen that one (even if it duplicates with #532172: Not working when Drupal is installed below DocumentRoot)
If drupal_base is different from / we can't access webdav repository

Added the fix for icons when using internal browser.
Tested on both configurations (with and without base path) : listing and view with internal browser and listing and edition with gnome client (gnome automount and gedit)

damienmckenna’s picture

Status: Needs review » Reviewed & tested by the community

Looks good. Could use some syntax improvements for the Drupal coding standards, but that's a general problem with the module.

mably’s picture

Issue summary: View changes
Status: Reviewed & tested by the community » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.