BugFix en la interfaz grafica en sistema/www/perfiles/perfiles.php

[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 badc3b287c001fb3e27612c3e8c858af0ba31944..0440f45cc6a71fe05b610a1c4ab4cb8dea3a43b2 100644 (file)
--- a/sistema/www/perfiles/perfiles-abm.php
+++ b/sistema/www/perfiles/perfiles-abm.php
@@ -20,9 +20,29 @@
 // $Id$
 // 
 //VERIFICO SI SE TIENE ACCESO A LA PAGINA{{{
 // $Id$
 // 
 //VERIFICO SI SE TIENE ACCESO A LA PAGINA{{{

-    if (!$SAMURAI_PERM->tiene(SAMURAI_ALTA_PERFIL,SAMURAI_BAJA_PERFIL,SAMURAI_MODI_PERFIL)) {
-//        header('Location: error')
-    }
+$SAMURAI_PERM->setSistema(SAMURAI_PERM);
+if (!$SAMURAI_PERM->tiene(SAMURAI_PERM_DEVELOPER)) {
+    $SAMURAI_PERM->setSistema($_SESSION['samurai']['id_sistema']);
+    $SAMURAI_PERM->chequear(SAMURAI_PERM_PERFIL_ALTA,SAMURAI_PERM_PERFIL_BAJA,SAMURAI_PERM_PERFIL_MODIF, SAMURAI_PERM_DEVELOPER);
+}
+$MARCO =& new MECON_Marco ('/var/www/sistemas/samurai/sistema/conf/confSecciones.php', $SAMURAI_PERM);
+//}}}
+
+//XXX OBTENGO LA ACCION {{{
+switch (@$_REQUEST['accion']) {
+    case 'm':
+        $b_accion = 'Modificar';
+        $accion = 'm';
+        break;
+    case 'e':
+        $b_accion = 'Eliminar';
+        $accion = 'e';
+        break;
+    default:
+        $b_accion = 'Grabar';
+        $accion = '';
+        break;
+}

 //}}}
 
 // REQUIRE_ONCE {{{
 //}}}
 
 // REQUIRE_ONCE {{{


@@ -32,7 +52,7 @@
     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';
+    require_once 'MECON/HTML/Image.php';

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


@@ -45,14 +65,16 @@
         $id_perfil = null;
     }
     $FORM = new MECON_HTML_QuickForm ('perfiles_abm','post','perfiles-abm');
         $id_perfil = null;
     }
     $FORM = new MECON_HTML_QuickForm ('perfiles_abm','post','perfiles-abm');

+    $FORM->renderer->updateAttributes('width="400"');

     $SISTEMA = new SAMURAI_Sistema ($DB, $_SESSION['samurai']['id_sistema']);
     $SISTEMA = new SAMURAI_Sistema ($DB, $_SESSION['samurai']['id_sistema']);

-    $PERFIL = new SAMURAI_Perfil ($DB, $id_perfil);
+    $PERFIL = new SAMURAI_Perfil ($DB, $id_perfil,
+            $_SESSION['samurai']['id_sistema']);

 
     if ($PERFIL->getResponsable()) {
         $responsable = $PERFIL->getResponsable();
     }
     else {
 
     if ($PERFIL->getResponsable()) {
         $responsable = $PERFIL->getResponsable();
     }
     else {

-        $responsable = $_SESSION['samurai']['login'];
+        $responsable = $_SESSION['usuario'];

     }
     
     if (!@$_GET['id_perfil']) {
     }
     
     if (!@$_GET['id_perfil']) {


@@ -79,13 +101,11 @@
         $FORM->addElement ('header', 'cabecera', 'ABM Perfiles');
         $FORM->addElement ('hidden', 'id_perfil', $id_perfil);
         $FORM->addElement ('hidden', 'desc_ant'   , $descripcion);
         $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 ('static', 'desc_perfil', 'Descripción', $descripcion);

         $FORM->addElement ('select', 'permisos'   , 'Permisos'   , $PERMISOS, array('size' => '5', 'multiple' => 'true'));
         $FORM->addElement ('select', 'permisos'   , 'Permisos'   , $PERMISOS, array('size' => '5', 'multiple' => 'true'));

-        if ($SAMURAI_PERM->tiene(SAMURAI_TIPO_PERFIL)) {
-            $FORM->addElement ('select', 'tipo_perfil', 'Tipo Perfil', array('E' => 'Externo', 'I' => 'Interno', 'D' => 'Dios'), array ('size' => '1'));
-        }

         $FORM->addElement ('hidden', 'responsable', $responsable);
         $FORM->addElement ('hidden', 'responsable', $responsable);

-        $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar');
+        $FORM->addElement ('hidden', 'accion', $accion);
+        $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , $b_accion);

         $group[] = HTML_QuickForm::createElement('submit', 'cancelar', 'Cancelar', array ('onClick' => 'javascript:window.location=\'perfiles\';return false;'));
         $FORM->addGroup($group,'botones');
     }
         $group[] = HTML_QuickForm::createElement('submit', 'cancelar', 'Cancelar', array ('onClick' => 'javascript:window.location=\'perfiles\';return false;'));
         $FORM->addGroup($group,'botones');
     }


