1

Buen día Necesito obtener los deudores de una comunidad, para iniciar sesión en la comunidad puede ser un superadmin, que puede ver todos los registros, pero si inicia sesión como un administrador de una comunidad, solo puede ver los registros de esa comunidad. Tengo la siguiente sql:

public function getReservas(){
// Obtenemos el ID del usuario actual
$id_usuario = $_SESSION['id_username'];

// Verificamos si el usuario es superadmin
$superadmin = $_SESSION['role_id'];

// Si es superadmin, mostramos todas las dependencias
if ($superadmin == 1) {
  $sql = "SELECT r.id_reserva, r.id_depto, r.id_lugar, r.fechaHoraInicio, r.fechaHoraTermino, r.valor, r.nota, r.estado, r.id_comunidad,
        d.id_depto, d.nro_depto
        FROM reservas r
        LEFT JOIN deptos d ON r.id_depto = d.id_depto";
} else {
  $sql = "SELECT r.id_reserva, r.id_depto, r.id_lugar, r.fechaHoraInicio, r.fechaHoraTermino, r.valor, r.nota, r.estado, r.id_comunidad,
  d.id_depto, d.nro_depto
  FROM reservas r
  LEFT JOIN deptos d ON r.id_depto = d.id_depto
  WHERE id_comunidad = (SELECT id_comunidad FROM admins WHERE role_id = > 1)";
}
$data = $this->selectAll($sql);
return $data;

}

adjunto imagen de estructura de la tabla admins, reservas y comunidades

9
  • la consulta no obtiene datos por que tienes un return $data; en medio de un archivo php que a su vez tiene un funcion publica ... extrala organizacion del codigo... me gustaria saber por que no te revienta ... ya que esa forma en la que presentas el codigo no es funcional... Commented el 31 oct. 2023 a las 14:00
  • Puede salir en una sola consulta, supongo que tienes una tabla de usuarios ¿se relaciona con admins?.
    – Sal
    Commented el 31 oct. 2023 a las 15:40
  • Dudo que sea útil utilizar AND role_id == 1 (cosa que tiene una sintaxis incorrecta) y el otro que tienes de AND role_id >= 1, ya que eso lo debes obtener desde antes para validar tus condicionales, algo como if($_SESSION['role_id']==1){$sql="...";}else{$sql="...";}. La parte verdadera de tu condicional será igual que el código que pusiste primero (sin el WHERE d.id_comunidad = a.id_comunidad AND role_id == 1"). Y en la parte falsa debes OBTENER primero el id de la comunidad de ese usuario activo para poder ponerlo como variable en la condición de tu WHERE
    – Chandew
    Commented el 31 oct. 2023 a las 22:44
  • @Sal solo tengo una tabla admins, no necesito usuarios. Commented el 1 nov. 2023 a las 11:32
  • @Marcos Tienes razón, ya modifique el código, voy a editar la pregunta, ya muestra los datos si es el superadmin, pero no cuando es un admin Commented el 1 nov. 2023 a las 11:40

1 respuesta 1

0

Utilizando tu código actual y solo en el caso de que NO sea superadmin bastaría con agregar:

  • Un JOIN entre las tablas reservas y admins
  • Un WHERE indicando que el id del admin es el del usuario que inició sesión.

Ejemplo:

public function getReservas() {
    // Obtenemos el ID del usuario actual
    $id_usuario = $_SESSION['id_username'];

    // Verificamos si el usuario es superadmin
    $superadmin = $_SESSION['role_id'];

    // Si es superadmin, mostramos todas las dependencias
    if ($superadmin == 1) {
        $sql = "SELECT r.id_reserva, r.id_depto, r.id_lugar, r.fechaHoraInicio, r.fechaHoraTermino, r.valor, r.nota, r.estado, r.id_comunidad,
            d.id_depto, d.nro_depto
            FROM reservas r
            LEFT JOIN deptos d ON r.id_depto = d.id_depto";
    } else {
        $sql = "SELECT r.id_reserva, r.id_depto, r.id_lugar, r.fechaHoraInicio, r.fechaHoraTermino, r.valor, r.nota, r.estado, r.id_comunidad,
            d.id_depto, d.nro_depto
            FROM reservas r
            LEFT JOIN deptos d ON r.id_depto = d.id_depto
            LEFT JOIN admins a ON r.id_comunidad = a.id_comunidad
            WHERE a.id = $id_usuario";
    }
    $data = $this->selectAll($sql);
    return $data;
}

Otra opción sería modificar el WHERE que actualmente utilizas así:

$sql = "SELECT ...
    WHERE r.id_comunidad = (SELECT id_comunidad FROM admins WHERE id = ".$id_usuario.")";

Y finalmente, como en la variable de sesión ya tenes el id_comunidad, podrías simplemente hacer:

$sql = "SELECT ...
    WHERE r.id_comunidad = " . $_SESSION['id_comunidad'];

Comentario:

Para que los JOIN y/o el WHERE sean más eficientes sería bueno que agregues indices a los campos que relacionan las tablas entre sí.

Ejemplo:

ALTER TABLE `reservas` ADD KEY `id_comunidad` (`id_comunidad`);
ALTER TABLE `admins` ADD KEY `id_comunidad` (`id_comunidad`);
# etc.
2
  • Gracias @Marcos, el asunto es que creo que a esa parte de la consulta no llega id_comunidad, provee, igual como me indicas. Con los dos where, agregue, los indices a todas las tablas con id_comunidad. Inicio sesión como superadmin y funciona bien, pero como admin, da error de dataTable, eso me hace pensar que esa consulta falta algún campo. Error: DataTables warning: table id=tblReservas - Invalid JSON response. For more information about this error, please see datatables.net/tn/1 Commented el 3 nov. 2023 a las 11:47
  • Hola @ClaudioGonzalez. El mensaje "Invalid JSON response" indica que la respuesta obtenida no es un JSON. Lo mas seguro es que este ocurriendo un error. ¿Podes entrar al chat asi me pasas la consulta que esta usando?
    – Marcos
    Commented el 3 nov. 2023 a las 12:59

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.