From 30b66b3ed7061eaaab51a23cc71e6aff0dbdc650 Mon Sep 17 00:00:00 2001 From: Ricardo Markiewicz Date: Thu, 15 Mar 2007 01:10:18 +0000 Subject: [PATCH 1/1] Muevo Grupo dentro de Curso. - 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 | 2 - sercom/subcontrollers/__init__.py | 1 - sercom/subcontrollers/curso/__init__.py | 2 + .../{ => curso}/grupo/__init__.py | 86 ++++++------------- .../{ => curso}/grupo/templates/__init__.py | 0 .../{ => curso}/grupo/templates/edit.kid | 8 +- .../{ => curso}/grupo/templates/list.kid | 15 ++-- .../{ => curso}/grupo/templates/new.kid | 6 +- .../{ => curso}/grupo/templates/show.kid | 6 +- .../subcontrollers/curso/templates/list.kid | 2 +- 10 files changed, 44 insertions(+), 84 deletions(-) rename sercom/subcontrollers/{ => curso}/grupo/__init__.py (77%) rename sercom/subcontrollers/{ => curso}/grupo/templates/__init__.py (100%) rename sercom/subcontrollers/{ => curso}/grupo/templates/edit.kid (75%) rename sercom/subcontrollers/{ => curso}/grupo/templates/list.kid (58%) rename sercom/subcontrollers/{ => curso}/grupo/templates/new.kid (63%) rename sercom/subcontrollers/{ => curso}/grupo/templates/show.kid (86%) diff --git a/sercom/controllers.py b/sercom/controllers.py index d584af2..b84b7c2 100644 --- a/sercom/controllers.py +++ b/sercom/controllers.py @@ -111,8 +111,6 @@ class Root(controllers.RootController): docente = DocenteController() - grupo = GrupoController() - alumno = AlumnoController() enunciado = EnunciadoController() diff --git a/sercom/subcontrollers/__init__.py b/sercom/subcontrollers/__init__.py index 3e58ec5..2297686 100644 --- a/sercom/subcontrollers/__init__.py +++ b/sercom/subcontrollers/__init__.py @@ -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 diff --git a/sercom/subcontrollers/curso/__init__.py b/sercom/subcontrollers/curso/__init__.py index 62dc1cc..0528def 100644 --- a/sercom/subcontrollers/curso/__init__.py +++ b/sercom/subcontrollers/curso/__init__.py @@ -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): diff --git a/sercom/subcontrollers/grupo/__init__.py b/sercom/subcontrollers/curso/grupo/__init__.py similarity index 77% rename from sercom/subcontrollers/grupo/__init__.py rename to sercom/subcontrollers/curso/grupo/__init__.py index d2dfd7c..8dad4a7 100644 --- a/sercom/subcontrollers/grupo/__init__.py +++ b/sercom/subcontrollers/curso/grupo/__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 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): diff --git a/sercom/subcontrollers/grupo/templates/__init__.py b/sercom/subcontrollers/curso/grupo/templates/__init__.py similarity index 100% rename from sercom/subcontrollers/grupo/templates/__init__.py rename to sercom/subcontrollers/curso/grupo/templates/__init__.py diff --git a/sercom/subcontrollers/grupo/templates/edit.kid b/sercom/subcontrollers/curso/grupo/templates/edit.kid similarity index 75% rename from sercom/subcontrollers/grupo/templates/edit.kid rename to sercom/subcontrollers/curso/grupo/templates/edit.kid index 8dbee1c..52273c0 100644 --- a/sercom/subcontrollers/grupo/templates/edit.kid +++ b/sercom/subcontrollers/curso/grupo/templates/edit.kid @@ -1,6 +1,6 @@ + py:extends="'../../../../templates/master.kid'"> edit @@ -8,7 +8,7 @@