Fix form-parsing for multilinks

If multiple new items are added to a multilink property, the old version
would create the new items but only link one.

Author: schlatterbeck

CHANGES.txt

@@ -162,6 +162,9 @@ Fixed:
   Suggested by Karl-Philipp Richter. (Bernhard Reiter)
 - issue2550884 roundup-mailgw --help text improved to explain the allowed 
   parameters better. Suggested by by Karl-Philipp Richter. (Bernhard Reiter)
+- Fix form-parsing: If multiple new items are added to a multilink
+  property, the old version would create the new items but only link
+  one. (Ralf Schlatterbeck)
 
 
 2013-07-06: 1.5.0

roundup/cgi/actions.py

@@ -490,6 +490,8 @@ def _editnodes(self, all_props, all_links):
                                 props[linkprop] = existing
                             else:
                                 props[linkprop] = nodeid
+                    elif isinstance(propdef, hyperdb.Multilink):
+                        props[linkprop].append(nodeid)
 
         return '\n'.join(m)
 

test/test_actions.py

@@ -234,14 +234,16 @@ class EditItemActionTestCase(ActionTestCase):
     def setUp(self):
         ActionTestCase.setUp(self)
         self.result = []
+        self.new_id = 16
         class AppendResult:
             def __init__(inner_self, name):
                 inner_self.name = name
             def __call__(inner_self, *args, **kw):
                 self.result.append((inner_self.name, args, kw))
                 if inner_self.name == 'set':
                     return kw
-                return '17'
+                self.new_id+=1
+                return str(self.new_id)
 
         self.client.db.security.hasPermission = true
         self.client.classname = 'issue'
@@ -274,6 +276,24 @@ def testMessageAttach(self):
             pass
         self.assertEqual(expect, self.result)
 
+    def testMessageMultiAttach(self):
+        expect = \
+            [ ('create',(),{'content':'t2'})
+            , ('create',(),{'content':'t'})
+            , ('set',('4711',), {'messages':['23','42','17','18']})
+            ]
+        self.client.db.classes.get = lambda a, b:['23','42']
+        self.client.parsePropsFromForm = lambda: \
+            ( {('msg','-1'):{'content':'t'},('msg','-2'):{'content':'t2'}
+              , ('issue','4711'):{}}
+            , [('issue','4711','messages',[('msg','-1'),('msg','-2')])]
+            )
+        try :
+            self.action.handle()
+        except Redirect, msg:
+            pass
+        self.assertEqual(expect, self.result)
+
     def testFileAttach(self):
         expect = \
             [('create',(),{'content':'t','type':'text/plain','name':'t.txt'})