Roundup Tracker - Issues

Message4932

Author rouilj
Recipients ThomasAH, rouilj
Date 2013-10-07.13:22:47
Message-id <201310071322.r97DMhbD004520@mx1.cs.umb.edu>
In-reply-to Your message of "Mon, 07 Oct 2013 06:29:44 -0000." <1381127384.24.0.717452925435.issue2550673@psf.upfronthosting.co.za> <1381127384.24.0.717452925435.issue2550673@psf.upfronthosting.co.za>
In message <1381127384.24.0.717452925435.issue2550673@psf.upfronthosting.co.za>
 <1381127384.24.0.717452925435.issue2550673@psf.upfronthosting.co.za>,
Thomas Arendsen Hein writes:
>Thomas Arendsen Hein added the comment:
>
>The "Make a copy" link is not a button, so (without some extra magic) it
>can't use the form data.
>I think improving that part and making the copy function work in general
>are separate things.

Ok good. That was my understanding as well.

>Regarding your patch: I think it looks good. I haven't checked, but I
>assume that the multilinks always only include numbers, so no escaping
>is needed, correct?

Yes. So far I have tested it with the nosy list (multilink to users),
dependson list (multilink to issues) and topic (multilink to
keywords). So I'm assuming it will work with other multilinks as well.

It is url encoded to:

   keyword=5%2C6%2C8&nosy=1%2C3

which is keyword=5,6,8 nosy=1,3.
History
Date User Action Args
2013-10-07 13:22:48rouiljsetrecipients: + rouilj, ThomasAH
2013-10-07 13:22:48rouiljlinkissue2550673 messages
2013-10-07 13:22:47rouiljcreate