#85 closed defect (fixed)
Descriptive svn: links generate different links than standard svn: links.
Reported by: | Owned by: | Jonas Borgström | |
---|---|---|---|
Priority: | highest | Milestone: | 0.5.2 |
Component: | wiki system | Version: | 0.5.1 |
Severity: | minor | Keywords: | |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
When a link is created of the form svn:/aDirectory/
the link points to /log/aDirectory/. When the link is created as [svn:/aDirectory/ Description]
the link points to /file/aDirectory/. This is problematic when /aDirectory/ is a directory as Trac attempts to retrieve the locations length from Subversion, generating an error.
This is caused by the fact that the fancysvnhref
regular expression is listed after the more generic fancylink
in svn:/trunk/trac/Wiki.py. Changing the order of the two regular expressions fixes the problem.
Attachments (0)
Change History (7)
comment:1 by , 21 years ago
comment:2 by , 21 years ago
Priority: | normal → highest |
---|---|
Severity: | normal → blocker |
comment:3 by , 21 years ago
Status: | new → assigned |
---|
comment:4 by , 21 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Probably fixed in [195]. Please reopen this ticket if it's still broken.
The new behavour is as follows:
svn:/path and [svn:/path description]
both creates a link to trac.cgi/browser/path
. The browser module will then display the directory contents, but the user will be redirected to
trac.cgi/file/path
if path points to a file instead of a directory.
comment:5 by , 21 years ago
Resolution: | fixed |
---|---|
Severity: | blocker → minor |
Status: | closed → reopened |
Changeset [195] fixes the problem that I reported, but I am unsure whether redirecting to trac.cgi/file/path when path is a file is desirable behaviour. It makes more sense in my mind to redirect to trac.cgi/log/path in that case, as this allows the user to view either the file's contents or the changeset if they so desire.
comment:6 by , 21 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Fixed in [196].
Yes you are right. The new behaviour is as follows:
File links are now redirected to the log module unless a specific file revision is requested.
svn:/path/file will display the file log svn:/path/file#rev will display the file contens for that revision.
comment:7 by , 19 years ago
Note that in 0.9 (and in 0.8 also, I'm pretty sure),
the svn:
prefix has no special meaning anymore:
- svn:/trunk
- svn:/trunk/README
Use source:
prefix instead (browser:
and repos:
are also supported)
Apologies for the bad formatting of the report.