Bugfix: se rompía el edit de Enunciado cuando había un archivo.

[software/sercom.git] / sercom / subcontrollers / grupo / __init__.py

diff --git a/sercom/subcontrollers/grupo/__init__.py b/sercom/subcontrollers/grupo/__init__.py
index 29ad202ea1a94e89dae45a62b61cda32283e55f6..d2dfd7c88dceec28743f4c745f0e5f6c974ea552 100644 (file)
--- a/sercom/subcontrollers/grupo/__init__.py
+++ b/sercom/subcontrollers/grupo/__init__.py
@@ -99,6 +99,12 @@ ajax = u"""
 
     function buscar_alumno()
     {
 
     function buscar_alumno()
     {

+        /* Obtengo el padron ingresado */
+        p = MochiKit.DOM.getElement('form_responsable');
+        padron = p.value;
+        if (padron == '') {
+            return;
+        }

         /* Obtengo el curso */
         l = MochiKit.DOM.getElement('form_cursoID');
         cursoid = l.options[l.selectedIndex].value;
         /* Obtengo el curso */
         l = MochiKit.DOM.getElement('form_cursoID');
         cursoid = l.options[l.selectedIndex].value;


@@ -106,13 +112,6 @@ ajax = u"""
             alert('Debe seleccionar un curso');
             return;
         }
             alert('Debe seleccionar un curso');
             return;
         }

-        /* Obtengo el padron ingresado */
-        p = MochiKit.DOM.getElement('form_responsable');
-        padron = p.value;
-        if (padron == '') {
-            alert('Debe ingresar el padrón del alumno responsable');
-            return;
-        }

         url = "/grupo/get_inscripto?cursoid="+cursoid+'&padron='+padron;
         var d = loadJSONDoc(url);
         d.addCallbacks(procesar, err);
         url = "/grupo/get_inscripto?cursoid="+cursoid+'&padron='+padron;
         var d = loadJSONDoc(url);
         d.addCallbacks(procesar, err);


@@ -123,7 +122,7 @@ ajax = u"""
         connect('form_responsable', 'onblur', buscar_alumno);
     }
 
         connect('form_responsable', 'onblur', buscar_alumno);
     }
 

-    function onsubmit()
+    function doSubmit()

     {
         /* TODO : Validar datos y evitar el submit si no esta completo */
 
     {
         /* TODO : Validar datos y evitar el submit si no esta completo */
 


@@ -145,13 +144,13 @@ class GrupoForm(W.TableForm):
     class Fields(W.WidgetsList):
         cursoID = W.SingleSelectField(label=_(u'Curso'), options = get_cursos, validator = V.Int(not_empty=True))
         nombre = W.TextField(label=_(u'Nombre'), validator=V.UnicodeString(not_empty=True,strip=True))
     class Fields(W.WidgetsList):
         cursoID = W.SingleSelectField(label=_(u'Curso'), options = get_cursos, validator = V.Int(not_empty=True))
         nombre = W.TextField(label=_(u'Nombre'), validator=V.UnicodeString(not_empty=True,strip=True))

-        responsable = CustomTextField(label=_(u'Responsable'), validator=V.UnicodeString(not_empty=True), attrs=dict(size='8'))
+        responsable = CustomTextField(label=_(u'Responsable'), validator=V.UnicodeString(), attrs=dict(size='8'))

         miembros = AjaxMultiSelect(label=_(u'Miembros'), validator=V.Int(), on_add="alumnos_agregar_a_la_lista")
         tutores = W.MultipleSelectField(label=_(u'Tutores'), validator=V.Int(), options=get_docentes)
 
     fields = Fields()
     javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('curso');"), W.JSSource(ajax)]
         miembros = AjaxMultiSelect(label=_(u'Miembros'), validator=V.Int(), on_add="alumnos_agregar_a_la_lista")
         tutores = W.MultipleSelectField(label=_(u'Tutores'), validator=V.Int(), options=get_docentes)
 
     fields = Fields()
     javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('curso');"), W.JSSource(ajax)]

-    form_attrs = dict(onsubmit='return onsubmit()')
+    form_attrs = dict(onsubmit='return doSubmit()')

 
 def get_cursos():
     return [(0, u'---')] + [(fk1.id, fk1.shortrepr()) for fk1 in Curso.select()]
 
 def get_cursos():
     return [(0, u'---')] + [(fk1.id, fk1.shortrepr()) for fk1 in Curso.select()]


@@ -213,8 +212,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
             resp = AlumnoInscripto.selectBy(cursoID=kw['cursoID'],
                 alumno=Alumno.byPadron(kw['responsable'])).getOne()
         except SQLObjectNotFound:
             resp = AlumnoInscripto.selectBy(cursoID=kw['cursoID'],
                 alumno=Alumno.byPadron(kw['responsable'])).getOne()
         except SQLObjectNotFound:

-            flash(_(u'El responsable %s no existe') % resp)
-            raise redirect('list')
+            resp = None

         kw['responsable'] = resp
 
         r = validate_new(kw)
         kw['responsable'] = resp
 
         r = validate_new(kw)


@@ -255,8 +253,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
             resp = AlumnoInscripto.selectBy(cursoID=kw['cursoID'],
                 alumno=Alumno.byPadron(kw['responsable'])).getOne()
         except SQLObjectNotFound:
             resp = AlumnoInscripto.selectBy(cursoID=kw['cursoID'],
                 alumno=Alumno.byPadron(kw['responsable'])).getOne()
         except SQLObjectNotFound:

-            flash(_(u'El responsable %s no existe') % resp)
-            raise redirect('../list')
+            resp = None

         kw['responsable'] = resp
         r = validate_set(id, kw)
         flash(_(u'El %s fue actualizado.') % name)
         kw['responsable'] = resp
         r = validate_set(id, kw)
         flash(_(u'El %s fue actualizado.') % name)