Bugfix: fallaba diff por problemas de \n.

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

diff --git a/sercom/subcontrollers/misentregas/__init__.py b/sercom/subcontrollers/misentregas/__init__.py
index 8ea647f016137b4a85121412da59ae23380af16b..d5ec3e4f6eb9342b768a5055679f097a44c79376 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, Grupo, Miembro, AlumnoInscripto
+from sercom.model import ComandoEjecutado, 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"""


@@ -114,11 +116,6 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
     def index(self):
         raise redirect('list')
 
     def index(self):
         raise redirect('list')
 

-    @expose(template='kid:%s.templates.new' % __name__)
-    def new(self, **kw):
-        """Create new records in model"""
-        return dict(name=name, namepl=namepl, form=form, values=kw)
-

     @expose(template='kid:%s.templates.list' % __name__)
     @paginate('records')
     def list(self):
     @expose(template='kid:%s.templates.list' % __name__)
     @paginate('records')
     def list(self):


@@ -133,15 +130,24 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
         r = cls.select(IN(cls.q.entregadorID, m))
         return dict(records=r, name=name, namepl=namepl)
 
         r = cls.select(IN(cls.q.entregadorID, m))
         return dict(records=r, name=name, namepl=namepl)
 

+    @expose(template='kid:%s.templates.new' % __name__)
+    def new(self, **kw):
+        """Create new records in model"""
+        return dict(name=name, namepl=namepl, form=form, values=kw)
+

     @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(archivo.file)
+            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')
 
         # por defecto el entregador es el user loggeado
             raise redirect('list')
 
         # por defecto el entregador es el user loggeado


@@ -169,7 +175,7 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
                 raise redirect('list')
 
             entregador = m.grupo
                 raise redirect('list')
 
             entregador = m.grupo

-        kw['archivos'] = archivo.file.read()
+        kw['archivos'] = archivo

         kw['entregador'] = entregador
         validate_new(kw)
         flash(_(u'Se creó una nueva %s.') % name)
         kw['entregador'] = entregador
         validate_new(kw)
         flash(_(u'Se creó una nueva %s.') % name)


@@ -186,12 +192,31 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
         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)
         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
 
         return r.archivos
 

-    @expose("json")
+    @expose()
+    def file(self, id):
+        from cherrypy import request, response
+        r = ComandoEjecutado.get(id)
+        response.headers["Content-Type"] = "application/zip"
+        response.headers["Content-disposition"] = "attachment;filename=comando_ejecutado_%d.zip" % (r.id)
+        return r.archivos
+
+    @expose()
+    def diff(self, id):
+        from cherrypy import request, response
+        r = ComandoEjecutado.get(id)
+        response.headers["Content-Type"] = "application/zip"
+        response.headers["Content-disposition"] = "attachment;filename=diferencias_%d.zip" % (r.id)
+        return r.diferencias
+
+    @expose('json')

     def instancias(self, ejercicio_id):
     def instancias(self, ejercicio_id):

-        c = Ejercicio.get(ejercicio_id)
-        return dict(instancias=c.instancias)
+        instancias = InstanciaDeEntrega.select(AND(
+                InstanciaDeEntrega.q.ejercicioID == ejercicio_id,
+                InstanciaDeEntrega.q.activo == True,
+                InstanciaDeEntrega.q.inicio <= DateTimeCol.now(),
+                InstanciaDeEntrega.q.fin >= DateTimeCol.now()))
+        return dict(instancias=instancias)

 #}}}
 
 #}}}