Recorrer un array dentro de un foreach

[rogertm]

Hola por aquí, tengo el siguiente problema. Tengo una función que me devuelve de la base de datos un array con una lista de usuarios que inicialmente tienen en su estado el status de Pendiente (pending), ahora, lo que quiero es que el encargado de estos usuarios les pueda cambiar el estado a Aprovado, Desaprovado o Pendiente, dependiendo del estado anterior.

Los datos de los usuarios que quiero cambiar los guardo en $request_users, supongo que el error está en el foreach dentro del foreach, pero ahora lo que sucede es que se actualiza solo el último elemento del array que viene por $_GET, y necesito que se actualizen todos...

Código PHP:

// Algo así es lo que tengo de la Base de Datos
$notes = array(
    'user_1'    => array(
        'user'        => 1,
        'status'    => 'pending',
    ),
    'user_2'    => array(
        'user'        => 2,
        'status'    => 'pending',
    ),
    'user_3'    => array(
        'user'        => 3,
        'status'    => 'pending',
    ),
    'user_4'    => array(
        'user'        => 4,
        'status'    => 'pending',
    ),
    'user_5'    => array(
        'user'        => 5,
        'status'    => 'pending',
    ),
);
 
$status         = ( isset( $_REQUEST['status'] ) ) ? $_REQUEST['status'] : null;
$updated_users     = array();
$request_users    = array();
foreach ( $_REQUEST['user'] as $single_user ) :
    array_push( $request_users, $single_user );
endforeach;
foreach ( $notes as $key => $value ) :
    foreach ( $request_users as $single_user ) :
        if ( $value['user'] == $single_user ) :
            if ( $status == 'approved' || $status == 'pending' ) :
                $update_user = array(
                    $key     => array(
                        'user'        => $single_user,
                        'status'    => $status,
                    ),
                );
            elseif ( $status == 'disapprove' ) :
                $update_user = array();
            endif;
        else :
            $update_user = array(
                $key    => array(
                    'user'        => $value['user'],
                    'status'    => $value['status'],
                ),
            );
        endif;
    endforeach;
        $updated_users = array_merge( $updated_users, array_slice( $update_user, -1 ) );
endforeach;
 

Como siempre agradezco cualquier ayuda que me puedan brindar.

Saludos!

[Alexis88]

Hace poco se me presentó un caso similar en el trabajo, en el cual, tenía un array que contenías arrays de información (un array asociativo), con la particularidad de que, cada array hijo, contenía la información de todos los registros a actualizar y no los tenías por separado, en otras palabras, tenía algo como esto:

Código PHP:

Array(
    'nombre' => Array( //Nombres
        0 => 'Juan',
        1 => 'Luis',
        2 => 'José'
    ),
    'apellido' => Array( //Apellidos
        0 => 'Pérez',
        1 => 'Gonzáles',
        2 => 'Gómez'
    ),
    'id' => Array( //IDs
        0 => 25,
        1 => 3,
        2 => 159
    ),
    'usuario' => 123
)
 

Y como hace un tiempo leí que ejecutar consultas SQL en un bucle con PHP no era muy eficiente que digamos, decidí utilizar la sintaxis CASE WHEN THEN END, que es así:

Código MySQL:

UPDATE tabla 
SET 
    nombre = CASE id
        WHEN 25 THEN 'Juan'
        WHEN 3 THEN 'Luis'
        WHEN 159 THEN 'José'
    END, 
    apellido = CASE id
        WHEN 25 THEN 'Pérez'
        WHEN 3 THEN 'Gonzáles'
        WHEN 159 THEN 'Gómez'
    END
WHERE usuario = 123
 

De este modo, actualizo a los tres registros en los dos campos (nombre y apellido), siempre y cuando el usuario que esté realizando la edición, tenga por identificador el '123'.

Para formar la cadena de consulta con PHP, utilicé un bucle con el cual recorrí al primer hijo del bucle padre, excepto al último pues no es un array. Antes de esto, creé dos variables en las cuales iré concatenando los valores. En cada iteración, concateno en cada variable los valores, más o menos así:

