From 804ac9256582156d0409199526c6652e0d7eb586 Mon Sep 17 00:00:00 2001
From: helmutm <helmutm@fd906abe-77d9-0310-91a1-e0d9ade77398>
Date: Fri, 6 Oct 2006 10:51:28 +0000
Subject: [PATCH] bug fixes for dialogs

git-svn-id: svn://svn.cy55.de/Zope3/src/loops/trunk@1390 fd906abe-77d9-0310-91a1-e0d9ade77398
---
 browser/form.py  | 2 +-
 browser/loops.js | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/browser/form.py b/browser/form.py
index 81704a0..65e6ca4 100644
--- a/browser/form.py
+++ b/browser/form.py
@@ -194,7 +194,7 @@ class EditObject(FormController):
         for v in value:
             if fieldName == 'old':
                 self.old.append(v)
-            elif fieldName == 'selected':
+            elif fieldName == 'selected' and v not in self.selected:
                 self.selected.append(v)
 
     def assignConcepts(self, obj):
diff --git a/browser/loops.js b/browser/loops.js
index 335a14e..e00b3d2 100644
--- a/browser/loops.js
+++ b/browser/loops.js
@@ -78,9 +78,10 @@ function objectDialog(dlgName, url) {
 }
 
 function addConceptAssignment() {
+    dojo.require('dojo.html')
     node = dojo.byId('form.assignments');
     els = document.forms[0].elements;
-    for (var i=0; i<els.length; i++) {
+    for (var i=0; i<els.length; i++) { //getElementsByName does not work in IE
         el = els[i];
         if (el.name == 'concept.search.text_selected') {
             cToken = el.value;
@@ -88,19 +89,18 @@ function addConceptAssignment() {
             title = el.value;
         }
     }
-    //cToken = document.getElementsByName('concept.search.text_selected')[0].value;
     if (cToken.length == 0) {
         alert('Please select a concept!');
         return false;
     }
     pToken = dojo.byId('concept.search.predicate').value;
     token = cToken + ':' + pToken;
-    //title = document.getElementsByName('concept.search.text')[0].value;
     var td = document.createElement('td');
-    td.setAttribute('colspan', '5');
+    td.colSpan = 5;
     td.innerHTML = '<input type="checkbox" name="form.assignments.selected:list" value="' + token + '" checked><span>' + title + '</span>';
     var tr = document.createElement('tr');
     tr.appendChild(td);
+    //alert(tr.firstChild.attributes['colSpan'].value);
     node.appendChild(tr);
 }