Bug Fixes

[mecon/samurai.git] / sistema / www / perfiles / perfiles-abm.php

diff --git a/sistema/www/perfiles/perfiles-abm.php b/sistema/www/perfiles/perfiles-abm.php
index cf7fb6637bd6cb6a9a5da16f194756614024bc8e..6624d619192a0d71a5505b69ee659f2893ca7630 100644 (file)
--- a/sistema/www/perfiles/perfiles-abm.php
+++ b/sistema/www/perfiles/perfiles-abm.php
@@ -23,9 +23,11 @@
 // REQUIRE_ONCE {{{
     require_once 'MECON/HTML/QuickForm.php';
     require_once 'MECON/HTML/Tabla.php';
 // REQUIRE_ONCE {{{
     require_once 'MECON/HTML/QuickForm.php';
     require_once 'MECON/HTML/Tabla.php';

+    require_once 'MECON/HTML/Error.php';

     require_once 'SAMURAI/Perfil.php';
     require_once 'SAMURAI/Permiso.php';
     require_once 'SAMURAI/Sistema.php';
     require_once 'SAMURAI/Perfil.php';
     require_once 'SAMURAI/Permiso.php';
     require_once 'SAMURAI/Sistema.php';

+    require_once 'HTML/Image.php';

 // }}}
 //CREO LOS OBJETO NECESARIOS {{{
     if (@$_GET['id_perfil']) {
 // }}}
 //CREO LOS OBJETO NECESARIOS {{{
     if (@$_GET['id_perfil']) {


@@ -68,22 +70,26 @@
     }
 // }}}
 //AGREGO LOS ELEMENTOS DEL FORM {{{
     }
 // }}}
 //AGREGO LOS ELEMENTOS DEL FORM {{{

-    $FORM->addElement ('header', 'cabecera', 'ABM Perfiles');
-    $FORM->addElement ('hidden', 'id_perfil', $id_perfil);
-    $FORM->addElement ('hidden', 'desc_ant'   , $descripcion);
-    $FORM->addElement ('static', 'desc_perfil', 'Descripcion', $descripcion);
-    $FORM->addElement ('select', 'permisos'   , 'Permisos'   , $PERMISOS, array('size' => '5', 'multiple' => 'true'));
-    $FORM->addElement ('select', 'tipo_perfil', 'Tipo Perfil', array('E' => 'Externo', 'I' => 'Interno', 'D' => 'Dios'), array ('size' => '1'));
-    $FORM->addElement ('hidden', 'responsable', $responsable);
-    $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar');
-    $group[] = HTML_QuickForm::createElement('submit', 'cancelar', 'Cancelar', array ('onClick' => 'javascript:history.go(-1);return true;'));
-    $FORM->addGroup($group,'botones');
+    if (@$PERMISOS) {
+        $FORM->addElement ('header', 'cabecera', 'ABM Perfiles');
+        $FORM->addElement ('hidden', 'id_perfil', $id_perfil);
+        $FORM->addElement ('hidden', 'desc_ant'   , $descripcion);
+        $FORM->addElement ('static', 'desc_perfil', 'Descripcion', $descripcion);
+        $FORM->addElement ('select', 'permisos'   , 'Permisos'   , $PERMISOS, array('size' => '5', 'multiple' => 'true'));
+        $FORM->addElement ('select', 'tipo_perfil', 'Tipo Perfil', array('E' => 'Externo', 'I' => 'Interno', 'D' => 'Dios'), array ('size' => '1'));
+        $FORM->addElement ('hidden', 'responsable', $responsable);
+        $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar', array ('onClick' => 'javascript:if(confirm(\'¿Esta Seguro?\')) return true; return false;'));
+        $group[] = HTML_QuickForm::createElement('submit', 'cancelar', 'Cancelar', array ('onClick' => 'javascript:window.location=\'perfiles\';return false;'));
+        $FORM->addGroup($group,'botones');
+    }
+    else {
+        $ERROR = new MECON_HTML_Error('El sistema no posee permisos asociados.');
+    }

 // }}}
 //AGREGO LAS REGLAS DE VALIDACION {{{
     $FORM->addRule ('permisos', 'Se debe seleccionar al menos un permiso', 'required');
 // }}}
 //CARGO LOS DATOS SI YA EXISTEN EN BASE {{{
 // }}}
 //AGREGO LAS REGLAS DE VALIDACION {{{
     $FORM->addRule ('permisos', 'Se debe seleccionar al menos un permiso', 'required');
 // }}}
 //CARGO LOS DATOS SI YA EXISTEN EN BASE {{{

-// TODO Terminar de armar la parte de modificacion y eliminacion

     if (isset($_GET['accion']) && $_GET['accion'] != '') {
         //MODIFICACION
         $id_perfil    =& $FORM->getElement  ('id_perfil'  );
     if (isset($_GET['accion']) && $_GET['accion'] != '') {
         //MODIFICACION
         $id_perfil    =& $FORM->getElement  ('id_perfil'  );


@@ -140,9 +146,7 @@
             $res = $PERFIL->guardarDatos($aceptar->getValue());
 
             if (PEAR::isError($res)) {
             $res = $PERFIL->guardarDatos($aceptar->getValue());
 
             if (PEAR::isError($res)) {

-                $TABLA = new Tabla ('cellspacing=0');
-                $row = array ('<font color="red"><b>'.$res->getMessage().'</b></font>');
-                $TABLA->addRow($row,'align=left');
+                $ERROR = new MECON_HTML_Error($res->getMessage());

                 if ($aceptar->getValue() == 'Eliminar') {
                     $FORM->freeze();
                 }
                 if ($aceptar->getValue() == 'Eliminar') {
                     $FORM->freeze();
                 }


@@ -155,15 +159,16 @@
     }
 // }}}
 //DIBUJO LA PAGINA {{{
     }
 // }}}
 //DIBUJO LA PAGINA {{{

+    $imagen = new HTML_Image('/MECON/images/vinetas_flecha_doble.gif');
+    $row    = array ($imagen->toHtml().'&nbsp;<b>SISTEMA: '.$SISTEMA->getNombre().'</b>');

     $TABLA3 = new Tabla ('cellspacing=0');
     $TABLA3 = new Tabla ('cellspacing=0');

-    $row   = array ('<b>SISTEMA: '.$SISTEMA->getNombre().'</b>');
-    $TABLA3->addRow($row,'titulo align=left');    
+    $TABLA3->addRow($row,'align=left');    

 
     $MARCO = new Marco ('../../conf/confSecciones.php');
     $MARCO->addTitle('ABM Perfil');
     $MARCO->addBody($TABLA3);
 
     $MARCO = new Marco ('../../conf/confSecciones.php');
     $MARCO->addTitle('ABM Perfil');
     $MARCO->addBody($TABLA3);

-    if (isset($TABLA)) {
-        $MARCO->addBody($TABLA);
+    if (isset($ERROR)) {
+        $MARCO->addBody($ERROR);

     }
     $MARCO->addBody($FORM);
     $MARCO->display();
     }
     $MARCO->addBody($FORM);
     $MARCO->display();