Roundup Tracker - Issues

Message6101

Author rouilj
Recipients rouilj
Date 2018-07-07.22:33:21
Message-id <1531002805.14.0.56676864532.issue2550964@psf.upfronthosting.co.za>
In-reply-to
Set up a classic tracker with the attached detector.

1 Create an issue and save it.
2 Now change the Title and commit the change.

Look at the history and you will see the change recorded. The right
hand side of the change will be the current title (state A).

3 Now change the title to include the word rejectme.

You should see an error:

  Edit Error: subject rejected because keyword rejectme used

if you look at the history at the bottom of the issue, you will see
the new title with the word rejectme where the original title was (state B).

E.G.

at A you see:

2018-07-07 18:06:04	admin	set	title: Do you like foo foo -> Do you like
foo bar
2018-07-07 18:00:36	admin	set	title: Do you like foo -> Do you like foo foo

at B you see:

2018-07-07 18:06:04	admin	set	title: Do you like foo foo -> Do you like
foo bar rejectme
2018-07-07 18:00:36	admin	set	title: Do you like foo -> Do you like foo foo

The rejected title is *not* committed to the database.
This can be seen with roundup-admin history issueX or display issueX.
So this is not the same as issue2550955.

I think the code that preserves changes in the web page is causing the
history mechanism to display the web form data and not the data in the
database.

So the "current" history element that is displayed on the right hand
side is the web form value.

It should be possible to insert TAL before the history generation
section of the template to retrieve/display the database value.
E.G. adding:

<span tal:omit-tag="python:True"
 tal:content="python:utils.set_form_wins(request.client,False)">
<span>

before the call to context.history in the template seems to fix it
in my test. So need to add to every template.

Fixing the rejected change fixes this issue. As a result I consider this
a cosmetic issue, so not marking it as a blocker.
History
Date User Action Args
2018-07-07 22:33:25rouiljsetrecipients: + rouilj
2018-07-07 22:33:25rouiljsetmessageid: <1531002805.14.0.56676864532.issue2550964@psf.upfronthosting.co.za>
2018-07-07 22:33:24rouiljlinkissue2550964 messages
2018-07-07 22:33:23rouiljcreate