Notice : Undefined index: id line 4

jdvd

Hello, I am in a problem and it is that it appears to me that I have indefinite id , I have actually been able to solve it. I give you my code waiting for advice. Thank you

https://github.com/jdvd2431/codigo.git

ginerjm

Since your error message tells you the line number how about posting the code from that area right here? Maybe 20-30 lines at most? Maybe the real error message as well?

Nachfolger

Your GitHub repository is private, no one but you can see the code. I'm also confused, you said you HAVE been able to solve it, so do you still need help?

jdvd

Nachfolger https://github.com/jdvd2431/codigo-problema.git

ginerjm

I think the OP is gone. I answered right after he posted and haven't heard back

jdvd

Notice
: Undefined index: id in
C:\xampp\htdocs\Proyecto\html\cuentas\consultar_c\gestionar\modificar.php
on line
4

<?php

$consulta = ConsultarCuenta($_GET['id']);
function ConsultarCuenta($id){
include 'conexion.php';
$sentencia ="SELECT * FROM cuenta WHERE id='$id'";
$resultado = $conexion->query($sentencia);
$fila = $resultado->fetch(PDO::FETCH_ASSOC);
return [
$fila['id'],
$fila['id_cuenta'],
$fila['fecha_de_pago'],
$fila['fecha_de_vencimiento de pago'],
$fila['valor_a_pagar'],
$fila['periodo_de_cuenta'],
$fila['estado_de_cuenta'],
$fila['registro_de_residente']
];
}
?>

ginerjm

And WHICH is line 4? I'm going to guess it's the line with the $_GET which means you either aren't going a GET or you didn't define a field name of 'id'.

jdvd

if that's the eror, I send that varible by the get, look at the code

<td><a class="gestionarm" href="modificar.php?id='.$fila['id'].'">Modificar</td>

ginerjm

Too many quotes. The parm needs to be broken out with double quotes and NOT have any quotes surrounding it. Try building the actual value use the fila value and put that var in the href

jdvd

already coregi and still shows me the same error, you don't have another method ?

NogDog

This would not be the final code I'd want to deploy, but for some quick debugging, let's get all defensive-coding about it:

<?php
ini_set('display_errors', true);
error_reporting(E_ALL);

if(empty($_GET['id'])) {
  die("<pre>ERROR: 'id' not set.</pre>");
}

$consulta = ConsultarCuenta($_GET['id']);
if($consulta == false) {
  die("<pre>Nothing returned by query</pre>");
}

function ConsultarCuenta($id)
{
  require 'conexion.php'; // it's not optional, so `require` it
  $sentencia = "SELECT * FROM cuenta WHERE id='$id'";
  $resultado = $conexion->query($sentencia);
  if($resultado == false) {
    die("<pre>QUERY ERROR:\n".print_r($conexion->errorInfo(), true)."\nQuery:\n$$sentencia</pre>");
  }
  $fila = $resultado->fetch(PDO::FETCH_ASSOC);
  return $fila;
}

My wild guess is that cuenta.id is an integer, not a string, in which case it should not be quoted in the SQL. This is where using prepared statements can automate some of that for you while sanitizing the inputs.:

function ConsultarCuenta($id)
{
  require 'conexion.php'; // it's not optional, so `require` it
  $sentencia = "SELECT * FROM cuenta WHERE id = :id";
  $stmt = $conexion->prepare($sentencia);
  if($stmt == false) {
    die("<pre>INVALID QUERY:\n".print_r($conexion->errorInfo())."\nSQL:\n$sentencia</pre>");
  }
  $resultado = $stmt->execute($stmt, array('id' => $id));
  if($resultado == false) {
    die("<pre>QUERY ERROR:\n".print_r($stmt->errorInfo(), true)."\nQuery:\n$$sentencia</pre>");
  }
  $fila = $resultado->fetch(PDO::FETCH_ASSOC);
  return $fila;
}

(Warning: all code is untested.)

jdvd

look use this code and the only error that was medium in the inputs part Notice: Undefined variable: query in <b>C:-xampp-htdocs-Project-html-accounts-query c-manage-modify.php</b> online 600

when I click the page button he sends me this message 'ERROR: 'id' not set'.

NogDog

<?php
ini_set('display_errors', true);
error_reporting(E_ALL);

if (!isset ($_GET ['id'])){
die("<pre>ERROR: 'id' not set.</pre>");
}

$consulta = ConsultarCuenta($_GET['id']);
if($consulta == false) {
die("<pre>Nothing returned by query</pre>");
}

function ConsultarCuenta($id)
{
require 'conexion.php'; // it's not optional, so require it
$sentencia = "SELECT * FROM cuenta WHERE id='$id'";
$resultado = $conexion->query($sentencia);
if($resultado == false) {
die("<pre>QUERY ERROR:\n".print_r($conexion->errorInfo(), true)."\nQuery:\n$$sentencia</pre>");
}
$fila = $resultado->fetch();
return $fila;
}
?>

ginerjm

Show us your corrected code please. Work with us.

jdvd

already correji the code

`<?php

ini_set('display_errors', true);
error_reporting(E_ALL);

if(empty($_GET['id'])) {
die("<pre>ERROR: 'id' not set.</pre>");
}

$consulta = ConsultarCuenta($_GET['id']);
if($consulta == false) {
die("<pre>Nothing returned by query</pre>");
}

ginerjm

I think the better approach should be

if (!isset($_GET['id']))

jdvd

I was proving and I gave the modficar button and I died the page

ginerjm

huh?

jdvd

I put if (! isset ($GET ['id'])) and the page also died

ginerjm

Died? With no error message? If you are going to work with us you HAVE to give us more information. Or did you mean the page did not run at all? In that case SHOW us the revised code with this change in it. PLEASE!!!

jdvd

line 60