Skip to main content
se añadió 1 carácter en el cuerpo
Origen Enlace
Alfabravo
  • 9.9k
  • 5
  • 24
  • 32

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut likeLIKE '%:p0%'p0 LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET['term']"%$_GET['term']%";

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($result, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut like '%:p0%' LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET['term']

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($result, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut LIKE :p0 LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = "%$_GET['term']%";

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($result, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

se añadieron 2 caracteres en el cuerpo
Origen Enlace
Alfabravo
  • 9.9k
  • 5
  • 24
  • 32

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut like '%:p0%' LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET['term']

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($stid$result, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut like '%:p0%' LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET['term']

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($stid, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut like '%:p0%' LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET['term']

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($result, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

se añadieron 5 caracteres en el cuerpo
Origen Enlace
Alfabravo
  • 9.9k
  • 5
  • 24
  • 32

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut like '%:p0%' LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET...$_GET['term']

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($stid, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut like '%:p0%' LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET...

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($stid, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

En la clase donde ejecutas las consultas Oracle te falta ejecutar la consulta con oci_execute.

Actualmente tienes

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   while($row=oci_fetch_array($result)) {
   $resultset[] = $row;
   }
   if(!empty($resultset))
        return $resultset;
   }
}

Ahí recibes la consulta, le haces el oci_parse por si toca prepararla y eso. Luego haces un fetch pero pues... no has ejecutado nada. Va a ser vacío y el error en el foreach es porque estás intentando iterar sobre una variable nula.

function runQuery($query) {
   $result = oci_parse($this->conn,$query);
   oci_execute($result);

   while($row=oci_fetch_array($result)) {
       $resultset[] = $row;
   }
   if(!empty($resultset))
       return $resultset;
   }
   // Yo controlaría el else y retornaría algo predecible si el arreglo está vacío...
}

EDITO Yo, además, no haría esa concatenación del GET directo en la consulta. Eso es vulnerable a SQL injection y es una pésima práctica. De paso, puede que tengas problemas con el contenido del GET porque no esté escapado o meta carácteres raros que rompan la consulta y generen todo este lío.

Podrías seguir la documentación y poner los parámetros de una mejor forma.

Tendrías que editar la página alumno cambiando la consulta pa usar placeholders. Parámetros. Así no concatenas nada.

$sql = "SELECT nombres, rut, apellidos, correo FROM alumno WHERE rut like '%:p0%' LIMIT 5";

Y crear un array con los parámetros de la consulta

$arrayParams[0] = $_GET['term']

Para entregárselo a la ejecución de la consulta

$faq = $db_handle->runQuery($sql, $arrayParams);

Luego, enviarle el array de parámetros al método de tu manejador de conexión

function runQuery($query, $paramArray) {
...
    $result = oci_parse($this->conn,$query);
    oci_bind_by_name($stid, ':p0', $paramArray[0]);
    oci_execute($result);
    ...

Y usar el binding pa meter los parámetros de forma segura. De paso, te ahorras líos con caracteres no escapados y cosas horrendas

se añadieron 232 caracteres en el cuerpo
Origen Enlace
Alfabravo
  • 9.9k
  • 5
  • 24
  • 32
Loading
se añadieron 811 caracteres en el cuerpo
Origen Enlace
Alfabravo
  • 9.9k
  • 5
  • 24
  • 32
Loading
se añadieron 58 caracteres en el cuerpo
Origen Enlace
Alfabravo
  • 9.9k
  • 5
  • 24
  • 32
Loading
Origen Enlace
Alfabravo
  • 9.9k
  • 5
  • 24
  • 32
Loading