Roundup Tracker - Issues

Issue 2550488

classification
make 'duplicate issue' more robust
Type: rfe Severity: normal
Components: Interface Versions:
process
Status: closed works for me
:
: rouilj : richard, rouilj, tonimueller
Priority: normal : patch

Created on 2009-01-30 22:29 by tonimueller, last changed 2016-07-02 20:24 by rouilj.

Files
File name Uploaded Description Edit Remove
roundup_templating.py.patch tonimueller, 2009-01-30 22:29
Messages
msg3493 Author: [hidden] (tonimueller) Date: 2009-01-30 22:29
When you have a due_date implemented in your tracker, like described in the docs, but with custom formatting (ie, read-only), then clicking 'duplicate issue' gives a traceback, saying 'str' object has no attribute 'local'. The attached patch fixes that. The result is that one can now duplicate an issue that has such a string field. The patch is against roundup 1.4.6.
msg5641 Author: [hidden] (rouilj) Date: 2016-06-26 23:34
I applied this patch and tests for cgi, templating pass.

But I don't know exactly how this is supposed to fail,
so I can't generate a test case.

So this is waiting for a reproducible fail that I can add to the tests.

Also finding how this is faliing may suggest other places that need
similar fixes.
msg5716 Author: [hidden] (rouilj) Date: 2016-07-02 20:24
I just tried the following:

Using the classic tracker in demo.py added:

                  due_date=Date()

to the end of the IssueClass("issue") definition.

then in issue.item.html I added:

<tr>
  <th i18n:translate="">Due Date</th>
     <td tal:content="structure
python:context.due_date.field(format='%Y-%m-%d')" />
</tr>

right before the Change Note.

If I click on "Make a copy" which I assume is the same as your
"duplicate issue" I get a perfectly good copy of the issue with all
fields copied.

I am closing this as works for me. If you can tell me how to reproduce
this in a demo.py tracker I will investigate further.

-- rouilj
History
Date User Action Args
2016-07-02 20:24:10rouiljsetstatus: open -> closed
assignee: richard -> rouilj
resolution: works for me
messages: + msg5716
2016-06-26 23:34:24rouiljsetkeywords: + patch
nosy: + rouilj
messages: + msg5641
2009-01-30 22:29:08tonimuellercreate