BugFixes a la primera version de addMenuVertical de Marco

[mecon/meconlib.git] / lib / MECON / Marco / Seccion.php

diff --git a/lib/MECON/Marco/Seccion.php b/lib/MECON/Marco/Seccion.php
index fc386977bbcfef13bb037be12ddc591b865e3a1f..661cfc8fda712bb41e520e2721f3c34fd38ea038 100644 (file)
--- a/lib/MECON/Marco/Seccion.php
+++ b/lib/MECON/Marco/Seccion.php
@@ -43,6 +43,7 @@ require_once 'MECON/Marco/MenuOculto.php';
 /**
  * Clase seccion para el manejo de las secciones
  *
+ * @package MECON_Marco
  * @access public
  */
 class Seccion extends Pagina {
@@ -105,34 +106,22 @@ class Seccion extends Pagina {
      * @access public
      * @static
      */
-    function Seccion($seccion, $configuracion)// ~X2C
+    function Seccion($seccion, $configuracion) // ~X2C
     {
-        if (array_key_exists('nombre',$seccion)) { 
-            $this->_nombre = $seccion['nombre'];
+        $this->_nombre   = @strval($seccion['nombre']);
+        $imgComun        = @strval($seccion['imagenComun']);
+        $imgMouseOn      = @strval($seccion['imagenMouseOn']);
+        $imgSelect       = @strval($seccion['imagenSelect']);
+        $this->_tipoMenu = @strval($seccion['tipoMenu']);
+        if (@$seccion['link']) {
+            parent::Pagina($seccion['link']);
         }        
-        if (array_key_exists('imagenComun', $seccion))
-            $imgComun = $seccion['imagenComun'];
-        else 
-            unset($imgComun);
-        if (array_key_exists('imagenMouseOn', $seccion))
-            $imgMouseOn = $seccion['imagenMouseOn'];
-        else
-            unset($imgMouseOn);
-        if (array_key_exists('imagenSelect', $seccion))
-            $imgSelect = $seccion['imagenSelect'];
-        else
-            unset($imgSelect);
-        $this->_imagen = new ImagenAnimada($imgComun, $imgMouseOn, $imgSelect, $configuracion['directorios']['imagenes']);
-        
-        if (array_key_exists('tipoMenu', $seccion)) {
-            $this->_tipoMenu = $seccion['tipoMenu'];
-        }
+        $this->_imagen = new ImagenAnimada($imgComun, $imgMouseOn, $imgSelect,
+            $configuracion['directorios']['imagenes'], $this->_nombre,
+            $configuracion['directorios']['root'].'/'.$this->_link);
         if (!is_null($configuracion)) {
             $this->_configuracion = $configuracion;
         }
-        if (array_key_exists('link', $seccion)) {
-            parent::Pagina($seccion['link']);
-        }        
         if (array_key_exists('hijos', $seccion)) {
             $this->_cargarHijos($seccion['hijos']);
         }
@@ -148,27 +137,15 @@ class Seccion extends Pagina {
      * @return string
      * @access public
      */
-    function toHtml($link_sel)// ~X2C
+    function toHtml($link_sel) // ~X2C
     {
         if (isset($_SESSION['deshabilitar_links']) && $_SESSION['deshabilitar_links']) {
-            $link_start  = '';
-            $link_end    = '';
-            $link_start2 = '';
-        }
-        else {
-            $link_start  = '<a href="'.$this->_configuracion['directorios']['root'].'/'.$this->_link.'">';
-            $link_start2 = '<a href="'.$this->_configuracion['directorios']['root'].'/'.$this->_link.'" onMouseOut="MM_swapImgRestore()" onMouseOver="MM_displayStatusMsg(\''.$this->_nombre.'\'); MM_swapImage(\''.$this->_nombre.'\',\'\',\''.$this->_imagen->_imgMouseOn.'\',1); return document.MM_returnValue" MM_swapImage(\''.$this->_nombre.'\',\'\',\''.$this->_imagen->_imgMouseOn.'\',1)>';
-            $link_end    = '</a>';
+            $this->_imagen->setHabilitada(false);
         }
-
         if ($this->verifSeccionSeleccionada($link_sel)) {
-            $sec    = $link_start."<img name=\"".$this->_nombre."\" src=\"".$this->_imagen->_imgSelect."\" border=\"0\" alt=\"".$this->_nombre."\">".$link_end;
-        }
-        else {
-            $sec = $link_start2."<img name=\"".$this->_nombre."\" src=\"".$this->_imagen->_imgComun."\" border=\"0\" alt=\"".$this->_nombre."\">".$link_end;
+            $this->_imagen->setSeleccionada(true);
         }
-        
-        return $sec;
+        return $this->_imagen->toHtml();
     }
     // -X2C
 
@@ -181,7 +158,7 @@ class Seccion extends Pagina {
      * @return void
      * @access private
      */
-    function _cargarHijos($hijos)// ~X2C
+    function _cargarHijos($hijos) // ~X2C
     {
         if ($this->_tipoMenu == 'vertical'){
             $tmp = new MenuVertical($this->_configuracion);
@@ -206,7 +183,7 @@ class Seccion extends Pagina {
      * @return void
      * @access private
      */
-    function _desSerializarArraySecciones()// ~X2C
+    function _desSerializarArraySecciones() // ~X2C
     {
         $file_cache = strtr($this->_configuracion['directorios']['root'],'/','_');
         $tmp = $this->_configuracion['directorios_fs']['cache'].'/'.ARRAYSECCIONES_SERIALIZADO.'_'.$file_cache;
@@ -231,7 +208,7 @@ Devuelve 1 si pertenece a la seccion, en caso contrario 0.
      * @return int
      * @access public
      */
-    function verifSeccionSeleccionada($link_sel)// ~X2C
+    function verifSeccionSeleccionada($link_sel) // ~X2C
     {
         $tmp = $this->_desSerializarArraySecciones();
         $retorno = 0;
@@ -256,7 +233,7 @@ Devuelve 1 si pertenece a la seccion, en caso contrario 0.
      * @return string
      * @access public
      */
-    function toHtmlVertical($link_sel, $ultimo = false)// ~X2C
+    function toHtmlVertical($link_sel, $ultimo = false) // ~X2C
     {
         $style = "text-decoration:none";