From: Leandro Lucarella <llucax@gmail.com>
Date: Mon, 28 Jul 2003 16:06:37 +0000 (+0000)
Subject: Se arregla el bug que había quedado pendiente que no permitía hacer altas
X-Git-Tag: svn_import~35
X-Git-Url: https://git.llucax.com/mecon/ai.git/commitdiff_plain/69a744cc170e8e853ec7946e9b61e7f35f0b2ae7?ds=sidebyside;hp=bf3397e09f20d6ee9ea3bfeec51e56f24118aae9

Se arregla el bug que había quedado pendiente que no permitía hacer altas
de sistemas.
---

diff --git a/TODO b/TODO
index 9b20dd7..daf478f 100644
--- a/TODO
+++ b/TODO
@@ -6,7 +6,3 @@ $Id$
 - Resolver tema de JOIN con SAMURAI.
 - Hacer subclase de ArbolDB al estilo de QuickForm.
 
---------------------
-URGENTE:
-- Ver el tema de AI_Sistema::getSistemasArray() y AI_Form para que ande de
-  nuevo la alta.
diff --git a/lib/AI/Sistema.php b/lib/AI/Sistema.php
index f0e907d..dc04622 100644
--- a/lib/AI/Sistema.php
+++ b/lib/AI/Sistema.php
@@ -200,7 +200,7 @@ class AI_Sistema extends AI_DBObject {
         extract($conf);
         // FIXME - ver como manejar JOINs - Ver de hacer el JOIN tambien en cargar.
         $query = "
-            SELECT    SA.nombre_sistema, SA.id_sistema
+            SELECT    SA.id_sistema, SA.nombre_sistema
             FROM      samurai.sistema as SA
             LEFT JOIN $base.$tabla as AI
             ON        AI.sistema = SA.id_sistema
@@ -209,7 +209,7 @@ class AI_Sistema extends AI_DBObject {
             $query .= " AND $where";
         }
         $query .= " ORDER BY SA.nombre_sistema ASC";
-        return $db->getCol($query);
+        return $db->getAssoc($query);
     }
     // -X2C
 
diff --git a/sistema/local_lib/AI/Form.php b/sistema/local_lib/AI/Form.php
index f17e12d..f83c623 100644
--- a/sistema/local_lib/AI/Form.php
+++ b/sistema/local_lib/AI/Form.php
@@ -87,7 +87,13 @@ class AI_Form extends MECON_HTML_QuickForm {
         if ($tipo == 'sistema') {
             require_once 'SAMURAI/Sistema.php';
             $sistemas = array('' => '--');
-            $sistemas += AI_Sistema::getSistemasArray($db, false);
+            if ($accion & AI_ALTA) {
+                // Si es un alta, tomo una lista de sistemas aún no agregados.
+                $sistemas += AI_Sistema::getSistemasArray($db);
+            } else {
+                // Si no, tomo una lista completa de sistemas.
+                $sistemas += SAMURAI_Sistema::getArraySistemas($db);
+            }
             $fId =& $this->addElement('select', $tipo, 'Sistema', $sistemas);
             $this->addRule($tipo, 'Debe ingresar un sistema.', 'required');
         }