В чем моя ошибка создания meta_value из множества массивов посредством update_user_meta?

Question

[Алексей]

Задача:
По клику submit выполнять update_user_meta ($user_id, 'list', $list_array);
Каждый новый submit это новый обьект, который добавляется в meta_key(Array).

Пример:
meta_key: 'list'
meta_value:
[
// submit1
1 {'key1': 'value1'},
// submit2
2 {'key2': 'value2'}
// submit3
3 {'key3': 'value4'}
....
]

Мой код:

function wdc_addclient() {
    $user_id = get_current_user_id();
    
    $list = $_POST['list'];
        
    if( isset($list) ) {
        update_user_meta( $user_id, 'list', $list);
    }
}

Каждый сабмит перезаписывает предыдущее значение.
ВОПРОС:
Как реализовать этот код чтобы он учитывал предыдущие значения и добавлял новое?
С тем учетом что я добавлю еще кнопку редактирование каждого из пунктов списка.

Answer 1

[Александр Соболев]

Алексей, давайте разберем Ваш код:

$list = $_POST['list'];  Присваиваем "листу" новое, пришедшее значение
f( isset($list) ) {  update_user_meta( $user_id, 'list', $list); // и если оно есть, а оно уже есть - вы его присвоили. Даже пустое, но есть. Даже undifined.. вы его переписываетe.

Если лист - это массив, то перед сохранением Вам нужно добавить значение в массив. Таким образом получим:
array_push($list, $_POST['list']);
Добавим проверку на существование и "не пустоту":

if (isset($_POST['list']) || !empty($_POST['list'])) {

Ну.. и извлечем лист, к которому нужно добавить новое значение:

$list = get_user_meta($user_id, 'list', false/*ибо массив*/);

Итого получим что-то вроде:

if (isset($_POST['list']) || !empty($_POST['list'])) { 
    $list = get_user_meta($user_id, 'list', false);
    $new_list = array_push($list, $_POST['list']);
    update_user_meta( $user_id, 'list', $new_list);
}

Comments

[Алексей]

почему то
print_r($new_list);
выдает значение 2 все время;

function wdc_addclient() {
    $user_id = get_current_user_id();
    
    if (isset($_POST['client']) || !empty($_POST['client'])) { 
        $clients = get_user_meta($user_id, 'clients', false);
        $new_list = array_push($clients, $_POST['client']);
        update_user_meta( $user_id, 'clients', $new_list);
    }
    print_r($new_list); // 2
    print_r($clients); // array 'client'
//    wp_safe_redirect( esc_url( site_url('/my-account/clients/') ) );
	exit();
}

[Алексей]

Александр Соболев ,
а вот как выглядит форма

<form name="addclientform" id="addclientform" action="<?php echo esc_url( admin_url('admin-post.php') ); ?>" method="post">
<input type="hidden" name="action" value="addclient_form">
<input type="text" name="client[initials]" id="client_initials" class="input" value="" placeholder="ФИО">
<input type="text" name="client[city]" id="client_сity" class="input" value="" placeholder="Город">
<input type="text" name="client[post]" id="client_post" class="input" value="" placeholder="НП№">
<input type="text" name="client[phone]" id="client_phone" class="input" value="" placeholder="Номер телефона">
<input type="submit" name="wp-submit" id="wp-submit" class="button button-primary button-large" value="Submit" />
</form>

Внутри client находится вот это (вроде все правильно)

Array ( [0] => 2 [1] => Array ( [initials] => 123 [city] => 123 [post] => 123 [phone] => 123 ) )

[Александр Соболев]

Структуру Ваших массивов увидеть бы.. такая вложенность действительно нужна?

[Алексей]

Александр Соболев ,

вот данные из meta_value:

a:4:{s:8:"initials";s:6:"ФИО";s:4:"city";s:10:"ГОРОД";s:4:"post";s:8:"НП№1";s:5:"phone";s:13:"[phone]";}

Задумка такая:
У пользователя есть список его клиентов.
Он будет сам обновлять этот список.

На сколько я понял, есть несколько вариантов хранения информации
1) в таблице wp_usermeta в столбце meta_value: Array(arr1,arr2,arr3);
2) создать новую таблицу с ключем user_id и в этой таблице хранить данные его клиентов;
____
На мой вгляд 1) вариант вполне подходит

Спасибо за то что вникли в вопрос, я фронтом занимаюсь, но иногда бывает что нужно делать бекенд (php).

[Александр Соболев]

Алексей, клиенты - пользователи системы? если да, то заполнение данных клиентами происходит отдельно, с user_meta, а привязка клиентов к пользователю по id отдельно.
если нет, то формируйте массив пользователя примерно так:
клиенты[клиент1["фио": данные, "группакрови":данные итд], клиент2["фио": данные, "группакрови":данные итд], и так по всем клиентам]
внимательно смотрите приходят ли и в каком виде приходят данные с формы, будь-то Ajax(предпочтительнее) или хук при нажатии

[Алексей]

Александр Соболев ,
Клиенты не пользователи системы поэтому сформирую по втором варианту.
Разобрался в чем была проблема:

$new_list // возвращал длину внутри массива.
$clients[] = $_POST['client']; // Решил воспользоваться этой конструкцией

Еще был нюанс на выходе

// вместо $user_clients родительским массивом для остальных массивов был
$user_clients[0]

Но это мелочи.
Осталось усовершенствовать это проверками и т д
Cпасибо за помощь!!