Usar datos de resultado de Query, fuera del While

KATHYU · #1 (**permalink**) 08/05/2018, 11:49

Buen día.

Tengo una duda y espero me den una luz.
En mi sistema, tengo varias opciones de menú a la que algunos no pueden ingresar, (Aprobar Ordenes de Compra por ejemplo) lo que hago es que al dar clic a esa opción se valla al php en cuestión y evalué si el usuario tiene privilegio de Aprobar Ordenes de Compra si tiene lo deja pasar y si no lo manda a una pagina de error y lo regresa a donde estaba antes de haber dado clic al enlace, hasta ahí todo bien, funciona y todo.

La cosa es que quiero implementar otro tipo de "seguridad" o mejor dicho otro tipo de método, quiero ocultar las opciones a las que no tenga permiso el usuario, por lo que se me ocurrió hacer un query donde consulto todos los privilegios por el usuario de la sesión

Código PHP:

Ver original$id = $_SESSION['iduser'];
$privilegios = $mysqli->query("SELECT accion,privilegio FROM privilegio WHERE usuario='$id' ") or trigger_error($mysqli->error);
while($row = $privilegios->fetch_array(MYSQLI_ASSOC)){
    $axx = $row['accion'];
    $pxx = $row['privilegio'];
    $pasox = array($axx => $pxx);
    // foreach($pasox as $acci=>$privi)
    // {
    // echo "Accion " . $acci . " Con Privilegio " . $privi;
    // echo "<br>";
    // }
    // var_dump($pasox);
}

* Esta porcion de codigo la quiero incluir en el menu para evitar ponerla en todos lados.

Pero haciendo pruebas, tengo el problema que no me logra sacar la info que necesito si no estoy dentro del while del result

Si habo el var_dump($pasox) dentro del while funciona y muestra todo el array ($pasox) pero si lo hago fuera solo muestra la ultima fila y ojo se que eso es normal pero lo que quiero saber es si puedo evitar eso, por eso es el tema

Incluso si quiero hacer un (if) fuera del while sucede lo mismo, obviamente cuando imprimo una tabla de datos lo hago dentro del while pero con esto no puedo hacerlo puesto que quiero tratar el resultado en otros archivos php, incluso pensé en meter el resultado en una variable de sesión pero no me sale.

Luego buscar en el array para ver si el fulano tiene o no "X" privilegio y por medio de CSS ocultarle algunas cosas

Código PHP:

Ver originalif ($axx = 21 ) {
    echo "Existe 1";
}
else{
    echo "No Existe 1";
}
 
if (in_array(21, $pasox) ) {
    echo "Existe 2";
}
else{
    echo "No Existe 2";
}

Tienen alguna idea de que podría hacer para sacar un array o una variable que tenga todos los privilegios de un usuario y usarlo donde sea y cuando sea?

Se que hay varios modelos de gestión de permisos por ahí pero yo necesito algo que se adecue a lo que ya tengo

ocp001a · #2 (**permalink**) 08/05/2018, 12:08

Dependerá mucho de cómo tengas almacenados los permisos, la forma en que puedes sacar la lista, pero creo que tu código falla aquí:

$pasox = array($axx => $pxx);

Con cada iteración estás sobreescribiendo el arreglo, creo que deberías hacerlo así:

$pasox[]=array($axx => $pxx);

KATHYU · #3 (**permalink**) 08/05/2018, 15:03

Muy bien, muy bien, vamos avanzando, ya se puede hacer el var_dump($pasox) desde fuera del while y se muestran todos los permisos.

La forma en como están los datos es así, columna acción, columna permiso, permiso puede ser 0 o 1, y después de sacar esos datos lo que necesito hacer casi que donde sea es evaluar, como así ?

Aprobar Orden de Compra es una acción en la DB con código 44

Pseudo codigo

Código:

Si acción 44 = 1 entonces imprimir menu // Osea tiene permiso este usuario
    Aprobar Orden de Compra
Fin Si

Así evaluaría unas cuantas acciones, después con CSS las ocultaría, esa es mi idea, tipo el sistema de roles de Symfony

vicram10 · #4 (**permalink**) 15/05/2018, 13:44

de la manera como yo lo hago es asi

Tengo una tabla donde es una clave compuesta, o sea se repite una palabra clave y es variable el id del departamento o role

Tabla: menu_acceso

codMenu / user_depart

agregar-usuario / 1
agregar-usuario / 2
agregar-usuario / 5
agregar-usuario / 10

Tengo una funcion que revisa dicha tabla acuerdo al ID_DEPART de cada usuario de la siguiente forma

