]> git.llucax.com Git - z.facultad/75.52/sercom.git/commitdiff
Muevo Grupo dentro de Curso.
authorRicardo Markiewicz <rmarkie@fi.uba.ar>
Thu, 15 Mar 2007 01:10:18 +0000 (01:10 +0000)
committerRicardo Markiewicz <rmarkie@fi.uba.ar>
Thu, 15 Mar 2007 01:10:18 +0000 (01:10 +0000)
 - Ajusto todo para que ande
 - Arreglo un bug que hacia que se guardara el ID de Docente en lugar del
 ID del DocenteInscripto como Tutor del curso. Para eso uso la magia de
 las closure's :)

sercom/controllers.py
sercom/subcontrollers/__init__.py
sercom/subcontrollers/curso/__init__.py
sercom/subcontrollers/curso/grupo/__init__.py [moved from sercom/subcontrollers/grupo/__init__.py with 77% similarity]
sercom/subcontrollers/curso/grupo/templates/__init__.py [moved from sercom/subcontrollers/grupo/templates/__init__.py with 100% similarity]
sercom/subcontrollers/curso/grupo/templates/edit.kid [moved from sercom/subcontrollers/grupo/templates/edit.kid with 75% similarity]
sercom/subcontrollers/curso/grupo/templates/list.kid [moved from sercom/subcontrollers/grupo/templates/list.kid with 58% similarity]
sercom/subcontrollers/curso/grupo/templates/new.kid [moved from sercom/subcontrollers/grupo/templates/new.kid with 63% similarity]
sercom/subcontrollers/curso/grupo/templates/show.kid [moved from sercom/subcontrollers/grupo/templates/show.kid with 86% similarity]
sercom/subcontrollers/curso/templates/list.kid

index d584af28819d58444cef7f0d2d4ba8df221a79a3..b84b7c209d78b563f9210d372546ee4649b20c21 100644 (file)
@@ -111,8 +111,6 @@ class Root(controllers.RootController):
 
     docente = DocenteController()
 
-    grupo = GrupoController()
-
     alumno = AlumnoController()
 
     enunciado = EnunciadoController()
index 3e58ec59a175e1e48e02956352ed2f566b6b7bab..22976868899db81400874d28b8784d83ea10154b 100644 (file)
@@ -4,7 +4,6 @@ from curso import CursoController
 from enunciado import EnunciadoController
 from ejercicio import EjercicioController
 from docente_inscripto import DocenteInscriptoController
-from grupo import GrupoController
 from correccion import CorreccionController
 from misentregas import MisEntregasController
 from grupo_admin import GrupoAdminController
index 62dc1cc6814c44a52ba1c6774267d3f2694b5fb3..0528def2609a34c60f8702eb7139d7f2db40549d 100644 (file)
@@ -16,6 +16,7 @@ from sqlobject import *
 from sqlobject.dberrors import *
 from sercom.widgets import *
 from alumno import AlumnoInscriptoController
+from grupo import GrupoController
 #}}}
 
 #{{{ Configuración
@@ -170,6 +171,7 @@ class CursoController(controllers.Controller, identity.SecureResource):
     require = identity.has_permission('admin')
     curso_alumno = CursoAlumnoController()
     alumno = AlumnoInscriptoController()
+    grupo = GrupoController()
 
     @expose()
     def default(self, tg_errors=None):
similarity index 77%
rename from sercom/subcontrollers/grupo/__init__.py
rename to sercom/subcontrollers/curso/grupo/__init__.py
index d2dfd7c88dceec28743f4c745f0e5f6c974ea552..8dad4a7500eefd14b4021a2659dadeefc8491bef 100644 (file)
@@ -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 sercom.model import Curso, AlumnoInscripto, Docente, Grupo, Alumno, Miembro
+from sercom.model import Curso, AlumnoInscripto, Docente, DocenteInscripto, Grupo, Alumno, Miembro
 from sqlobject import *
 
 from sercom.widgets import *
@@ -21,36 +21,16 @@ from sercom.widgets import *
 cls = Grupo
 name = 'grupo'
 namepl = 'grupos'
