Aca esta mi bugfixing, para que le llevemos algo que le guste mas.

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

diff --git a/src/lib/Usuario.php b/src/lib/Usuario.php
index 3de140892c939e94fe2474a3d9acaa646bb75cc4..2d051910bd1882eb23519f6cdb7bf0cfc9ca5125 100644 (file)
--- a/src/lib/Usuario.php
+++ b/src/lib/Usuario.php
@@ -102,7 +102,7 @@ class Usuario
     {
         // El asesor puede preguntar siempre
         if ($this->esAsesor()) return false;
     {
         // El asesor puede preguntar siempre
         if ($this->esAsesor()) return false;

-        file_log_add($this->_id, "El administrador le agrega $n créditos");
+        $this->log("El administrador le agrega $n créditos");

         return file_creditos_add($this->_id, $n);
     }
 
         return file_creditos_add($this->_id, $n);
     }
 


@@ -118,6 +118,10 @@ class Usuario
         if (!file_admins_replace($this->getId(), $user->getId()))
             return 'Error al intercambiar los administradores.';
         $this->reload();
         if (!file_admins_replace($this->getId(), $user->getId()))
             return 'Error al intercambiar los administradores.';
         $this->reload();

+        $this->log('Cede permisos de administración a '
+            .$user->getNombre().' '.$user->getApellido().'.');
+        $user->log('Recibe permisos de administración de '
+            .$this->getNombre().' '.$this->getApellido().'.');

         return '';
     }
 
         return '';
     }
 


@@ -132,10 +136,10 @@ class Usuario
     /// Muestra el usuario como una opción de un select HTML.
     function toHtmlOption($selected = null)
     {
     /// 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))
         $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);
         return sprintf("<option value=\"%s\"%s>%s%s %s%s</option>\n",
             $this->getId(), $selected, $admin, $this->getNombre(),
             $this->getApellido(), $cred);


@@ -144,14 +148,25 @@ class Usuario
     /// Entrada al sistema.
     function login()
     {
     /// Entrada al sistema.
     function login()
     {

-        file_creditos_login($this->_id); // Actualizamos créditos
-        file_log_add($this->_id, 'Ingreso al sistema');
+        $this->log('Ingreso al sistema');
+        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.
     function logout()
     {
     }
 
     /// Salida del sistema.
     function logout()
     {

-        file_log_add($this->_id, 'Salida del sistema');
+        $this->log('Salida del sistema');
+    }
+
+    /// Graba un mensaje de log.
+    function log($msg)
+    {
+        file_log_add($this->_id, $msg);

     }
 
     /**
     }
 
     /**