X-Git-Url: https://git.llucax.com/software/sercom.git/blobdiff_plain/9fdaef310788cfc8ba5893424bccbb7287f7003c..bc805a3f2564d28b78c8cff1ea6b8580a6231ea6:/sercom/model.py diff --git a/sercom/model.py b/sercom/model.py index 3442a19..7cb1880 100644 --- a/sercom/model.py +++ b/sercom/model.py @@ -155,7 +155,8 @@ class Curso(SQLObject): #{{{ class Usuario(InheritableSQLObject): #{{{ # Clave (para docentes puede ser un nombre de usuario arbitrario) - usuario = UnicodeCol(length=10, alternateID=True) + usuario = UnicodeCol(length=10, alternateID=True, + alternateMethodName='by_usuario') # Campos contrasenia = UnicodeCol(length=255, default=None) nombre = UnicodeCol(length=255, notNone=True) @@ -187,6 +188,10 @@ class Usuario(InheritableSQLObject): #{{{ def _get_user_name(self): # para identity return self.usuario + @classmethod + def byUsuario(cls, usuario): # TODO eliminar, backward compat + return cls.by_usuario(usuario) + @classmethod def by_user_name(cls, user_name): # para identity user = cls.byUsuario(user_name) @@ -266,9 +271,13 @@ class Alumno(Usuario): #{{{ self.usuario = padron @classmethod - def byPadron(cls, padron): + def byPadron(cls, padron): # TODO eliminar, backward compat return cls.byUsuario(unicode(padron)) + @classmethod + def by_padron(cls, padron): + return cls.by_usuario(unicode(padron)) + def __repr__(self): return 'Alumno(id=%s, padron=%s, nombre=%s, password=%s, email=%s, ' \ 'telefono=%s, activo=%s, creado=%s, observaciones=%s)' \ @@ -430,9 +439,7 @@ class Enunciado(SQLObject): #{{{ descripcion = UnicodeCol(length=255, default=None) autor = ForeignKey('Docente', cascade='null') creado = DateTimeCol(notNone=True, default=DateTimeCol.now) - archivo = BLOBCol(default=None) - archivo_name = UnicodeCol(length=255, default=None) - archivo_type = UnicodeCol(length=255, default=None) + archivos = BLOBCol(default=None) # Joins ejercicios = MultipleJoin('Ejercicio') casos_de_prueba = MultipleJoin('CasoDePrueba') @@ -634,7 +641,16 @@ class Grupo(Entregador): #{{{ def add_miembro(self, alumno, **kw): if isinstance(alumno, AlumnoInscripto): alumno = alumno.id - return Miembro(grupo=self, alumnoID=alumno, **kw) + # FIXME acá habría que sacarle la unicidad a Miembro.pk para que + # un alumno pueda ser miembro varias veces del mismo grupo, de + # manera de tener la historia completa, pero hay que tener cuidad + # y arreglar todos los lugares donde se asume esa unicidad + try: + m = Miembro.selectBy(grupo=self, alumnoID=alumno).getOne() + m.baja = None # si ya existía, le sacamos la fecha de baja + return m + except SQLObjectNotFound: # creo uno nuevo + return Miembro(grupo=self, alumnoID=alumno, **kw) def remove_miembro(self, alumno): if isinstance(alumno, AlumnoInscripto): @@ -645,7 +661,13 @@ class Grupo(Entregador): #{{{ def add_tutor(self, docente, **kw): if isinstance(docente, DocenteInscripto): docente = docente.id - return Tutor(grupo=self, docenteID=docente, **kw) + # FIXME ídem add_miembro() + try: + t = Tutor.selectBy(grupo=self, docenteID=docente).getOne() + t.baja = None # si ya existía, le sacamos la fecha de baja + return t + except SQLObjectNotFound: # creo uno nuevo + return Tutor(grupo=self, docenteID=docente, **kw) def remove_tutor(self, docente): if isinstance(docente, DocenteInscripto):