El motivo por el que no obtienes la información que deseas en $acc2pay['value']
, $acc2pay['user_id']
y $acc2pay['ptsu_id']
es debido a que estás tratando de hacer un PDOStatement::fetch()
de una consulta preparada que no ha sido ejecutada aún con PDOStatement::execute()
, sólo preparada con PDOStatement::prepare()
.
La solución más fácil a tu problema sería cambiar el PDO::prepare()
por un PDO::query()
como el resto de consultas.
No hay que olvidar usar PDO::quote()
para no sufrir los graves problemas de seguridad asociados a la inyección SQL al concatenar cadenas de texto a consultas SQL:
<?php
/* Realizamos la consulta (usando PDO::quote) */
$acc2pay = $conexion->query("
SELECT
user_id, value, ptsu_id
FROM ptsu_requests
WHERE id = " . $conexion->quote($mid) . "
");
/* Comprobamos si hubo un error en la consulta */
if ($acc2pay === false) {
/* Obtenemos los detalles del error */
$errores = $conexion->errorInfo();
/* Mostramos el texto del error */
die('Error SQL: ' . htmlspecialchars($errores[2]));
}
/* Obtenemos el primer y único registro (si existe) */
$acc2pay = $acc2pay->fetch();
/* Comprobamos si existe o no algún registro con dicho "id" */
if ($acc2pay === false) {
die('No existe ningún registro con ese "id"');
}
He controlado si $acc2pay
vale false
. En ese caso significaría que no había ningún registro con ese identificador.
También he comprobado si query()
devuelve false
. En ese otro caso significaría que hubo un error o problema con la consulta.
Además, como bien sugiere @A. Cedano, deberías usar consultas preparadas en vez de abusar del uso de PDO::quote()
.
Aquí tienes un ejemplo de cómo quedaría tu código usando consultas preparadas y comprobado la salida de cada consulta:
<?php
try {
$conexion = new PDO("mysql:host=$servidor;dbname=$db", $usuario, $contrasena);
} catch (PDOException $e) {
echo "There was an unexpected error. Please try again, or contact us with concerns";
}
$mid = '2597';
/* Realizamos la consulta (usando consultas preparadas) */
$acc2pay = $conexion->prepare("
SELECT
user_id, value, ptsu_id
FROM ptsu_requests
WHERE id = :id
");
/* Ejecutamos la consulta asignando el valor adecuado a :id y
comprobamos si hubo un error durante la ejecución de la consulta */
if (
$acc2pay->execute([
':id' => $mid,
]) === false
) {
/* Obtenemos los detalles del error */
$errores = $acc2pay->errorInfo();
/* Mostramos el texto del error */
die('Error SQL: ' . htmlspecialchars($errores[2]));
}
/* Obtenemos el primer y único registro (si existe) */
$acc2pay = $acc2pay->fetch();
if ($acc2pay === false) {
die('No existe ningún registro con ese "id"');
}
$upd = $conexion->prepare("
UPDATE ptsu_requests
SET status = 'Completed'
WHERE id = :id
");
if (
$upd->execute([
':id' => $mid,
]) === false
) {
/* Obtenemos los detalles del error */
$errores = $acc2pay->errorInfo();
/* Mostramos el texto del error */
die('Error SQL: ' . htmlspecialchars($errores[2]));
}
$upd = $conexion->query("
UPDATE members
SET money = money + :value
WHERE id = :id
");
if (
/* OJO: Esta consulta requiere definir :value e :id */
$upd->execute([
':value' => $acc2pay['value'],
':id' => $mid,
]) === false
) {
/* Obtenemos los detalles del error */
$errores = $acc2pay->errorInfo();
/* Mostramos el texto del error */
die('Error SQL: ' . htmlspecialchars($errores[2]));
}
$upd = $conexion->query("
UPDATE ptsu_offers
SET approved = approved + 1, pending = pending - 1
WHERE id = :id
");
if (
$upd->execute([
':id' => $mid,
]) === false
) {
/* Obtenemos los detalles del error */
$errores = $acc2pay->errorInfo();
/* Mostramos el texto del error */
die('Error SQL: ' . htmlspecialchars($errores[2]));
}
Se podrían evitar tantos bloques if/then/else
con un único bloque try/catch
activando la generación de excepciones en caso de error con $conexion->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION)
.
Por último, quizá deberías realizar las actualizaciones dentro de una transacción para que todo se realice o no en bloque. Consulta la documentación de PDO::beginTransaction()
y PDO::commit()
para más información.
fetch
de unprepare
sin haber ejecutado la consulta con unexecute
. Además, no deberías concatenar cadenas de caracteres a consultas SQL, para eso están precisamente las consultas preparadas (usandoprepare
).