alguna modificacion de algo que no validaba

[z.facultad/75.43/tp1.git] / src / lib / Usuario.php

diff --git a/src/lib/Usuario.php b/src/lib/Usuario.php
index 3daa98e51a3f93d4e7017c9526f4739387171e17..2d051910bd1882eb23519f6cdb7bf0cfc9ca5125 100644 (file)
--- a/src/lib/Usuario.php
+++ b/src/lib/Usuario.php
@@ -136,10 +136,10 @@ class Usuario
     /// Muestra el usuario como una opción de un select HTML.
     function toHtmlOption($selected = null)
     {
-        $cred = $this->esAsesor() ? '' : ' ('.$this->getCreditos().')';
+        $cred = $this->esAsesor() ? '' : (' ('.$this->getCreditos().')');
         $admin =  $this->esAdmin() ? '* ' : '';
         if (!is_null($selected))
-            $selected = ($this->getId() == $user) ? ' selected="selected"' : '';
+            $selected = ($this->getId() == $selected) ? ' selected="selected"' : '';
         return sprintf("<option value=\"%s\"%s>%s%s %s%s</option>\n",
             $this->getId(), $selected, $admin, $this->getNombre(),
             $this->getApellido(), $cred);
@@ -149,9 +149,12 @@ class Usuario
     function login()
     {
         $this->log('Ingreso al sistema');
-        $res = file_creditos_login($this->_id); // Actualizamos créditos
-        if (is_array($res))
-            $this->log("Se restaron créditos ({$res[0]}) por no preguntar.");
+        if (!$this->esAsesor())
+        {
+            $res = file_creditos_login($this->_id); // Actualizamos créditos
+            if (is_array($res))
+                $this->log("Se restaron créditos ($res[0]) por no preguntar.");
+        }
     }
 
     /// Salida del sistema.