Modify

Opened 9 years ago

Closed 9 years ago

#2943 closed defect (fixed)

Filename listed in download list should not include the uploader's local directory path

Reported by: anonymous Owned by: Radek Bartoň
Priority: normal Component: DownloadsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

After uploading a file using the DownloadsPlugin admin page, the file becomes listed using the absolute pathname, as uploaded by the uploader. This includes the local directories where the uploaded file lived as it was uploaded. The directory path should be stripped as it reflects the uploaders directory structure and is not relevant, and most likely unrelated to any downloader's directory structure.

Attachments (1)

trac_downloads_plugin_bug.jpg (50.2 KB) - added by Eric Swenson 9 years ago.
Screen shot of downloads view with full path of uploaded file displayed

Download all attachments as: .zip

Change History (8)

comment:1 Changed 9 years ago by Radek Bartoň

Status: newassigned

Which UA on which OS? Which server on which OS? Please.

comment:2 Changed 9 years ago by anonymous

I'm using Internet Explorer 7, on Windows XP SP2.

There is a further problem -- not only are the downloads listed using the absolute pathname of the file (as it was uploaded), but the file ends up being stored somewhere other than the location as configured by the [downloads] "path" setting. I think this is because the pathname (as it comes from the upload) is being interpreted as the location to store the uploaded entry on the trac server.

comment:3 Changed 9 years ago by Eric Swenson

I've just registered on the site, but I'm the guy who submitted this ticket.

I forgot to provide you the information that the server machine is also a Windows XP SP2 machine and the server is tracd. (I'm actually evaluating this plugin for use on an apache-based trac instance, where it fails in the same way, but was doing testing on my local (Windows) machine, which is just using tracd). If you cannot reproduce, I'll get some screen shots and upload them tomorrow.

Changed 9 years ago by Eric Swenson

Screen shot of downloads view with full path of uploaded file displayed

comment:4 Changed 9 years ago by Radek Bartoň

OK, thx. I hope that it is a Windows UA (I don't test on Windows) which gives full path. I'll append stripping of this later this day...

comment:5 Changed 9 years ago by Radek Bartoň

Please test it with r3545.

comment:6 in reply to:  5 Changed 9 years ago by Eric Swenson

Replying to Blackhex:

Please test it with r3545.

It worked perfectly. Thanks much. -- Eric

comment:7 Changed 9 years ago by Radek Bartoň

Resolution: fixed
Status: assignedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Radek Bartoň.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.