Bugfix: la carga masiva de alumnos guardaba el password sin encriptar.

[z.facultad/75.52/sercom.git] / sercom / subcontrollers / misentregas / __init__.py

diff --git a/sercom/subcontrollers/misentregas/__init__.py b/sercom/subcontrollers/misentregas/__init__.py
index e6f97fb1ba0c517aaeb83577bccca67cda7d6c9a..31c76783293a28a790a810ad1a978e2550fb3d22 100644 (file)
--- a/sercom/subcontrollers/misentregas/__init__.py
+++ b/sercom/subcontrollers/misentregas/__init__.py
@@ -10,7 +10,7 @@ from turbogears import identity
 from turbogears import paginate
 from docutils.core import publish_parts
 from sercom.subcontrollers import validate as val
 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
 from sqlobject import *
 from zipfile import ZipFile, BadZipfile
 from cStringIO import StringIO


@@ -105,6 +105,8 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
     """Basic model admin interface"""
     require = identity.has_permission('entregar')
 
     """Basic model admin interface"""
     require = identity.has_permission('entregar')
 

+    hide_to_admin = 1
+

     @expose()
     def default(self, tg_errors=None):
         """handle non exist urls"""
     @expose()
     def default(self, tg_errors=None):
         """handle non exist urls"""


@@ -123,36 +125,76 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
     @paginate('records')
     def list(self):
         """List records in model"""
     @paginate('records')
     def list(self):
         """List records in model"""

-        r = cls.select(cls.q.entregadorID == identity.current.user.id)
+        # Grupos en los que el usuario formo parte
+        m = [i.grupo.id for i in Grupo.selectByAlumno(identity.current.user)]
+        try:
+            entregador = AlumnoInscripto.selectByAlumno(identity.current.user)
+            m.append(entregador.id)
+        except:
+            pass
+        r = cls.select(IN(cls.q.entregadorID, m))

         return dict(records=r, name=name, namepl=namepl)
 
         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"""
     @validate(form=form)
     @error_handler(new)
     @expose()
     def create(self, archivo, ejercicio, **kw):
         """Save or create record to model"""

+        archivo = archivo.file.read()

         try:
         try:

-            zfile = ZipFile(StringIO(archivo.file.read()), 'r')
+            zfile = ZipFile(StringIO(archivo), 'r')

         except BadZipfile:
         except BadZipfile:

-            flash(_(u'El archivo ZIP no es valido'))
+            flash(_(u'El archivo ZIP no es válido'))
+            raise redirect('list')
+        if zfile.testzip() is not None:
+            flash(_(u'El archivo ZIP tiene errores de CRC'))

             raise redirect('list')
 
             raise redirect('list')
 

-        kw['archivos'] = archivo.file.read()
-        kw['entregador'] = identity.current.user
+        # por defecto el entregador es el user loggeado
+        entregador = AlumnoInscripto.selectByAlumno(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
+        kw['entregador'] = entregador

         validate_new(kw)
         flash(_(u'Se creó una nueva %s.') % name)
         raise redirect('list')
 
         validate_new(kw)
         flash(_(u'Se creó una nueva %s.') % name)
         raise redirect('list')
 

+    @expose(template='kid:%s.templates.corrida' % __name__)
+    def corrida(self, entregaid):
+        e = validate_get(entregaid)
+        return dict(entrega=e)
+
+    @expose()
+    def get_archivo(self, entregaid):
+        from cherrypy import request, response
+        r = validate_get(entregaid)
+        response.headers["Content-Type"] = "application/zip"
+        response.headers["Content-disposition"] = "attachment;filename=Ej_%s-Entrega_%s-%s.zip" % (r.instancia.ejercicio.numero, r.instancia.numero, r.entregador.nombre)
+        flash(_(u'El %s fue eliminado permanentemente.') % name)
+        return r.archivos
+

     @expose("json")
     def instancias(self, ejercicio_id):
         c = Ejercicio.get(ejercicio_id)
     @expose("json")
     def instancias(self, ejercicio_id):
         c = Ejercicio.get(ejercicio_id)