From: Leandro Lucarella <llucax@gmail.com>
Date: Fri, 6 May 2005 18:40:45 +0000 (+0000)
Subject: Se agregan nuevos métodos estáticos a Usuario: getSocios() y getAsesores() para
X-Git-Tag: svn_import~51
X-Git-Url: https://git.llucax.com/z.facultad/75.43/tp1.git/commitdiff_plain/237c40dc072e4a14ca3ce9c53d0de95c3370eba5?ds=sidebyside;hp=a273886fc36b8c938dddba092794f2c12dd862ac

Se agregan nuevos métodos estáticos a Usuario: getSocios() y getAsesores() para
obtener los usuarios que sean de un tipo particular. Además se agrega un
parámetro extra $admins para indicar si se quieren incluir usuarios
administradores.
Se usan los nuevos métodos para obtener listas de usuarios con más sentido a la
hora de agregar créditos (sólo se muestran Socios) y de cambiar el administrador
(se muestran usuarios que no sean administradores).
---

diff --git a/src/admin.admin.php b/src/admin.admin.php
index 40631f8..b8c932b 100644
--- a/src/admin.admin.php
+++ b/src/admin.admin.php
@@ -25,7 +25,7 @@ if (isset($_POST['admin_ceder']) and isset($_POST['admin_ceder_pass']))
     {
         marco_cabecera('Ceder Administración', true);
         error($err);
-        admin_form_ceder_admin(Usuario::getAll(), $_POST['admin_ceder']);
+        admin_form_ceder_admin(Usuario::getAll(false), $_POST['admin_ceder']);
     }
     else // ok
     {
@@ -39,7 +39,7 @@ if (isset($_POST['admin_ceder']) and isset($_POST['admin_ceder_pass']))
 else // Si no llenó el formulario
 {
     marco_cabecera('Ceder Administración', true);
-    admin_form_ceder_admin(Usuario::getAll());
+    admin_form_ceder_admin(Usuario::getAll(false));
 }
 
 marco_pie('Leandro Lucarella', 'llucare@fi.uba.ar');
diff --git a/src/admin.creditos.php b/src/admin.creditos.php
index c04af01..f98c434 100644
--- a/src/admin.creditos.php
+++ b/src/admin.creditos.php
@@ -28,17 +28,17 @@ if (isset($_POST['admin_cred']) and isset($_POST['admin_cred_user'])
         $u->addCreditos($_POST['admin_cred']);
         ok('Se agregaron '.$_POST['admin_cred'].' créditos al usuario '
             .$u->getNombre().' '.$u->getApellido());
-        admin_form_creditos(Usuario::getAll(), $u->getId(),$_POST['admin_cred']);
+        admin_form_creditos(Usuario::getSocios(), $u->getId(),$_POST['admin_cred']);
     }
     else
     {
-        admin_form_creditos(Usuario::getAll(), $_POST['admin_cred_user'],
+        admin_form_creditos(Usuario::getSocios(), $_POST['admin_cred_user'],
             $_POST['admin_cred']);
     }
 }
 else // Si no llenó el formulario
 {
-    admin_form_creditos(Usuario::getAll());
+    admin_form_creditos(Usuario::getSocios());
 }
 
 marco_pie('Leandro Lucarella', 'llucare@fi.uba.ar');
diff --git a/src/lib/Usuario.php b/src/lib/Usuario.php
index bdef593..3de1408 100644
--- a/src/lib/Usuario.php
+++ b/src/lib/Usuario.php
@@ -221,17 +221,57 @@ class Usuario
         return '';
     }
 
+    /**
+     * Obtiene una lista de los usuarios que son socios.
+     * Si $admins es false, no incluye administradores.
+     * @return array con los objetos de usuarios.
+     * @static
+     */
+    function getSocios($admins = true)
+    {
+        $r = array();
+        foreach (file_users_get_all() as $i)
+        {
+            $u = new Usuario($i[0]);
+            if (!$u->esAsesor()
+                    and (!$u->esAdmin() or $u->esAdmin() and $admins))
+                $r[] = $u;
+        }
+        return $r;
+    }
+
+    /**
+     * Obtiene una lista de los usuarios que son asesores.
+     * Si $admins es false, no incluye administradores.
+     * @return array con los objetos de usuarios.
+     * @static
+     */
+    function getAsesores($admins = true)
+    {
+        $r = array();
+        foreach (file_users_get_all() as $i)
+        {
+            $u = new Usuario($i[0]);
+            if ($u->esAsesor()
+                    and (!$u->esAdmin() or $u->esAdmin() and $admins))
+                $r[] = $u;
+        }
+        return $r;
+    }
+
     /**
      * Obtiene una lista de todos los usuarios asociados al sistema.
+     * Si $admins es false, no incluye administradores.
      * @return array con los objetos de usuarios.
      * @static
      */
-    function getAll()
+    function getAll($admins = true)
     {
         $r = array();
-        foreach (file_users_get_all() as $u)
+        foreach (file_users_get_all() as $i)
         {
-            $r[] = new Usuario($u[0]);
+            $u = new Usuario($i[0]);
+            if (!$u->esAdmin() or $u->esAdmin() and $admins) $r[] = $u;
         }
         return $r;
     }