Código PHP:

Ver original//para saber si que permisos tiene el usuario que se loguea
    function PermisosMenu($menu)
    {
        global $capaNegocios, $datosUsuario, $user_key;
        
        //el administrador puede ver todo
        if ($datosUsuario['es_admin'] == 1)
        {
            return true;
        }else{
            $idDepart = (int) $datosUsuario['id_depart'];
            //verificamos si tiene los permisos necesarios
            $tienePermiso = $capaNegocios->ContadorRegistrosTabla('menu_acceso', "where codMenu = '$menu' and user_depart = $idDepart");
            if ($tienePermiso > 0)
                return true;
            else
                return false;
        }
    }

Obs.: la variable que uso $datosUsuario es un array con los datos del usuario que esta logueado

Para usar dicha funcion lo hago desde el array o constructor de mi menu principal de la siguiente forma

Código PHP:

Ver original$menu_principal = array(
    'usuarios' => array(
            'titulo_menu' => 'Agregar Usuarios',
            'url' => '?opt=Usuarios/Agregar', //o la url que usas para cada accion
            'visible' => PermisosMenu('agregar-usuario'), //este devuelve true o false como se puede ver arriba
    ),
);

Entonces donde imprimo el menu hago asi

Código PHP:

Ver originalforeach($menu_principal as $key => $value){
    if ($value['visible']){
        echo '<a href="', $value['url'] ,'">', $value['titulo_menu'] ,'</a>';
    }
}

KATHYU · #5 (**permalink**) 15/05/2018, 14:03

Agradezco mucho su respuesta

Su menú se crea cada vez que se crea la pagina? a mi no me gusta eso de los menús dinámicos

Por otro lado, mi forma de manejo de permisos ya esta creada y funciona bien, lo que quería era hacer un array de los permisos de los usuarios y poder evaluar si un usuario tiene o no permiso a X opción, si no tiene pues se lo escondo.

Analizare mejor su código y veré si puedo sacar algo de ahí, aunque recuerde el asunto central de mi post, incluso esta en el tema Usar datos de resultado de Query, fuera del While

vicram10 · #6 (**permalink**) 15/05/2018, 14:10

yap.. en parte te puede servir

Código PHP:

Ver original$pasox[] = array($axx => $pxx);

Este array contienen los permisos que posee el usuario (permisos de acceso a cada menu)?.

y lo segundo seria como armas el menu para poder insertar ese vector y comparar si tiene o no acceso al menu..

KATHYU · #7 (**permalink**) 15/05/2018, 15:53

Cita:

Iniciado por vicram10

yap.. en parte te puede servir

Código PHP:

Ver original$pasox[] = array($axx => $pxx);

Este array contienen los permisos que posee el usuario (permisos de acceso a cada menu)?.

y lo segundo seria como armas el menu para poder insertar ese vector y comparar si tiene o no acceso al menu..

Como dijo Jack el Destripador, por partes por favor...

El array $pasox[] en efecto contiene todas las claves de acciones ($axx) apuntando a su debido permiso ($pxx) que este puede ser 0 o 1

En escencia, si se loguea el usuario FDW su array de permisos seria algo así

Código:

accion1 => 1
accion2 => 1
accion3 => 0
accion4 => 1
accion5 => 0
accion6 => 1

El menu yo no lo genero, yo tengo un archivo llamado menu.php con varias listas de menu, y por ejemplo en el menu tengo la opcion INACTIVAR USUARIO que pertenece a la acción 5 y como FDW tiene permiso 0, yo pondria como hidden esa opcion de menu y FDW no la podria ver.

Mi idea es incluir en el menu.php un archivo que me extraiga los permisos, (include permisos.php;) porque? porque asi yo puedo incluir ese archivo no solo en menus si no en cualquier otra cosa, al estar incluido, yo podria tomar el array $pasox[] generado en permisos.php y hacer magia, ocultando cosas segun me convenga

Por eso mi urgencia de manejar un array o result del query, fuera del while($row = $privilegios->fetch_array(MYSQLI_ASSOC)){}

PD: los nombres de las variables son ejemplos, no crean que así nombro mis variables

ocp001a · #8 (**permalink**) 15/05/2018, 16:53

No entiendo si lo último que expones es simplemente un comentario sobre tu procedimiento o si tienes alguna duda...

De cualquier manera como sugerencia, dentro del script php donde realices la acción o procedimiento, revisa que el usuario efectivamente tiene permiso de hacerlo, ya sabes, por que el cliente podría falsear el menú, nunca hay que confiar en los datos de entrada.