Código PHP:

//Todos los datos fueron desinfectados antes de llegar a este punto y los extraje del array padre con la función extract()
//El array con los nombres se llama $nombre y el de apellidos, $apellido
$consulta = 'UPDATE tabla SET ';
$nombres = 'nombre = CASE id ';
$apellidos = 'END, apellido = CASE id ';
$condicion = 'WHERE usuario = ' . $usuario;
for ($i = 0, $l = count($nombre); $i < $l; $i++){
    $nombres .= "WHEN " . $id[$i] . " THEN '" . $nombre[$i] . "' ";
    $apellidos .= "WHEN " . $id[$i] . " THEN '" . $apellido[$i] . "' ";
}
$consulta .= $nombres . $apellidos . ' END ' . $condicion;
mysqli_query($conexion, $consulta) or exit ('Se produjo un error');
 

Ojalá pueda serte de ayuda.

Saludos

[joaquincreatividad]

por que no haces una consulta simple selecionado los eelemntos que tenga tal estado y cambiarlos de golpe?

bueno yo haria una consulta asi:

selecionar todos los elementos de la tabla Nombre donde estado sea igual a "Pendiente"

y ya con un mysql fetch array los recupero todos bueno eso haria yo

[rogertm]

A ver chicos, agradezco la ayuda, pero la cosa es básicamente de php, lo seguro es que no me expliqué bien antes.

Obtengo un array de la base de datos y lo que necesito es basándome en ese array, construir uno nuevo, y ese nuevo es el que inserto en la base de datos mediante una función.

El código es simple en la manera que lo planteo aquí, porque mi duda es pura de php, pero es un pedazo de código de una función de WordPress, osea, ya tengo la función que me da el array de la base de datos, y tengo otra que hace el update al registro.

En fin y básicamente lo que necesito es: por cada iteración que hago al array que viene de la BD, comparar cada usuario que viene por $_GET (el foreach dentro del foreach) y si es el mismo, pues actualizar su estado (status), una vez comparado todo, pues construyo un nuevo array y este es el que inserto en el registro... Pero ahora como está mi código, solo actualiza el último elemento del array que obtengo por $_GET...

Si ponen ese código en un archivo y acceden via web con una url del tipo ?user=1?user=2?user=3?status=approved y hacen un print_r($notes) y print_r($updated_users) se darán cuenta de lo que digo...

Quiero cambiar el estado a todos los usuarios que vienen por $_GET

Gracias nuevamente...

[Alexis88]

Fíjate que, en el ejemplo que te muestro, también utilizo un array asociativo y realizo actualizaciones según dos datos. En tu caso que solo deseas cambiar el estado, en lugar de formar dos grupos de casos, solo utilizarías uno.

Saludos

[IsaBelM]

es un problema de sobre-escritura. prueba ahora

$status = ( isset( $_REQUEST['status'] ) ) ? $_REQUEST['status'] : null;
$updated_users = array();
$request_users = array();
foreach ( $_REQUEST['user'] as $single_user ) :
array_push( $request_users, $single_user );
endforeach;


foreach ( $notes as $key => $value ) :
for ($i = 0; $i < sizeof($request_users); $i++) :

if ( $value['user'] == $request_users[$i] ) :

if ( $status == 'approved' || $status == 'pending' ) :
$update_user = array(
$key => array(
'user' => $request_users[$i],
'status' => $status,
),
);
break;
elseif ( $status == 'disapprove' ) :
$update_user = array();
endif;
else :

$update_user = array(
$key => array(
'user' => $value['user'],
'status' => $value['status'],
),
);
endif;
endfor;
$updated_users = array_merge( $updated_users, array_slice( $update_user, -1 ) );
endforeach;

pd: Alexis88 me ha gustado la consulta desconectada

[rogertm]

Bueno, debo agradecer, literalmente me han resuelto el problema...

Finalmente el código quedó así:

Código PHP:

