Roundup Tracker - Issues

Message4928

Author rouilj
Recipients ThomasAH, rouilj
Date 2013-10-07.03:38:08
Message-id <1381117089.74.0.497277812851.issue2550673@psf.upfronthosting.co.za>
In-reply-to
I think this is a problem with any multilink field.

The following patch to roundup/cgi/templating.py I think fixes it

         for name in self._props.keys():
             if name not in exclude:
-                query[name] = self[name].plain()
+                prop = self._props[name]
+                if not isinstance(prop, hyperdb.Multilink):
+                    query[name] = self[name].plain()
+                else:
+                    query[name] =
",".join(self._klass.get(self._nodeid, name))
+
         return self._classname + "?" + "&".join(

If it's not a multilink then copy the attributes across as it
does currently. If it's a multilink, get the list of multilink
values from the database and turn it into a comma separated list.

AFAIK, copy_url always works with the data from the database
and not any changes done to the form displaying the "Make a copy"
url. Can anybody confirm this assumption?

If my assumption is correct, I will check in this change for further
testing and release on the next release cycle.
History
Date User Action Args
2013-10-07 03:38:09rouiljsetmessageid: <1381117089.74.0.497277812851.issue2550673@psf.upfronthosting.co.za>
2013-10-07 03:38:09rouiljsetrecipients: + rouilj, ThomasAH
2013-10-07 03:38:09rouiljlinkissue2550673 messages
2013-10-07 03:38:08rouiljcreate