Roundup Tracker - Issues

Issue 2550819

classification
Title: "Make a copy" always tries to write change not to @note although :note is also valid
Type: behavior Severity: normal
Components: Versions:
process
Status: new Resolution:
Dependencies: Superseder:
Assigned To: rouilj Nosy List: rouilj
Priority: Keywords: Effort-Medium

Created on 2013-10-07 03:49 by rouilj, last changed 2016-07-09 21:02 by rouilj.

Messages
msg4929 Author: [hidden] (rouilj) Date: 2013-10-07 03:49
In:

http://www.roundup-tracker.org/docs/customizing.html#special-form-variables

it is stated that:
===================
In the following, <bracketed> values are variable, “@” may be either ”:”
or “@”, and other text “required” is fixed.
[...]
Two special form values are supported for backwards compatibility:

@note
    This is equivalent to:
      @link@messages=msg-1
      msg-1@content=value
    except that in addition, the “author” and “date” properties of
    “msg-1” are set to the userid of the submitter, and the current
    time, respectively.
=======

If the user renames the message @note field to :note, the textarea
is not populated with the text from the "Make a copy" url created by
copy_url.

One idea I have to address this is to add an additional parameter
to copy_url(..., special_char='@', ...) that can be overridden
when called to use ":". It looks like only @note is affected
since the only other special variable is @template which is
interpreted by the roundup code and not used in a template.

Does this seem to be a reasonable way to handle it?

The only other way I can think of doing this is to copy the function
into the tracker and make the change in the copy then call the new
function from the templates. Blech.

Any other ideas, comments?

Also it just occurred to me, is the : or the @ form the preferred
form. I remember one of them is the legacy character, but I don't
remember which.
History
Date User Action Args
2016-07-09 21:02:36rouiljsettype: behavior
2016-07-09 21:02:27rouiljsetkeywords: + Effort-Medium
2013-10-07 03:49:37rouiljcreate