0

Tengo una duda, quisiera guardar un UPDATE dentro de una variable, por ejemplo, este es mi código:

    public static function cancelTareasP(Usuario $_Usuario, $proyecto_id) {    

        $conex = $_Usuario->getConexBD();    

        $tareasPend = Tarea::getTareasPendientes($conex);
        $ids = array_column($tareasPend, 'id');    

        foreach ($ids as $id) {
            $conex->query("UPDATE Tareas SET estatus = 'Cancelada' WHERE proyecto_id = '$proyecto_id' AND id = '$id'");
        }

        return regHistorico($conex, 4, 'Tareas', $idTareasModificadas, $proyecto_id);
    }

Al momento de hacer el UPDATE se modifica el estatus de las tareas, esos cambios deben registrarse en un historial, mi duda es si se puede guardar ese UPDATE como una variable ($idTareasModificadas) que sea el array de los id de las tareas que se modificaron, para poder hacer el registro.

Espero me haya dado a entender.

2
  • Esto huele a tarea como para un Store procedure o un trigger.. podrias aclarar que base de datos estas usando?? y haciendo un select, podrias saber que columnas va a actualizar el update
    – gbianchi
    Commented el 2 ago. 2017 a las 18:06
  • ¿Cómo que cuál base de datos estoy usando? Commented el 2 ago. 2017 a las 19:00

2 respuestas 2

1

si deseas guardar en un historico los id de las tareas solamente debes colocar la función regHistorico dentro del foreach:

public static function cancelTareasP(Usuario $_Usuario, $proyecto_id) {    

    $conex = $_Usuario->getConexBD();    
    $tareasPend = Tarea::getTareasPendientes($conex);
    $ids = array_column($tareasPend, 'id');    

    foreach ($ids as $id) {
        $conex->query("UPDATE Tareas SET estatus = 'Cancelada' WHERE proyecto_id = '$proyecto_id' AND id = '$id'");
        regHistorico($conex, 4, 'Tareas', $id, $proyecto_id);

    }
}
7
  • El $id es el id de todas las tareas con estatus = 'Pendiente', y yo sólo quiero los id de las tareas que se han modificado dependiendo del proyecto($proyecto_id) que se haya seleccionado. Al poner el regHistorico dentro del foreach me arroja todos los id. Commented el 2 ago. 2017 a las 18:30
  • Pero si tienes el foreach ($ids as $id) la variable $ids es donde tienes almacenado todos los ids pendientes de las tareas y al colocar el as $id estas recorriendo id por id, también como veo tu actualización: UPDATE Tareas SET estatus = 'Cancelada' WHERE proyecto_id = '$proyecto_id' AND id = '$id'. solo estas actualizando un id dentro del foreach ¿Cuál es el problema que también ejecutes el histórico ahí adentro? solo guardaras el histórico para ese id. Es lo que puedo entenderte. Commented el 2 ago. 2017 a las 18:53
  • Pero ya lo he hecho así, con el regHistorico dentro del foreach y me arroja todos los id de las tareas pendientes, yo sólo quiero los id de las tareas a las que se les modificó el estatus con el UPDATE. Commented el 2 ago. 2017 a las 18:57
  • Y que variable estas consultando? $ids o $id. te recomiento hacer un echo "<pre>"; var_dump($ids); $exit(); para que veas que esa variable tiene todos los id y un echo "<pre>"; var_dump($id); $exit(); para que verifiques si solo tiene el id que tu necesitas actualizar y guardar en el histórico y nos cuentas. Commented el 2 ago. 2017 a las 19:00
  • Con echo "<pre>"; var_dump($ids); exit(); si me arroja todos los id de las tareas con estatus = 'Pendiente. Con echo "<pre>"; var_dump($id); exit(); esto es lo que me arroja: string(1) "1". Commented el 2 ago. 2017 a las 19:07
0

Podrías mejorar la función getTareasPendientes para obtener las tareas pendientes con el $proyecto_id; así te aseguras que esa función te retorne las tareas pendientes de dicho proyecto o ejecutas el query allí en sustitución de la función, ya que el problema radica en que obtienes todas las tareas pendientes, ejecutando el query allí traerías las que tengan estatus = 'Pendiente' y pertenezcan al proyecto_id.

$tareasPend = Tarea::getTareasPendientes($conex, $proyecto_id);
    o
$tareasPend = $conex->query("SELECT id FROM Tareas WHERE estatus = 'Pendiente' AND proyecto_id = '$proyecto_id'");   

foreach ($tareasPend as $id) {
    $conex->query("UPDATE Tareas SET estatus = 'Cancelada' WHERE id = '$id'");
    regHistorico($conex, 4, 'Tareas', $id, $proyecto_id);
}
3
  • Al hacerlo de esa forma no me cancela las tareas que debería y en el registro histórico me arroja el id de una tarea que ni siquiera está en el proyecto que seleccioné Commented el 2 ago. 2017 a las 19:38
  • el problema radica en que obtienes todas las tareas pendientes, ejecutando el query allí traerías las que tengan estatus = 'Pendiente' y pertenezcan al proyecto_id. $tareasPend = $conex->query("SELECT id FROM Tareas WHERE estatus = 'Pendiente' AND proyecto_id = '$proyecto_id'"); foreach ($tareasPend as $id) { $conex->query("UPDATE Tareas SET estatus = 'Cancelada' WHERE id = '$id'"); regHistorico($conex, 4, 'Tareas', $id, $proyecto_id); } Commented el 2 ago. 2017 a las 20:43
  • Ya lo tenía así, ejecutando el query, pero mi jefe me dijo que no era necesario porque ya tenía un método que me arrojaba todas las tareas pendientes, así que tuve que cambiarlo, pero si funciona así, sólo tuve que agregar otra línea de código ya que me marcaba array to string conversion: $tareasPend = $conex->query("SELECT id FROM Tareas WHERE estatus = 'Pendiente' AND proyecto_id = '$proyecto_id'"); foreach ($tareasPend as $ids) { $id = $ids['id']; $conex->query("UPDATE Tareas SET estatus = 'Cancelada' WHERE id = '$id'"); regHistorico($conex, 4, 'Tareas', $id, $proyecto_id); } Commented el 2 ago. 2017 a las 20: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.