Limpiar y simplificar GrupoController.

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

diff --git a/sercom/subcontrollers/curso/grupo/__init__.py b/sercom/subcontrollers/curso/grupo/__init__.py
index 149f99bb37031f871535a2d00d3aa5e391be9d96..b76e9a4509f2178eb77d1e6cb322978524e415d6 100644 (file)
--- a/sercom/subcontrollers/curso/grupo/__init__.py
+++ b/sercom/subcontrollers/curso/grupo/__init__.py
@@ -489,7 +489,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
             flash(_(u'Error A %s.' % e))
             raise redirect(url('/curso/grupo/list' % int(cursoID)))
         # seteo el reponsable del grupo
             flash(_(u'Error A %s.' % e))
             raise redirect(url('/curso/grupo/list' % int(cursoID)))
         # seteo el reponsable del grupo

-        if int(responsableA) != 0:
+        if responsableA and int(responsableA) != 0:

             grupoA.responsable = AlumnoInscripto.get(int(responsableA))
 
         for t in tutoresA:
             grupoA.responsable = AlumnoInscripto.get(int(responsableA))
 
         for t in tutoresA:


@@ -539,7 +539,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
                 flash(_(u'Error B %s.' % e))
                 raise redirect(url('/curso/grupo/list/%d' % int(cursoID)))
             # seteo el reponsable del grupo
                 flash(_(u'Error B %s.' % e))
                 raise redirect(url('/curso/grupo/list/%d' % int(cursoID)))
             # seteo el reponsable del grupo

-            if int(responsableB) != 0:
+            if responsableB and int(responsableB) != 0:

                 grupoB.responsable = AlumnoInscripto.get(int(responsableB))
 
             #Elimino el grupo si quedo vacio
                 grupoB.responsable = AlumnoInscripto.get(int(responsableB))
 
             #Elimino el grupo si quedo vacio