Skip to main content
se añadieron 2766 caracteres en el cuerpo
Origen Enlace
Diego Saravia
  • 632
  • 1
  • 6
  • 16

A simple vista puedo ver que tu error está enPara poder darle solución a este planteamiento he tenido que estás creando tumodificar algunas cosas y escribir algo más de código:

  1. He vuelto a reestructurar el código HTML ya que estaba muy mal organizado (deberías de repasar como estructurar bien el código HTML.

  2. Lo que hice fue cambiar el valor de los input por el id del dato que está recorriendo el foreach() en ese momento. Con esto consigo poder obtener los id de las fechas de una manera más fácil a la hora de mandarlos a la consulta UPDATE.

  3. He añadido un <input type="hidden"> para poder obtener también el id de la fecha que actualmente tiene el estado activo (1). Esto con el fin de actualizar este valor a 0 cuando se realice la consulta ya que no puedes tener dos fechas con estado 1.

4)En los execute() de cada consulta preparadaUPDATE he cambiado los (aunquebindParam() por un poco insegura) pero nuncaarray dentro de la estás ejecutandomisma función execute(). En este código puedo verEsto lo hice ya que es más seguro usar un array, hacen la instrucciónmisma función que te falta es ellos $d->executebindParam([]); y además de, con los bindParam, hay datos que debes de incluir el archivo donde tienes la conexión en donde estás realizandotienen cierta "restricción" a la consulta. Ademáshora de estose mandados. Por ejemplo, te recomiendo que al final de la consulta cierres el cursor para no gastar recursos del usuario y que pongas todo esto dentro de un try{}catch(PDOException $e){}deja mandar números como 0 o 1.

  1. Añadí try{}catch(){} para poder manejar mejor algún error futuro.

El código php al finalde los archivos quedaría asíde la siguiente manera:

index.php:

<?php 
    
    include 'conexión.php';

    $response = $conn->prepare("SELECT * FROM fecha");
    $response->execute();

 ?>
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Document</title>
</head>
<body>
    <form action="fecha_php.php" method="POST">
        <table class="pure-table pure-table-horizontal">
            <thead>
                <tr>
                    <th>Fecha</th>
                    <th>Estado</th>
                </tr>
            </thead>
            <tbody>
            <?php foreach ($response as $t): ?>
                <tr>
                    <td><?php echo $t['numero_fecha']; ?></td>
                    <td>
                    <?php if ($t['estado'] == 1): ?>
                        <input type="radio" name="estado" checked value="<?php echo $t['id_fecha']; ?>" />
                        <input type="hidden" name="idEstadoAnterior" value="<?php echo $t['id_fecha'] ?>">
                    <?php else: ?>
                        <input type="radio" name="estado" value="<?php echo $t['id_fecha']; ?>" />
                    <?php endif ?>
                    </td>
                </tr>
            <?php endforeach ?>
            </tbody>
        </table>
        <input type="submit" value="actualizar" name="submit"/>
    </form>
</body>
</html>

conexion.php:

<?php
    $server = 'localhost';
    $username = 'root';
    $password = '';
    $database = 'prode-login';
    try {
      $conn = new PDO("mysql:host=$server;dbname=$database;", $username, $password);
    } catch (PDOException $e) {
      die('Conexión fracasada: ' . $e->getMessage());
    }
 ?>

fecha_php.php:

