Se agrega método toHtmlOption() al Usuario para mostrarlo más fácil y

consistente dentro de los SELECT.

diff --git a/src/lib/Usuario.php b/src/lib/Usuario.php
index bf18d9e698deed5b3676a1046f681a40e0de0bc7..864f2c3a207a3f7d3cb2e341513aeeae83189e49 100644 (file)
--- a/src/lib/Usuario.php
+++ b/src/lib/Usuario.php
@@ -101,13 +101,25 @@ class Usuario
     }
 
     /// Muestra el usuario como HTML (foto + apellido)
-    function toHTML()
+    function toHtml()
     {
         return sprintf('<img src="%s" alt="Foto de %s %s" align="middle" /> %s',
             $this->getFotoFilename(), $this->getNombre(), $this->getApellido(),
             $this->getApellido());
     }
 
+    /// Muestra el usuario como una opción de un select HTML.
+    function toHtmlOption($selected = null)
+    {
+        $cred = $this->esAsesor() ? '' : ' ('.$this->getCreditos().')';
+        $admin =  $this->esAdmin() ? '* ' : '';
+        if (!is_null($selected))
+            $selected = ($this->getId() == $user) ? ' selected="selected"' : '';
+        return sprintf("<option value=\"%s\"%s>%s%s %s%s</option>\n",
+            $this->getId(), $selected, $admin, $this->getNombre(),
+            $this->getApellido(), $cred);
+    }
+
     /// Entrada al sistema.
     function login()
     {

diff --git a/src/lib/admin.forms.php b/src/lib/admin.forms.php
index 5d389954c4001d16e5f7a980db548c742ca20c63..7c0288151dbac9ed7476a3942f5887bbd3a8f7b6 100644 (file)
--- a/src/lib/admin.forms.php
+++ b/src/lib/admin.forms.php
@@ -133,12 +133,7 @@ function admin_form_creditos($users, $user = '', $creditos = 1, $action = 'admin
                 <input type="text" name="admin_cred" size="1" maxlength="1" value="<?php echo $creditos ?>" />
                 créditos a
                 <select name="admin_cred_user">
-                <?php foreach ($users as $u)
-                {
-                    printf("<option value=\"%s\"%s>%s %s (%d)</option>\n",
-                        $u->getId(), ($u->getId() == $user) ? ' selected="selected"' : '',
-                        $u->getNombre(), $u->getApellido(), $u->getCreditos());
-                } ?>
+                    <?php foreach ($users as $u) echo $u->toHtmlOption($user) ?>
                 </select><br />
                 Contraseña: <input type="password" name="admin_cred_pass" /> (esta operación requiere contraseña)
             </p>