Bugfix: El zip se subía roto en las entregas.

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

diff --git a/sercom/subcontrollers/ejercicio/__init__.py b/sercom/subcontrollers/ejercicio/__init__.py
index 8a8a7ab1d4fa8f17a687cccfeb691dd43fdc74fa..58a36b214671ffaee610bbb8bdac9f4d4947e538 100644 (file)
--- a/sercom/subcontrollers/ejercicio/__init__.py
+++ b/sercom/subcontrollers/ejercicio/__init__.py
@@ -71,6 +71,9 @@ def validate_new(data):
     validate_fk(data)
     validate_fk1(data)
     return val.validate_new(cls, name, data)
     validate_fk(data)
     validate_fk1(data)
     return val.validate_new(cls, name, data)

+
+def validate_del(id):
+    return val.validate_del(cls, name, id)

 #}}}
 
 #{{{ Formulario
 #}}}
 
 #{{{ Formulario


@@ -79,10 +82,21 @@ def get_options():
 
 # Un poco de ajax para llenar los cursos
 ajax = """
 
 # Un poco de ajax para llenar los cursos
 ajax = """

+    function showHint()
+    {
+        MochiKit.DOM.showElement('hint')
+    }
+
+    function hideHint()
+    {
+        MochiKit.DOM.hideElement('hint')
+    }
+

     function clearEnunciados ()
     {
         l = MochiKit.DOM.getElement('form_enunciadoID');
         l.options.length = 0;
     function clearEnunciados ()
     {
         l = MochiKit.DOM.getElement('form_enunciadoID');
         l.options.length = 0;

+        l.disabled = true;

     }
 
     function mostrarEnunciados (res)
     }
 
     function mostrarEnunciados (res)


@@ -93,11 +107,14 @@ ajax = """
             label = res.enunciados[i].nombre;
             MochiKit.DOM.appendChildNodes("form_enunciadoID", OPTION({"value":id}, label))
         }
             label = res.enunciados[i].nombre;
             MochiKit.DOM.appendChildNodes("form_enunciadoID", OPTION({"value":id}, label))
         }

+        l.disabled = false;
+        hideHint();

     }
 
     function err (err)
     {
         alert("The metadata for MochiKit.Async could not be fetched :(");
     }
 
     function err (err)
     {
         alert("The metadata for MochiKit.Async could not be fetched :(");

+        hideHint();

     }
 
     function actualizar_enunciados ()
     }
 
     function actualizar_enunciados ()


@@ -112,11 +129,18 @@ ajax = """
         url = "/enunciado/de_curso?curso_id="+id;
         var d = loadJSONDoc(url);
         d.addCallbacks(mostrarEnunciados, err);
         url = "/enunciado/de_curso?curso_id="+id;
         var d = loadJSONDoc(url);
         d.addCallbacks(mostrarEnunciados, err);

+        showHint();

     }
 
     function prepare()
     {
         connect('form_cursoID', 'onchange', actualizar_enunciados);
     }
 
     function prepare()
     {
         connect('form_cursoID', 'onchange', actualizar_enunciados);

+        hideHint();
+        clearEnunciados();
+        actualizar_enunciados();
+        if (select_enunciado) {
+            wait(0.1).addCallback(function (res) { return select_enunciado() });
+        }

     }
 
     MochiKit.DOM.addLoadEvent(prepare)
     }
 
     MochiKit.DOM.addLoadEvent(prepare)


@@ -141,7 +165,7 @@ form = EjercicioForm()
 #{{{ Controlador
 class EjercicioController(controllers.Controller, identity.SecureResource):
     """Basic model admin interface"""
 #{{{ Controlador
 class EjercicioController(controllers.Controller, identity.SecureResource):
     """Basic model admin interface"""

-    require = identity.has_permission('admin')
+    require = identity.has_permission('entregar')

 
     entrega = EntregaController()
 
 
     entrega = EntregaController()
 


@@ -163,12 +187,14 @@ class EjercicioController(controllers.Controller, identity.SecureResource):
         return dict(records=r, name=name, namepl=namepl, parcial=autor)
 
     @expose(template='kid:%s.templates.new' % __name__)
         return dict(records=r, name=name, namepl=namepl, parcial=autor)
 
     @expose(template='kid:%s.templates.new' % __name__)

+    @identity.require(identity.has_permission('admin'))

     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)
     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)

+    @identity.require(identity.has_permission('admin'))

     @expose()
     def create(self, **kw):
         """Save or create record to model"""
     @expose()
     def create(self, **kw):
         """Save or create record to model"""


@@ -177,6 +203,7 @@ class EjercicioController(controllers.Controller, identity.SecureResource):
         raise redirect('list')
 
     @expose(template='kid:%s.templates.edit' % __name__)
         raise redirect('list')
 
     @expose(template='kid:%s.templates.edit' % __name__)

+    @identity.require(identity.has_permission('admin'))

     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)
     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)


@@ -185,6 +212,7 @@ class EjercicioController(controllers.Controller, identity.SecureResource):
     @validate(form=form)
     @error_handler(edit)
     @expose()
     @validate(form=form)
     @error_handler(edit)
     @expose()

+    @identity.require(identity.has_permission('admin'))

     def update(self, id, **kw):
         """Save or create record to model"""
         r = validate_set(id, kw)
     def update(self, id, **kw):
         """Save or create record to model"""
         r = validate_set(id, kw)


@@ -198,10 +226,10 @@ class EjercicioController(controllers.Controller, identity.SecureResource):
         return dict(name=name, namepl=namepl, record=r)
 
     @expose()
         return dict(name=name, namepl=namepl, record=r)
 
     @expose()

+    @identity.require(identity.has_permission('admin'))

     def delete(self, id):
         """Destroy record in model"""
     def delete(self, id):
         """Destroy record in model"""

-        r = validate_get(id)
-        r.destroySelf()
+        validate_del(id)

         flash(_(u'El %s fue eliminado permanentemente.') % name)
         raise redirect('../list')
 
         flash(_(u'El %s fue eliminado permanentemente.') % name)
         raise redirect('../list')