<?php 

    include 'conexión.php';

    $idFecha = $_POST['estado'];

    try {
        $add = "UPDATE fecha SET estado=:estado WHERE id_fecha=:id_fecha";
        $d = $conn->prepare($add);
        $d->bindParam>execute('[
            ":id_fecha'id_fecha" => $idFecha, 
 $_POST['id_fecha']           ":estado" => "1"
        ]);
    } $dcatch (PDOException $e) {
        die("Error: " . $e->bindParam>getMessage(') . "<br>on line:estado', $_POST['estado']" . $e->getLine() . "<br>on file: " . $e->getFile());
    }

    if(isset($_POST['idEstadoAnterior'])){
        $idFechaAnterior = $_POST['idEstadoAnterior'];

        try {
            $add = "UPDATE fecha SET estado=:estado WHERE id_fecha=:id_fecha";
            $d = $conn->execute>prepare([]$add);
            $d->closeCursor>execute([
                ":id_fecha" => $idFechaAnterior,
                ":estado" => "0"
            ]);
        } catch (PDOException $e) {
            die("Error: " . $e->getMessage() . "<br>on line: " . $e->getLine() . "<br>on file: " . $e->getFile());
        }
    }

    header("Location: index.php");
 ?>

Prueba con ese código y si no funciona, estoy atento a resolver cualquier otro problema que se te presente

A simple vista puedo ver que tu error está en que estás creando tu consulta preparada (aunque un poco insegura) pero nunca la estás ejecutando. En este código puedo ver que la instrucción que te falta es el $d->execute([]); además de que debes de incluir el archivo donde tienes la conexión en donde estás realizando la consulta. Además de esto, te recomiendo que al final de la consulta cierres el cursor para no gastar recursos del usuario y que pongas todo esto dentro de un try{}catch(PDOException $e){}. El código php al final quedaría así:

 try {
     $add = "UPDATE fecha SET estado=:estado WHERE id_fecha=:id_fecha";
     $d = $conn->prepare($add);
     $d->bindParam(':id_fecha', $_POST['id_fecha']);
     $d->bindParam(':estado', $_POST['estado']);
     $d->execute([]);
     $d->closeCursor();
 } catch(PDOException $e){
     die("Error: " . $e->getMessage() . "<br>on line: " . $e->getLine() . "<br>on file: " . $e->getFile());
 }

Prueba con ese código y si no funciona, estoy atento a resolver cualquier otro problema que se te presente

Para poder darle solución a este planteamiento he tenido que modificar algunas cosas y escribir algo más de código:

  1. He vuelto a reestructurar el código HTML ya que estaba muy mal organizado (deberías de repasar como estructurar bien el código HTML.

  2. Lo que hice fue cambiar el valor de los input por el id del dato que está recorriendo el foreach() en ese momento. Con esto consigo poder obtener los id de las fechas de una manera más fácil a la hora de mandarlos a la consulta UPDATE.

  3. He añadido un <input type="hidden"> para poder obtener también el id de la fecha que actualmente tiene el estado activo (1). Esto con el fin de actualizar este valor a 0 cuando se realice la consulta ya que no puedes tener dos fechas con estado 1.

4)En los execute() de cada consulta UPDATE he cambiado los bindParam() por un array dentro de la misma función execute(). Esto lo hice ya que es más seguro usar un array, hacen la misma función que los bindParam() y además, con los bindParam, hay datos que tienen cierta "restricción" a la hora de se mandados. Por ejemplo, no deja mandar números como 0 o 1.

  1. Añadí try{}catch(){} para poder manejar mejor algún error futuro.

El código de los archivos quedaría de la siguiente manera:

index.php:

<?php 
    
    include 'conexión.php';

    $response = $conn->prepare("SELECT * FROM fecha");
    $response->execute();

 ?>
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Document</title>
</head>
<body>
    <form action="fecha_php.php" method="POST">
        <table class="pure-table pure-table-horizontal">
            <thead>
                <tr>
                    <th>Fecha</th>
                    <th>Estado</th>
                </tr>
            </thead>
            <tbody>
            <?php foreach ($response as $t): ?>
                <tr>
                    <td><?php echo $t['numero_fecha']; ?></td>
                    <td>
                    <?php if ($t['estado'] == 1): ?>
                        <input type="radio" name="estado" checked value="<?php echo $t['id_fecha']; ?>" />
                        <input type="hidden" name="idEstadoAnterior" value="<?php echo $t['id_fecha'] ?>">
                    <?php else: ?>
                        <input type="radio" name="estado" value="<?php echo $t['id_fecha']; ?>" />
                    <?php endif ?>
                    </td>
                </tr>
            <?php endforeach ?>
            </tbody>
        </table>
        <input type="submit" value="actualizar" name="submit"/>
    </form>
</body>
</html>

conexion.php:

<?php
    $server = 'localhost';
    $username = 'root';
    $password = '';
    $database = 'prode-login';
    try {
      $conn = new PDO("mysql:host=$server;dbname=$database;", $username, $password);
    } catch (PDOException $e) {
      die('Conexión fracasada: ' . $e->getMessage());
    }
 ?>

fecha_php.php:

<?php 

    include 'conexión.php';

    $idFecha = $_POST['estado'];

    try {
        $add = "UPDATE fecha SET estado=:estado WHERE id_fecha=:id_fecha";
        $d = $conn->prepare($add);
        $d->execute([
            ":id_fecha" => $idFecha, 
            ":estado" => "1"
        ]);
    } catch (PDOException $e) {
        die("Error: " . $e->getMessage() . "<br>on line: " . $e->getLine() . "<br>on file: " . $e->getFile());
    }

    if(isset($_POST['idEstadoAnterior'])){
        $idFechaAnterior = $_POST['idEstadoAnterior'];

        try {
            $add = "UPDATE fecha SET estado=:estado WHERE id_fecha=:id_fecha";
            $d = $conn->prepare($add);
            $d->execute([
                ":id_fecha" => $idFechaAnterior,
                ":estado" => "0"
            ]);
        } catch (PDOException $e) {
            die("Error: " . $e->getMessage() . "<br>on line: " . $e->getLine() . "<br>on file: " . $e->getFile());
        }
    }

    header("Location: index.php");
 ?>
Origen Enlace
Diego Saravia
  • 632
  • 1
  • 6
  • 16

A simple vista puedo ver que tu error está en que estás creando tu consulta preparada (aunque un poco insegura) pero nunca la estás ejecutando. En este código puedo ver que la instrucción que te falta es el $d->execute([]); además de que debes de incluir el archivo donde tienes la conexión en donde estás realizando la consulta. Además de esto, te recomiendo que al final de la consulta cierres el cursor para no gastar recursos del usuario y que pongas todo esto dentro de un try{}catch(PDOException $e){}. El código php al final quedaría así:

 try {
     $add = "UPDATE fecha SET estado=:estado WHERE id_fecha=:id_fecha";
     $d = $conn->prepare($add);
     $d->bindParam(':id_fecha', $_POST['id_fecha']);
     $d->bindParam(':estado', $_POST['estado']);
     $d->execute([]);
     $d->closeCursor();
 } catch(PDOException $e){
     die("Error: " . $e->getMessage() . "<br>on line: " . $e->getLine() . "<br>on file: " . $e->getFile());
 }

Prueba con ese código y si no funciona, estoy atento a resolver cualquier otro problema que se te presente