-
-fkcls = Curso
-fkname = 'curso'
-fknamepl = fkname + 's'
 #}}}
 
 #{{{ Validación
-def validate_fk(data):
-    fk = data.get(fkname + 'ID', None)
-    if fk == 0: fk = None
-    if fk is not None:
-        try:
-            fk = fkcls.get(fk)
-        except LookupError:
-            flash(_(u'No se pudo crear el nuevo %s porque el %s con '
-                'identificador %d no existe.' % (name, fkname, fk)))
-            raise redirect('new', **data)
-    data.pop(fkname + 'ID', None)
-    data[fkname] = fk
-    return fk
-
 def validate_get(id):
     return val.validate_get(cls, name, id)
 
 def validate_set(id, data):
-    validate_fk(data)
     return val.validate_set(cls, name, id, data)
 
 def validate_new(data):
-    validate_fk(data)
     return val.validate_new(cls, name, data)
 
 def validate_del(id):
@@ -61,8 +41,10 @@ def validate_del(id):
 def get_docentes():
     return [(fk1.id, fk1.shortrepr()) for fk1 in Docente.select()]
 
-def get_cursos():
-    return [(0, u'---')] + [(fk1.id, fk1.shortrepr()) for fk1 in Curso.select()]
+def get_docentes_inscriptos(id):
+    def func():
+        return [(fk1.id, fk1.shortrepr()) for fk1 in DocenteInscripto.select(DocenteInscripto.q.cursoID==id)]
+    return func
 
 ajax = u"""
     function alumnos_agregar_a_la_lista(texto, lista)
@@ -74,11 +56,11 @@ ajax = u"""
             alert("No deberias ver esto, y quiere decir que tu form esta roto.\\nTe falta un combo de curso");
             return;
         }
-        if (curso.options[curso.selectedIndex].value <= 0) {
+        if (curso.value <= 0) {
             alert('Debes seleccionar un curso primero');
             return;
         }
-        url = "/grupo/get_inscripto?cursoid="+curso.options[curso.selectedIndex].value+"&padron="+t.value;
+        url = "/curso/grupo/get_inscripto?cursoid="+curso.value+"&padron="+t.value;
         t.value = "";
         return url;
     }
