cleanup ajax y mejor manejo de errores

[z.facultad/75.52/sercom.git] / sercom / widgets.py

diff --git a/sercom/widgets.py b/sercom/widgets.py
index 218fb3aee343a4f21214c74dfc56670a8ac5600e..b4b7e13f9093528112466290e057ee7095f1b8da 100644 (file)
--- a/sercom/widgets.py
+++ b/sercom/widgets.py
@@ -28,7 +28,7 @@ AlumnoMultiSelectAjax = '''
             alert(results.msg);
             return;
         }
-        MochiKit.DOM.appendChildNodes(lista, OPTION(results.msg));
+        MochiKit.DOM.appendChildNodes(lista, OPTION({'value':results.msg.id}, results.msg.value));
     }
 
     function _on_alumno_get_error(results)
@@ -39,10 +39,24 @@ AlumnoMultiSelectAjax = '''
 
     function agregar_a_la_lista(texto, lista)
     {
+        t = MochiKit.DOM.getElement(texto);
+
+        /* Como no se si se puede hacer de otra manera, asumo que tengo en
+         * el form un Combo que se llama curso en el codigo, y tiro error si
+         * no existe
+         */
+        curso = MochiKit.DOM.getElement('form_cursoID');
+        if (!curso) {
+            alert("No deberias ver esto, y quiere decir que tu form esta roto.\\nTe falta un combo de curso");
+            return;
+        }
+        if (curso.options[curso.selectedIndex].value <= 0) {
+            alert('Debes seleccionar un curso primero');
+            return;
+        }
         load = MochiKit.DOM.getElement('loading');
         load.style.visibility = 'visible';
-        t = MochiKit.DOM.getElement(texto);
-        url = "/grupo/get_inscripto?cursoid=1&padron="+t.value;
+        url = "/grupo/get_inscripto?cursoid="+curso.options[curso.selectedIndex].value+"&padron="+t.value;
         var d = loadJSONDoc(url);
         d.addCallbacks(partial(_on_alumno_get_result, lista), _on_alumno_get_error);
         t.value = "";