KATHYU · #9 (**permalink**) 16/05/2018, 09:16

Cita:

Iniciado por ocp001a

No entiendo si lo último que expones es simplemente un comentario sobre tu procedimiento o si tienes alguna duda...

Pues la duda sigue, la duda del tema PRINCIPAL.

Cita:

Iniciado por ocp001a

De cualquier manera como sugerencia, dentro del script php donde realices la acción o procedimiento, revisa que el usuario efectivamente tiene permiso de hacerlo, ya sabes, por que el cliente podría falsear el menú, nunca hay que confiar en los datos de entrada.

Eso siempre lo haré, y es lo que actualmente hago, lo único que quiero es esconder opciones de menú sin permiso, pero tomando como base mi actual sistema de permisos, explicado en el primer post

ocp001a · #10 (**permalink**) 16/05/2018, 10:09

Bien, entonces, si no entendí mal, en tu menu.php tienes varias listas de menú predefinidas.

De ser así, no lo veo muy práctico, y en cambio te recomiendo generar el menú con base en los permisos de usuario en tiempo real, es decir, podrías guardar los permisos del usuario en una variable de sesión, y en el php (o javascript) tener una función que genere el menú según los permisos dados.

Yo tengo un sistema de permisos de la siguiente forma:

En mi sistema hay secciones, dentro de las secciones módulos, cada módulo puede tener de 1 a 10 pestañas, y dentro de cada pestaña pueden hacer 4 acciones: Guardar, modificar, borrar y reportear.
Obviamente habrá pestañas que no hagan las 4 acciones, pero así se puede generalizar.

Tengo una tabla donde registro los módulos y sus pestañas.

Así, se puede definir menús de forma ilimitada, estableciendo perfiles de usuario, y asignando a cada usuario un perfil/menú.

Si un día se necesita que a un usuario se le defina más o menos permisos, se modifica su perfil o se crea un perfil especial para el usuario.

En el login, se carga la lista de permisos del perfil ligado al usuario y se guarda en sesión.

Le paso el arreglo con los permisos a javascript mediante JSON, y ahí tengo una función que dibuja el menú, organizado por secciones y módulos.

Dentro del sistema, cada módulo sólo necesita leer los permisos del usuario para dicho módulo, por ejemplo:

módulo catálogo de clientes:

Código PHP:

Ver originalif(!isset($usuario['permisos']['cat_clientes']))
   die('No tiene permitido entrar a este modulo');
if($usuario['permisos']['cat_clientes']['pes1']==1)
   //dibujar pestaña 1
if($usuario['permisos']['cat_clientes']['pes2']==1)
   //dibujar pestaña 2

Espero que esto te ayude en algo. De lo contrario , te sugiero describir con más detalle cómo dibujas el menú, con el código que llevas para tartar de ayudarte de manera más específica.

KATHYU · #11 (**permalink**) 16/05/2018, 14:53

Cita:

Iniciado por ocp001a

Espero que esto te ayude en algo. De lo contrario , te sugiero describir con más detalle cómo dibujas el menú, con el código que llevas para tartar de ayudarte de manera más específica.

En realidad mi me dio una idea, y es dibujar el menu al "vuelo", dinamicamente. Claro eso lo tendria que pensar y replantear a futuro.

Con lo citado: Yo no dibujo el menu, creo que asi le dicen cuando el menu es dinamico, yo ya tengo el archivo menu.php, y es un menu que con Bootstrap hace que se vea decente.

Código HTML:

Ver original<ul class="dropdown-menu mega-dropdown-menu row">
    <li class="col-sm-3">
        <ul>
            <li class="dropdown-header">Proveedores</li>
            <li><a href="CREAR_PROVEEDOR">CREAR</a></li>
            <li><a href="ADMIN_PROVEEDOR">ADMINISTRAR</a></li>
            <li class="divider"></li>
        </ul>
    </li>
    <li class="col-sm-3">
        <ul>
            <li class="dropdown-header">Solicitud de Compra</li>
            <li><a href="ORDEN_COMPRA">NUMERAR SOLICITUD</a></li>
            <li class="divider"></li>
        </ul>
    </li>
</ul>

pues en ese archivo, quiero hacer un include del archivo permisos.php, ese archivo tendría una query a la base y guardaría en un array los permisos del usuario del tipo accion => permiso, obviamente tomando como parámetro el usuario logeado

Código:

CREAR_PROVEEDOR => 1
ADMIN_PROVEEDOR=> 1
ORDEN_COMPRA => 0

