Roundup Tracker - Issues

Issue 2551181

classification
Allow issuexxx#fragment to generate a link with fragment
Type: rfe Severity: normal
Components: Web interface Versions: 2.2.0
process
Status: fixed fixed
:
: rouilj : rouilj
Priority: normal : Effort-Low

Created on 2021-12-11 22:11 by rouilj, last changed 2021-12-16 04:55 by rouilj.

Messages
msg7414 Author: [hidden] (rouilj) Date: 2021-12-11 22:11
To go along with issue2551180, it would be nice to allow:

  issue2551180#msg7413

  issue 2551180#msg7413

be generate hyperlinks to https://issues.roundup-tracker.org/issue2551180#msg7413

Currently I think it only generates https://issues.roundup-tracker.org/issue2551180
or possibly two links one to the issue and one to the message.

This may be tricky to make work for the markdown tests.
They call _hyper_repl_item in templating.py and their patterns
may need to be changed to support this.

I suspect the regexp needs to look like:

  (?P<item>(?P<class>[A-Za-z_]+)(\s*)(?P<id>\d+)(#(?P<frag>[^][#%^{}\"<>\s]+)?)

so add an optional #fragment not including the characters shown plus space characters
as we use those as the end of fragment delimiter. This will not handle %20 url
encoding to allow spaces but for a first pass this should work.
msg7422 Author: [hidden] (rouilj) Date: 2021-12-16 04:55
Done in changeset:   6564:21c7c2041a4b

I did not include the % sign in the excluded character list, so %20 and other
URL encodings should work.
History
Date User Action Args
2021-12-16 04:55:04rouiljsetstatus: new -> fixed
versions: + 2.2.0
messages: + msg7422
priority: normal
assignee: rouilj
keywords: + Effort-Low, - Effort-Medium
resolution: fixed
2021-12-11 22:11:27rouiljcreate