@@ -102,10 +122,6 @@
         $id_perfil    =& $FORM->getElement  ('id_perfil'  );
         $desc_perfil  =& $FORM->getElement  ('desc_perfil');
         $permisos     =& $FORM->getElement  ('permisos'   );
         $id_perfil    =& $FORM->getElement  ('id_perfil'  );
         $desc_perfil  =& $FORM->getElement  ('desc_perfil');
         $permisos     =& $FORM->getElement  ('permisos'   );

-        
-        if ($SAMURAI_PERM->tiene(SAMURAI_TIPO_PERFIL)) {
-            $tipo_perfil  =& $FORM->getElement  ('tipo_perfil');
-        }

         $responsable  =& $FORM->getElement  ('responsable');
         $group        =& $FORM->getElement  ('botones'    );
         $group        =& $group->getElements('aceptar'    );
         $responsable  =& $FORM->getElement  ('responsable');
         $group        =& $FORM->getElement  ('botones'    );
         $group        =& $group->getElements('aceptar'    );


@@ -113,10 +129,6 @@
 
         $permisos->setSelected($PERFIL->getPermisos());
         
 
         $permisos->setSelected($PERFIL->getPermisos());
         

-        if ($SAMURAI_PERM->tiene(SAMURAI_TIPO_PERFIL)) {
-            $tipo_perfil->setSelected($PERFIL->getTipo());
-        }
-

         if ($PERFIL->getResponsable() != '') {
             $responsable->setValue($PERFIL->getResponsable());
         }
         if ($PERFIL->getResponsable() != '') {
             $responsable->setValue($PERFIL->getResponsable());
         }


@@ -144,11 +156,6 @@
             $id_perfil    =& $FORM->getElement  ('id_perfil'  );
             $desc_perfil  =& $FORM->getElement  ('desc_perfil');
             $permisos     =& $FORM->getElement  ('permisos'   );
             $id_perfil    =& $FORM->getElement  ('id_perfil'  );
             $desc_perfil  =& $FORM->getElement  ('desc_perfil');
             $permisos     =& $FORM->getElement  ('permisos'   );

-            
-            if ($SAMURAI_PERM->tiene(SAMURAI_TIPO_PERFIL)) {
-                $tipo_perfil  =& $FORM->getElement  ('tipo_perfil');
-                $tipo_perfil_tmp  =  $tipo_perfil->getSelected();
-            }

             $responsable  =& $FORM->getElement  ('responsable');
             $group        =& $FORM->getElement  ('botones'    );
             $group        =& $group->getElements('aceptar'    );
             $responsable  =& $FORM->getElement  ('responsable');
             $group        =& $FORM->getElement  ('botones'    );
             $group        =& $group->getElements('aceptar'    );


@@ -158,10 +165,6 @@
             $PERFIL->setResponsable($responsable->getValue());
             $PERFIL->setPermisos   ($permisos->getSelected()); //Le asigno al perfil los permisos seleccionados
             
             $PERFIL->setResponsable($responsable->getValue());
             $PERFIL->setPermisos   ($permisos->getSelected()); //Le asigno al perfil los permisos seleccionados
             

-            if ($SAMURAI_PERM->tiene(SAMURAI_TIPO_PERFIL)) {
-                $PERFIL->setTipo       ($tipo_perfil_tmp['0']   );
-            }
-            

             //Grabo y verifico si se produjo algun error
             $res = $PERFIL->guardarDatos($aceptar->getValue());
 
             //Grabo y verifico si se produjo algun error
             $res = $PERFIL->guardarDatos($aceptar->getValue());
 


@@ -179,7 +182,7 @@
     }
 // }}}
 //DIBUJO LA PAGINA {{{
     }
 // }}}
 //DIBUJO LA PAGINA {{{

-    $imagen = new HTML_Image('/MECON/images/vinetas_flecha_doble.gif');
+    $imagen = new MECON_HTML_Image('/MECON/images/vinetas_flecha_doble.gif');

     $row    = array ($imagen->toHtml().'&nbsp;<b>SISTEMA: '.$SISTEMA->getNombre().'</b>');
     $TABLA3 = new MECON_HTML_Tabla ('cellspacing=0');
     $TABLA3->addRow($row,'align=left');    
     $row    = array ($imagen->toHtml().'&nbsp;<b>SISTEMA: '.$SISTEMA->getNombre().'</b>');
     $TABLA3 = new MECON_HTML_Tabla ('cellspacing=0');
     $TABLA3->addRow($row,'align=left');