En el menu.php haría un simple IF, en cada opción, que si en mi array de permisos, esta a 1 X opcion la muestre, y si no que le ponga la clase hidden y por ende la oculte.

Código PHP:

Ver originalIF ($ORDEN_COMPRA =1 ){
 <a href="CREAR_PROVEEDOR">CREAR</a>
}
else{
  <a href="CREAR_PROVEEDOR" class="hidden">CREAR</a>
}

Como ven no es mucha ciencia, lo único que necesito es manejar el array de permisos y pues no queria variable de sesión pero a como veo la cosa, creo que tocara algo asi

ocp001a · #12 (**permalink**) 16/05/2018, 15:33

No necesitas obligatoriamente usar una variable de sesión, es lo que se usa comúnmente para no estar leyendo de la base de datos cada vez que se refresca la url, pero no es que no se pueda.

Por otra parte, más que tener un

Código PHP:

Ver originalIF ($ORDEN_COMPRA ==1 ){
     <a href="CREAR_PROVEEDOR">CREAR</a>
    }
    else{
      <a href="CREAR_PROVEEDOR" class="hidden">CREAR</a>
    }

sería mejor un

Código PHP:

Ver originalIF ($ORDEN_COMPRA ==1 ){
     <a href="CREAR_PROVEEDOR">CREAR</a>
    }

Es decir, si no tiene el permiso, ni siquiera se crea la opción, no tiene sentido ocultar algo y cargar html que no se va a ver al cliente, después de todo, si alguien inspecciona el html podría habilitar la oción, aunque la valides luego en el servidor, no le veo caso tenerla.

Ahora bien, sobre la pregunta original, podrías crear un arreglo asociativo, para más comodidad:

Código PHP:

Ver original$id = $_SESSION['iduser'];
    $privilegios = $mysqli->query("SELECT accion,privilegio FROM privilegio WHERE usuario='$id' ") or trigger_error($mysqli->error);
    while($row = $privilegios->fetch_array(MYSQLI_ASSOC)){
        $pasox[$row['accion']] =$row['privilegio'];
        // foreach($pasox as $acci=>$privi)
        // {
        // echo "Accion " . $acci . " Con Privilegio " . $privi;
        // echo "<br>";
        // }
        // var_dump($pasox);
    }

Así, en el menú podrías hacer algo como:

Código PHP:

Ver originalIF ($pasox['ORDEN_COMPRA'] ==1 ){
     <a href="CREAR_PROVEEDOR">CREAR</a>
    }

KATHYU · #13 (**permalink**) 17/05/2018, 11:12

Cita:

Iniciado por ocp001a

No necesitas obligatoriamente usar una variable de sesión, es lo que se usa comúnmente para no estar leyendo de la base de datos cada vez que se refresca la url, pero no es que no se pueda.

Por otra parte, más que tener un

Es decir, si no tiene el permiso, ni siquiera se crea la opción, no tiene sentido ocultar algo y cargar html que no se va a ver al cliente, después de todo, si alguien inspecciona el html podría habilitar la oción, aunque la valides luego en el servidor, no le veo caso tenerla.

Ahora bien, sobre la pregunta original, podrías crear un arreglo asociativo, para más comodidad:

Se me fue la logica con lo primero sorry, es cierto, el if solo dibuja el menu cuando el privilegio es 1

Con lo segundo, con lo del array asociativo. El ultimo IF que pone deberia ponerlo en el menu.php, mas no hacer include, ya que ese if en cuestion lo debo hacer dentro del while, que quiero decir? que no puedo usar el query de permisos con otro archivo al cual yo le quiera ocultar algo.

Pero me parece buena la idea.

Gracias

vicram10 · #14 (**permalink**) 17/05/2018, 16:24

tu query de privilegios agregalo en uno de tus archivos que es llamado por todos tus otros archivos, entonces por asociacion de archivos lo tendras disponible en todos lados.

si tu menu.php se llama desde todos los lados, entonces ahi dentro podes tener el query con la variable de tus permisos.

y podras usarlo en cualquier otro lado.

KATHYU · #15 (**permalink**) 18/05/2018, 08:53

Cita:

Iniciado por vicram10

tu query de privilegios agregalo en uno de tus archivos que es llamado por todos tus otros archivos, entonces por asociacion de archivos lo tendras disponible en todos lados.

si tu menu.php se llama desde todos los lados, entonces ahi dentro podes tener el query con la variable de tus permisos.

y podras usarlo en cualquier otro lado.

Es una buena idea