issue2550748: Crash when creating new issues with non-existing multilink values (in classic template). Applied patch provided by user. (John Rouillard)

Author: rouilj

CHANGES.txt

@@ -31,6 +31,9 @@ Fixed:
 - issue2550601: gsoc-2009 "bug" class doesn't have "patches" property
   Added multilink to patches to the bug schema in the devel template.
   (John Rouillard)
+- issue2550748: Crash when creating new issues with non-existing
+  multilink values (in classic template). Applied patch so it
+  now errors the same way as an update does.  (John Rouillard)
 
 2016-01-11: 1.5.1
 

roundup/cgi/templating.py

@@ -363,7 +363,11 @@ def lookupKeys(linkcl, key, ids, num_re=num_re):
     l = []
     for entry in ids:
         if num_re.match(entry):
-            label = linkcl.get(entry, key)
+            try:
+		label = linkcl.get(entry, key)
+	    except IndexError:
+		# fall back to id if illegal (avoid template crash)
+                label = entry
             # fall back to designator if label is None
             if label is None: label = '%s%s'%(linkcl.classname, entry)
             l.append(label)