@@ -107,12 +89,12 @@ ajax = u"""
         }
         /* Obtengo el curso */
         l = MochiKit.DOM.getElement('form_cursoID');
-        cursoid = l.options[l.selectedIndex].value;
+        cursoid = l.value;
         if (cursoid <= 0) {
             alert('Debe seleccionar un curso');
             return;
         }
-        url = "/grupo/get_inscripto?cursoid="+cursoid+'&padron='+padron;
+        url = "/curso/grupo/get_inscripto?cursoid="+cursoid+'&padron='+padron;
         var d = loadJSONDoc(url);
         d.addCallbacks(procesar, err);
     }
@@ -137,30 +119,19 @@ ajax = u"""
     MochiKit.DOM.addLoadEvent(prepare)
 
 """
-def get_docentes():
-    return [(fk1.id, fk1.shortrepr()) for fk1 in Docente.select()]
 
 class GrupoForm(W.TableForm):
     class Fields(W.WidgetsList):
-        cursoID = W.SingleSelectField(label=_(u'Curso'), options = get_cursos, validator = V.Int(not_empty=True))
+        cursoID = W.HiddenField()
         nombre = W.TextField(label=_(u'Nombre'), validator=V.UnicodeString(not_empty=True,strip=True))
         responsable = CustomTextField(label=_(u'Responsable'), validator=V.UnicodeString(), attrs=dict(size='8'))
         miembros = AjaxMultiSelect(label=_(u'Miembros'), validator=V.Int(), on_add="alumnos_agregar_a_la_lista")
         tutores = W.MultipleSelectField(label=_(u'Tutores'), validator=V.Int(), options=get_docentes)
 
     fields = Fields()
-    javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('curso');"), W.JSSource(ajax)]
+    javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('form_nombre');"), W.JSSource(ajax)]
     form_attrs = dict(onsubmit='return doSubmit()')
 
-def get_cursos():
-    return [(0, u'---')] + [(fk1.id, fk1.shortrepr()) for fk1 in Curso.select()]
-
-class GrupoFiltros(W.TableForm):
-    class Fields(W.WidgetsList):
-        cursoID = W.SingleSelectField(label=_(u'Curso'), options = get_cursos, validator = V.Int(not_empty=True))
-    fields = Fields()
-
-filtro = GrupoFiltros()
 form = GrupoForm()
 
 #}}}
@@ -173,33 +144,25 @@ class GrupoController(controllers.Controller, identity.SecureResource):
     @expose()
     def default(self, tg_errors=None):
         """handle non exist urls"""
-        raise redirect('list')
+        raise redirect(tg.url('/curso/list'))
 
     @expose()
     def index(self):
-        raise redirect('list')
+        raise redirect(tg.url('/curso/list'))
 
     @expose(template='kid:%s.templates.list' % __name__)
     @paginate('records')
-    def list(self, cursoID = 0):
+    def list(self, cursoID):
         """List records in model"""
-        vfilter = dict(cursoID = cursoID)
-        if int(cursoID) == 0:
-            r = cls.select()
-        else:
-            r = cls.select(cls.q.cursoID == cursoID)
-        return dict(records=r, name=name, namepl=namepl, form=filtro, vfilter=vfilter)
-
-    @expose()
-    def activate(self, id, activo):
-        """Save or create record to model"""
-        r = validate_get(id)
-        raise redirect('../../list')
+        r = cls.select(cls.q.cursoID == cursoID)
+        return dict(records=r, name=name, namepl=namepl, cursoID=int(cursoID))
 
     @expose(template='kid:%s.templates.new' % __name__)
-    def new(self, **kw):
+    def new(self, cursoID, **kw):
         """Create new records in model"""
-        return dict(name=name, namepl=namepl, form=form, values=kw)
+        form.fields[0].attrs['value'] = cursoID
+        form.fields[4].options = get_docentes_inscriptos(cursoID)
+        return dict(name=name, namepl=namepl, cursoID=int(cursoID), form=form, values=kw)
 
     @validate(form=form)
     @error_handler(new)
@@ -217,7 +180,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
 
         r = validate_new(kw)
         flash(_(u'Se creó un nuevo %s.') % name)
-        raise redirect('list')
+        raise redirect('list/%d' % int(kw['cursoID']))
 
     @expose(template='kid:%s.templates.edit' % __name__)
     def edit(self, id, **kw):
@@ -227,6 +190,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
         # de manera comoda y facil de formatear segun lo que espera la UI (que
         # en este caso es super particular). Ese EmptyClass no se si hay algo estandar
         # en python que usar, puse {} y [] pero cuando quiero hacer values.id = XX explota.
+        form.fields[4].options = get_docentes_inscriptos(r.curso.id)
         class EmptyClass:
             pass
         values = EmptyClass()
@@ -257,7 +221,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
         kw['responsable'] = resp
         r = validate_set(id, kw)
         flash(_(u'El %s fue actualizado.') % name)
-        raise redirect('../list')
+        raise redirect('../list/%d' % r.curso.id)
 
     @expose(template='kid:%s.templates.show' % __name__)
     def show(self,id, **kw):
@@ -266,11 +230,11 @@ class GrupoController(controllers.Controller, identity.SecureResource):
         return dict(name=name, namepl=namepl, record=r)
 
     @expose()
-    def delete(self, id):
+    def delete(self, cursoID, id):
         """Destroy record in model"""
         validate_del(id)
         flash(_(u'El %s fue eliminado permanentemente.') % name)
-        raise redirect('../list')
+        raise redirect('../../list/%d' % int(cursoID))
 
     @expose('json')
     def get_inscripto(self, cursoid, padron):
similarity index 75%
rename from sercom/subcontrollers/grupo/templates/edit.kid
rename to sercom/subcontrollers/curso/grupo/templates/edit.kid
index 8dbee1c6f9c5d4599b367860b991b64df508ba61..52273c083d9d463431bc045e1bcdc9b1529c5138 100644 (file)
@@ -1,6 +1,6 @@
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xmlns:py="http://purl.org/kid/ns#"
-    py:extends="'../../../templates/master.kid'">
+    py:extends="'../../../../templates/master.kid'">
 <head>
 <meta content="text/html; charset=utf-8" http-equiv="Content-Type" py:replace="''"/>
 <title>edit</title>
@@ -8,7 +8,7 @@
 <script type="text/javascript">
        function init_data() {
                MochiKit.DOM.getElement('form_responsable').focus();
-               MochiKit.DOM.getElement('form_cursoID').focus();
+               MochiKit.DOM.getElement('form_nombre').focus();
                <span py:for="i in record.miembros" py:strip="True">
                        MochiKit.DOM.appendChildNodes("form_miembros", OPTION({"value":${i['id']}}, '${i['label']}'))
                </span>
 <body>
 <h1>Modificación de <span py:replace="name">Objeto</span></h1>
 
-<div py:replace="form(value=record, action=tg.url('/grupo/update/%d' % record.id),
+<div py:replace="form(value=record, action=tg.url('/curso/grupo/update/%d' % record.id),
        submit_text=_(u'Guardar'))">Formulario</div>
 
 <br/>
-<a href="${tg.url('/grupo/list')}">Volver (cancela)</a>
+<a href="${tg.url('/curso/grupo/list')}">Volver (cancela)</a>
 
 </body>
 </html>
similarity index 58%
rename from sercom/subcontrollers/grupo/templates/list.kid
rename to sercom/subcontrollers/curso/grupo/templates/list.kid
index c166fe9a8b7c15721eee85f32443c381ee798391..31be7009e397ede5e8c6f42e1baa65949a9ddc4b 100644 (file)
@@ -1,6 +1,6 @@
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xmlns:py="http://purl.org/kid/ns#"
-    py:extends="'../../../templates/master.kid'">
+    py:extends="'../../../../templates/master.kid'">
 <head>
 <meta content="text/html; charset=utf-8" http-equiv="Content-Type" py:replace="''"/>
 <title>list</title>
@@ -9,29 +9,26 @@
 
 <h1>Administración de <span py:replace="namepl">Objetos</span></h1>
 
-<div py:replace="form(value=vfilter, action=tg.url('/grupo/list'),     submit_text=_(u'Filtrar'))">Filtros</div>
-
 <table class="list">
     <tr>
-        <th>Curso</th>
         <th>Nombre</th>
         <th>Responsable</th>
         <th>Operaciones</th>
     </tr>
     <tr py:for="record in records">
-            <td><a href="${tg.url('/curso/show/'+str(record.curso.id))}" py:content="record.curso.shortrepr()">curso</a></td>
         <td><span py:replace="record.nombre">nombre</span></td>
                 <td><a py:if="record.responsable is not None" href="${tg.url('/alumno/show/'+str(record.responsable.alumno.id))}" py:content="record.responsable.alumno.shortrepr()"></a></td>
                 <td>
-                    <a href="${tg.url('/grupo/show/%d' % record.id)}">Ver</a>
-                    <a href="${tg.url('/grupo/edit/%d' % record.id)}">Editar</a>
-                    <a href="${tg.url('/grupo/delete/%d' % record.id)}" onclick="if (confirm('${_(u'Estás seguro? Tal vez sólo quieras desactivarlo mejor...')}')) { var f = document.createElement('form'); this.parentNode.appendChild(f); f.method = 'POST'; f.action = this.href; f.submit(); };return false;">Eliminar</a>
+                    <a href="${tg.url('/curso/grupo/show/%d' % record.id)}">Ver</a>
+                    <a href="${tg.url('/curso/grupo/edit/%d' % record.id)}">Editar</a>
+                    <a href="${tg.url('/curso/grupo/delete/%d/%d' % (record.curso.id, record.id))}" onclick="if (confirm('${_(u'Estás seguro? Tal vez sólo quieras desactivarlo mejor...')}')) { var f = document.createElement('form'); this.parentNode.appendChild(f); f.method = 'POST'; f.action = this.href; f.submit(); };return false;">Eliminar</a>
                 </td>
     </tr>
 </table>
 
 <br/>
-<a href="${tg.url('/grupo/new')}">Agregar</a>
+<a href="${tg.url('/curso/grupo/new/%d' % cursoID)}">Agregar</a> |
+<a href="${tg.url('/curso/list')}">Volver a Cursos</a>
 <br/>
 <br/>
 <div py:for="page in tg.paginate.pages">
similarity index 63%
rename from sercom/subcontrollers/grupo/templates/new.kid
rename to sercom/subcontrollers/curso/grupo/templates/new.kid
index 359422d32e9ea0e558a5664bceb7af483717d7df..d5fdd20a3fedd082e79fb5b9be22239813418f30 100644 (file)
@@ -1,6 +1,6 @@
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xmlns:py="http://purl.org/kid/ns#"
-    py:extends="'../../../templates/master.kid'">
+    py:extends="'../../../../templates/master.kid'">
 <head>
 <meta content="text/html; charset=utf-8" http-equiv="Content-Type" py:replace="''"/>
 <title>new</title>
@@ -9,10 +9,10 @@
 
 <h1>Crear Nuevo <span py:replace="name">Objeto</span></h1>
 
-<p py:replace="form(action=tg.url('/grupo/create'), value=values, submit_text=_('Crear'))">Formulario</p>
+<p py:replace="form(action=tg.url('/curso/grupo/create'), value=values, submit_text=_('Crear'))">Formulario</p>
 
 <br/>
-<a href="${tg.url('/grupo/list')}">Cancelar</a>
+<a href="${tg.url('/curso/grupo/list/%d' % cursoID)}">Cancelar</a>
 
 </body>
 </html>
similarity index 86%
rename from sercom/subcontrollers/grupo/templates/show.kid
rename to sercom/subcontrollers/curso/grupo/templates/show.kid
index 3d593d116a6698918b9ddba9125b72d989721301..8bc880e978cd407922b19e7b7c61d2fe0418f054 100644 (file)
@@ -1,6 +1,6 @@
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xmlns:py="http://purl.org/kid/ns#"
-    py:extends="'../../../templates/master.kid'">
+    py:extends="'../../../../templates/master.kid'">
 <head>
 <meta content="text/html; charset=utf-8" http-equiv="Content-Type" py:replace="''"/>
 <title>show</title>
@@ -41,8 +41,8 @@
 </table>
 
 <br/>
-<a href="${tg.url('/grupo/edit/%d' % record.id)}">Editar</a> |
-<a href="${tg.url('/grupo/list')}">Volver</a>
+<a href="${tg.url('/curso/grupo/edit/%d' % record.id)}">Editar</a> |
+<a href="${tg.url('/curso/grupo/list/%d' % record.curso.id)}">Volver</a>
 
 </body>
 </html>
index 005d41d2cbfc6619f97a69455802bfa644df6c67..13eeb1ccc1b3c87aaec0d4635a0431a20a56d14e 100644 (file)
@@ -33,7 +33,7 @@
         <td><span py:replace="record.descripcion">descripcion</span></td>
         <td><a py:if="len(record.docentes)" href="${tg.url('/docente/list')}"><span py:replace="len(record.docentes)">Docentes</span></a></td>
         <td><a py:if="len(record.alumnos)"  href="${tg.url('/curso/alumno/list/%d' % record.id)}"><span py:replace="len(record.alumnos)">Alumnos</span></a></td>
-        <td><a py:if="len(record.grupos)"  href="${tg.url('/grupo/list/%d' % record.id)}"><span py:replace="len(record.grupos)">Grupos</span></a></td>
+        <td align="center"><a href="${tg.url('/curso/grupo/list/%d' % record.id)}"><span py:replace="len(record.grupos)">Grupos</span></a></td>
         <td><a py:if="len(record.ejercicios)" href="${tg.url('/ejercicio/list')}"><span py:replace="len(record.ejercicios)">Ejercicio</span></a></td>
         <td><a href="${tg.url('/curso/show/%d' % record.id)}">Ver</a>
             <a href="${tg.url('/curso/edit/%d' % record.id)}">Editar</a>