fix shortrepr

[software/sercom.git] / sercom / model.py

diff --git a/sercom/model.py b/sercom/model.py
index 01b908e1531450a49bde6cadaaff88e0ba81cddd..e91c056522566e77763336f958ab4ad0cdde117c 100644 (file)
--- a/sercom/model.py
+++ b/sercom/model.py
@@ -161,7 +161,7 @@ class Curso(SQLObject, ByObject): #{{{
 
     def shortrepr(self):
         return '%s.%s.%s' \
-            % (self.anio, self.cuatrimestre, self.numero, self.descripcion)
+            % (self.anio, self.cuatrimestre, str(self.numero))
 #}}}
 
 class Usuario(InheritableSQLObject, ByObject): #{{{
@@ -442,8 +442,9 @@ class Ejercicio(SQLObject, ByObject): #{{{
 
     def __init__(self, curso=None, numero=None, enunciado=None, grupal=False,
             **kargs):
-        SQLObject.__init__(self, cursoID=curso.id, numero=numero,
-            enunciadoID=enunciado.id, grupal=grupal, **kargs)
+        if curso and enunciado:
+            SQLObject.__init__(self, cursoID=curso.id, numero=numero,
+                enunciadoID=enunciado.id, grupal=grupal, **kargs)
 
     def add_instancia(self, numero, inicio, fin, *args, **kargs):
         return InstanciaDeEntrega(self, numero, inicio, fin, *args, **kargs)
@@ -456,7 +457,7 @@ class Ejercicio(SQLObject, ByObject): #{{{
 
     def shortrepr(self):
         return '(%s, %s, %s)' \
-            % (self.curso.shortrepr(), self.nombre, \
+            % (self.curso.shortrepr(), str(self.numero), \
                 self.enunciado.shortrepr())
 #}}}
 
@@ -477,9 +478,10 @@ class InstanciaDeEntrega(SQLObject, ByObject): #{{{
 
     def __init__(self, ejercicio=None, numero=None, inicio=None, fin=None,
             observaciones=None, activo=True, tareas=(), **kargs):
-        SQLObject.__init__(self, ejercicioID=ejercicio.id, numero=numero,
-            fin=fin, inicio=inicio, observaciones=observaciones, activo=activo,
-            **kargs)
+        if ejercicio:
+            SQLObject.__init__(self, ejercicioID=ejercicio.id, numero=numero,
+                fin=fin, inicio=inicio, observaciones=observaciones, activo=activo,
+                **kargs)
         if tareas:
             self.tareas = tareas