Roundup Tracker - Issues

Message4386

Author joseph_myers
Recipients joseph_myers
Date 2011-08-21.23:00:19
Message-id <1313967620.84.0.742469956767.issue2550722@psf.upfronthosting.co.za>
In-reply-to
If you select "no selection" on a multilink field when editing an item,
and in fact nothing is selected for that multilink in the database item,
and you make no other changes (so submitting a null edit), a templating
error occurs on attempting to render the results of the null edit.  No
redirection occurs for a null edit (unlike for a non-null edit);
instead, the page for that item is rendered directly from the form
submission - with a context containing the values from that form
submission, and in particular a negative value for the multilink, which
is not a valid member of that class.

An example may make this clearer.  I verified the issue with the present
tracker as follows: I found an issue, issue2550709, with a multilink
with no values selected.  I submitted a first, erroneous, edit, that
both set "Components" to None (changing the no-values-selected multilink
to have a value selected) and cleared the title (having JavaScript
disabled so no client-side detection of the missing title could occur).
 This produced the expected error page with "Required issue property
title not supplied", and "Components" correctly showing "None" (i.e.,
that edit was not lost) (and the original title back) - and because of
"None" being selected, the "no selection" option was also present, which
it wasn't on the original issue page.  I then changed "Components" to
"no selection" (selecting "no selection", unselecting "None") and
resubmitted - this being the null edit, since the database already had
no Components value set.  The result was the "An error has occurred" page.
History
Date User Action Args
2011-08-21 23:00:20joseph_myerssetrecipients: + joseph_myers
2011-08-21 23:00:20joseph_myerssetmessageid: <1313967620.84.0.742469956767.issue2550722@psf.upfronthosting.co.za>
2011-08-21 23:00:20joseph_myerslinkissue2550722 messages
2011-08-21 23:00:20joseph_myerscreate