Se arregla el bug que había quedado pendiente que no permitía hacer altas

de sistemas.

diff --git a/TODO b/TODO
index 9b20dd7df4dc0239f5cbcdfe41d6225f03599698..daf478fbf6bcb6644f229b60cf1b513c4b4b0762 100644 (file)
--- 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 f0e907dc3af6043ad3ea658232309f7c5e2af53e..dc0462211b23a906284d7616a7036c90caa37ab2 100644 (file)
--- 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 f17e12d2222d36547250b1e636c65d4f95226f1b..f83c623d9f1b94489dd8d813a4dea2d0914985c7 100644 (file)
--- 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');
         }