#9 Action links (D/E) incorrect from root

[David Ethell]

I have several problems related to how the action
links are displayed by cvs-web/read/cvswebread.cgi as
follows:

When in the root directory, if I have a directory
called dirname the link to download that directory
will appear as:

http://blah.com/cvs-web/cvswebedit.cgi///dirname/?
edit=download-dir-page

The triple slash is messing up the download
functionality. You can see that when cvswebedit.cgi
receives the directory to be download it sees it as:

//dirname

The tar file will be empty because it is unable to
check any such directory out of cvs. If I correct the
URL after I have clicked the download to remove the
redundant slashes the download functionality works.

Perhaps I have a configuration error, but it would
seem to be in the way the changeDirectory function
builds the $where variable.

The download link from a subdirectory also has
problems:

dirname/subdirname

this link will appear as:

http://blah.com/cvs-
web/cvswebedit.cgi/dirname//subdirname/?edit=download-
dir-page

this generates the same problem such that I am unable
to download any directories using the link. I must
click the link and then edit the URL to get the
correct contents in the tar file.

Secondly, I'm having similar problems with the Edit
link, but only from the root directory. If I have a
file in the root dir the edit link looks like:

http://blah.com/cvs-web/edit/cvswebedit.cgi//New_File?
edit=start

You can see the double slash after cvswebedit.cgi.
Once the link opens I can correct the URL and I'm able
to edit the file just fine. Subdirectories do not have
this problem and I'm able to edit correctly.

Thank you,

David Ethell
dethell@sscdinc.com

[Kumar M. Deepak]

Logged In: YES
user_id=147044

Any patches for these problem. If so please let me know.

Deepak Kumar
kmdeepak@operamail.com

[David Ethell]

Logged In: YES
user_id=152930

I submitted a patch that fixes part of it. I think the
patch number is: 416541. It corrects downloading from a
subdirectory. Trying to download from the topmost level
still doesn't work with this patch. There is one too many
slashes.