Se arregla un bug que hacía que sea posible asignar de padre al mismo

[mecon/ai.git] / lib / AI / Form.php

diff --git a/lib/AI/Form.php b/lib/AI/Form.php
index b74bd02ce3fbc530a2a8dbe713d4f63ec714ea24..141e9c4d0b5428a620e2be72a6066c7b899a95a3 100644 (file)
--- a/lib/AI/Form.php
+++ b/lib/AI/Form.php
@@ -96,10 +96,14 @@ class AI_Form extends MECON_HTML_QuickForm {
             $fId->freeze();
         }
         if ($tipo == 'grupo' or $tipo == 'servicio') {
             $fId->freeze();
         }
         if ($tipo == 'grupo' or $tipo == 'servicio') {

-            $tipos = array('' => '--', '0' => 'Página Principal') +
-                arbol2array(AI_DB::connect('../conf/DB.ini'),
-                ($tipo == 'grupo') ? 'grupo_secciones' : $tipo, 0, $tipo, 'nombre',
-                $tipo . '_padre', 'ASC');
+            $tipos = array('' => '--', '0' => 'Página Principal')
+                + arbol2array(AI_DB::connect('../conf/DB.ini'),
+                    ($tipo == 'grupo') ? 'grupo_secciones' : $tipo, 0, $tipo,
+                    'nombre', $tipo . '_padre', 'ASC');
+            // Saco el elemento actual si hay uno cargado (no puede ser padre de si mismo).
+            if ($obj->$tipo) {
+                unset($tipos[$obj->$tipo]);
+            }

             $fPadre  =& $this->addElement('select', $padre, 'Padre', $tipos);
             $fNombre =& $this->addElement('text', 'nombre', 'Nombre');
             // Validación.
             $fPadre  =& $this->addElement('select', $padre, 'Padre', $tipos);
             $fNombre =& $this->addElement('text', 'nombre', 'Nombre');
             // Validación.


@@ -118,7 +122,7 @@ class AI_Form extends MECON_HTML_QuickForm {
                 array(3 => '3 días', 1 => '1 día', 7 => '1 semana'));
             $fSecciones    =& $this->addElement('select', 'secciones', 'Secciones',
                 AI_GrupoSecciones::getSeccionesArray(AI_DB::connect('../conf/DB.ini')),
                 array(3 => '3 días', 1 => '1 día', 7 => '1 semana'));
             $fSecciones    =& $this->addElement('select', 'secciones', 'Secciones',
                 AI_GrupoSecciones::getSeccionesArray(AI_DB::connect('../conf/DB.ini')),

-                array('multiple' => 'multiple', 'size' => 5));
+                array('multiple' => 'multiple', 'size' => 8));

             $fMostrarHijos =& $this->addElement('checkbox', 'mostrar_hijos', 'Mostrar hijos');
             // Validación.
             $this->addRule('antiguedad', 'La antigüedad debe ser un número natural.',
             $fMostrarHijos =& $this->addElement('checkbox', 'mostrar_hijos', 'Mostrar hijos');
             // Validación.
             $this->addRule('antiguedad', 'La antigüedad debe ser un número natural.',