Especificar con más detalle TODO del backend.

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

diff --git a/sercom/subcontrollers/misentregas/__init__.py b/sercom/subcontrollers/misentregas/__init__.py
index a516c8817c03472bbdab60c15be047e685d4ca06..a7c3c3138285067c13057af809fcee45445bc202 100644 (file)
--- a/sercom/subcontrollers/misentregas/__init__.py
+++ b/sercom/subcontrollers/misentregas/__init__.py
@@ -10,8 +10,10 @@ from turbogears import identity
 from turbogears import paginate
 from docutils.core import publish_parts
 from sercom.subcontrollers import validate as val
-from sercom.model import Entrega, Correccion, Curso, Ejercicio, InstanciaDeEntrega
+from sercom.model import Entrega, Correccion, Curso, Ejercicio, InstanciaDeEntrega, Grupo, Miembro, AlumnoInscripto
 from sqlobject import *
+from zipfile import ZipFile, BadZipfile
+from cStringIO import StringIO
 
 #}}}
 
@@ -34,7 +36,9 @@ def validate_new(data):
 
 def get_ejercicios_activos():
     # TODO : Mostrar solo los ejercicios con instancias de entrega activos
-    return [(0, _(u'--'))] + [(fk.id, fk.shortrepr()) for fk in Ejercicio.select()]
+    return [(0, _(u'--'))] + [(a.id, a.shortrepr()) for a in (Ejercicio.select(
+        AND(Ejercicio.q.id==InstanciaDeEntrega.q.ejercicioID, InstanciaDeEntrega.q.inicio <= DateTimeCol.now(),
+            InstanciaDeEntrega.q.fin >= DateTimeCol.now())))]
 
 ajax = """
     function clearInstancias ()
@@ -122,23 +126,44 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
         r = cls.select(cls.q.entregadorID == identity.current.user.id)
         return dict(records=r, name=name, namepl=namepl)
 
-    @expose(template='kid:%s.templates.show' % __name__)
-    def show(self,id, **kw):
-        """Show record in model"""
-        r = validate_get(id)
-        if r.observaciones is None:
-            r.obs = ''
-        else:
-            r.obs = publish_parts(r.observaciones, writer_name='html')['html_body']
-        return dict(name=name, namepl=namepl, record=r)
-
     @validate(form=form)
     @error_handler(new)
     @expose()
     def create(self, archivo, ejercicio, **kw):
         """Save or create record to model"""
+        try:
+            zfile = ZipFile(archivo.file)
+        except BadZipfile:
+            flash(_(u'El archivo ZIP no es valido'))
+            raise redirect('list')
+
+        # por defecto el entregador es el user loggeado
+        entregador = identity.current.user
+
+        ejercicio = Ejercicio.get(int(ejercicio))
+        if ejercicio.grupal:
+            # Como es grupal, tengo que hacer que la entrega la haga
+            # mi grupo y no yo personalmente. Busco el grupo
+            # activo.
+
+            # Con esto obtengo todos los grupos a los que pertenece el Alumno
+            # y que estan activos
+            try:
+                # TODO : Falta filtrar por curso!!
+                m = Miembro.select(
+                    AND(
+                        Miembro.q.alumnoID == AlumnoInscripto.q.id,
+                        AlumnoInscripto.q.alumnoID == identity.current.user.id,
+                        Miembro.q.baja == None
+                    )
+                ).getOne()
+            except:
+                flash(_(u'No puedes realizar la entrega ya que el ejercicio es Grupal y no perteneces a ningún grupo.'))
+                raise redirect('list')
+
+            entregador = m.grupo
         kw['archivos'] = archivo.file.read()
-        kw['entregador'] = identity.current.user
+        kw['entregador'] = entregador
         validate_new(kw)
         flash(_(u'Se creó una nueva %s.') % name)
         raise redirect('list')