$notes = array(
    'user_1'    => array(
        'user'        => 1,
        'status'    => 'pending',
    ),
    'user_22'    => array(
        'user'        => 22,
        'status'    => 'pending',
    ),
    'user_13'    => array(
        'user'        => 13,
        'status'    => 'pending',
    ),
    'user_4'    => array(
        'user'        => 4,
        'status'    => 'pending',
    ),
    'user_512'    => array(
        'user'        => 512,
        'status'    => 'pending',
    ),
);
 
$status         = ( isset( $_REQUEST['status'] ) ) ? $_REQUEST['status'] : null;
$updated_users     = array();
foreach ( $notes as $key => $value ) :
    for ($i = 0; $i < count($_REQUEST['user']); $i++) :
        if ( $value['user'] == $_REQUEST['user'][$i] ) :
            if ( $status == 'approved' || $status == 'pending' ) :
                $update_user = array(
                    $key     => array(
                        'user'        => $_REQUEST['user'][$i],
                        'status'    => $status,
                    ),
                );
            elseif ( $status == 'disapprove' ) :
                $update_user = array();
            endif;
            break;
        else :
            $update_user = array(
                $key    => array(
                    'user'        => $value['user'],
                    'status'    => $value['status'],
                ),
            );
        endif;
    endfor;
        $updated_users = array_merge( $updated_users, array_slice( $update_user, -1 ) );
endforeach;
 

Cambié la función sizeof() por count() por una cuestión pura de manía, y cambié el break al final de la sentencia if... pues si el estado es disapprove debo eliminar el usuario del registro en la BD

Para ampliar un poquito, y matar la curiocidad de quien la padezca, cuando me pasan este tipo de cosas trabajando con WP, trato de simplificar el código y lo trabajo en un archivo php simple y sato, que es lo que he mostrado aquí, pero también les traigo la función original, por si hace algún bien a alguien

Código PHP:

function humi_mentor_update_multi_supported_users(){
    $status            = ( isset( $_REQUEST['status'] ) ) ? $_REQUEST['status'] : null;
    $current_users    = get_user_meta( get_current_user_id(), 'humi_mentor_supported_users', true );
    $updated_users    = array();
 
    // Check for multi user update
    $multi_nonce    = ( isset( $_REQUEST['update_bulk_users_field'] ) ) ? $_REQUEST['update_bulk_users_field'] : null;
    if ( ! wp_verify_nonce( $multi_nonce, 'update_bulk_users_attr' ) ) return;
 
    if ( wp_verify_nonce( $multi_nonce, 'update_bulk_users_attr' ) ) :
        // Check if there is something to do
        if ( $status == 'null' ) :
            $redirect = apply_filters( 'humi_mentor_update_supported_users_fail', add_query_arg( array( 'update-multi-user' => 'false' ), get_permalink() ) );
            wp_safe_redirect( $redirect );
            exit();
        endif;
        foreach ( $current_users as $key => $value ) :
            for ( $i = 0; $i < count( $_REQUEST['user'] ); $i++ ) :
                if ( $value['user'] == $_REQUEST['user'][$i] ) :
                    if ( $status == 'approved' || $status == 'pending' ) :
                        $update_user = array(
                            $key     => array(
                                'user'        => $_REQUEST['user'][$i],
                                'status'    => $status,
                            ),
                        );
                    elseif ( $status == 'disapprove' ) :
                        $update_user = array();
                        delete_user_meta( $value['user'], 'humi_user_mentor' );
                    endif;
                    break;
                else :
                    $update_user = array(
                        $key    => array(
                            'user'        => $value['user'],
                            'status'    => $value['status'],
                        ),
                    );
                endif;
            endfor;
            $updated_users = array_merge( $updated_users, array_slice( $update_user, -1 ) );
        endforeach;
        update_user_meta( get_current_user_id(), 'humi_mentor_supported_users', $updated_users );
 
        $redirect = apply_filters( 'humi_mentor_update_supported_users_success', add_query_arg( array( 'update-multi-user' => 'true' ), get_permalink() ) );
        wp_safe_redirect( $redirect );
        exit();
    endif;
}
add_action( 'template_redirect', 'humi_mentor_update_multi_supported_users' );
 

Gracias nuevamente por la ayuda y la prontitud...