js bug

[old-projects.git] / philter / philter / philter.js

diff --git a/philter/philter/philter.js b/philter/philter/philter.js
index 9ccd363..7ce229e 100644 (file)
--- a/philter/philter/philter.js
+++ b/philter/philter/philter.js
@@ -29,8 +29,11 @@ function getElement(obj) {
     return false;
 }
 
-function del_onsubmit() {
-    return window.confirm("Voulez vous réellement supprimer cette adresse mail ?");
+function del_onclick(val) {
+    if(window.confirm("Do you really want to delete this email address ?")) {
+        getElement('emailsDel').value = val;
+        document.forms['emails'].submit();
+    }
 }
 
 /********** ORDER FORM FUNCTIONS **********/
@@ -57,7 +60,7 @@ function order_up() {
 
 function order_dn() {
     form = document.forms['order'];
-    select = getElement['orderSelect'];
+    select = getElement('orderSelect');
 
     if(select.selectedIndex>0 && select.selectedIndex<select.length-1) {
         i = select.selectedIndex;
@@ -77,9 +80,9 @@ function order_dn() {
 
 function order_submit(obj) {
     if(obj.name == 'order[submit]')
-        obj.form.elements['orderAction'].value = 'submit';
+        obj.form.elements['order[action]'].value = 'submit';
     if(obj.name == 'order[delete]')
-        obj.form.elements['orderAction'].value = 'delete';
+        obj.form.elements['order[action]'].value = 'delete';
     obj.form.submit();
 }
 
@@ -101,12 +104,14 @@ var matches_i = 0;
 
 /********** RULE.ACTION FORM FUNCTIONS **********/
 
-function createActionSelect(data,base) {
+function createActionSelect(div,data) {
     var i,j;
     var select = document.createElement("select");
     var sel = 0;
-    select.name = base+'[0]';
+    select.name = div.name+'[0]';
 
+    // OG: must insert select node before creating options because of IE5 bug
+    div.appendChild(select);
     for(i=0, j=0; i<actions_list.length; i++) {
         if(data==i) sel = j;
         if(actions_list[i]) select.options[j++] = new Option(actions_list[i],i,false,(data==i));
@@ -120,7 +125,6 @@ function createActionSelect(data,base) {
 
 function createActionRow(Node,data) {
     var div = document.createElement("div");
-    div.className = 'row';
     div.name = 'rule[actions]['+actions_i+']';
     actions_i++;
     Node.appendChild(div);
@@ -133,8 +137,7 @@ function createActionRow(Node,data) {
 
     div.appendChild(document.createTextNode(" "));
 
-    var select = createActionSelect(data[0],div.name);
-    div.appendChild(select);
+    var select = createActionSelect(div,data[0]);
 
     actions_func[select.options[select.selectedIndex].value](div,data);
 }
@@ -145,12 +148,14 @@ function newAction() {
 
 /********** RULE.MATCH FORM FUNCTIONS **********/
 
-function createMatchSelect(data,base) {
+function createMatchSelect(div,data) {
     var i,j;
     var select = document.createElement("select");
     var sel = 0;
-    select.name = base+'[0]';
+    select.name = div.name+'[0]';
 
+    // OG: must insert select node before creating options because of IE5 bug
+    div.appendChild(select);
     for(i=0, j=0; i<matches_list.length; i++) {
         if(data==i) sel = j;
         if(matches_list[i]) select.options[j++] = new Option(matches_list[i],i,false,(data==i));
@@ -164,7 +169,6 @@ function createMatchSelect(data,base) {
 
 function createMatchRow(Node,data) {
     var div = document.createElement("div");
-    div.className = 'row';
     div.name = 'rule[matches]['+actions_i+']';
     actions_i++;
     Node.appendChild(div);
@@ -177,8 +181,7 @@ function createMatchRow(Node,data) {
 
     div.appendChild(document.createTextNode(" "));
 
-    var select = createMatchSelect(data[0],div.name);
-    div.appendChild(select);
+    var select = createMatchSelect(div,data[0]);
 
     matches_func[select.options[select.selectedIndex].value](div,data);
 }