Как изменить изменения в бд для всех значений?

Question

[lil_koi]

у меня есть код, который меняет одно значение из формы, где есть: name, email,phone. Мой код для смены имени выглядит так

<?php 
$data = $_POST; 
if (isset($data['do_changes'])){ 
session_start(); 
$user = R::findOne('users', 'name = ?', array($_SESSION['logged_user']['name'])); 
$usr = R::load('users', $user->id); 
$usr->name = $data['new_name']; 
R::store($usr); 
} 
?>

тут ред бин, если что. Мне надо, что если заполнены ещё значения, то их тоже заменять. Если нет, то оставлять без изменения.
В БД и название инпута подписаны одинакого, но в инпуте приставка new_
следовательно, в бд старое имя в name, а новое в вели в new_name

Answer 1

[qpPeW]

$data = $_POST; 
if (isset($data['do_changes'])){ 
session_start(); 
$user = R::findOne('users', 'name = ?', array($_SESSION['logged_user']['name'])); 
$usr = R::load('users', $user->id);
if (!empty($data['new_name']) && $data['new_name'] != $usr->name) {
$usr->name = $data['new_name'];
}
R::store($usr); 
}

Если Вы правильно описали и я правильно понял, то должно быть именно так.

Comments

[maddog670]

фреш, случайно не сидел когда-то на tw?

[qpPeW]

maddog670, когда-то был такой грешок =)

[maddog670]

qpPeW, а то смотрю знакомые ник увидел))

[qpPeW]

maddog670, значит не даром я выбрал такой ник, что он запоминается...

[maddog670]

qpPeW, память хорошая просто))

[qpPeW]

maddog670, это хорошо ))

[lil_koi]

мой скрипт работает только если менять имя, а мне надо, чтобы можно было менять и имя, и почту, и телефон, но если значения не вписаны, то тогда не менялось

[qpPeW]

lil_koi, так добавьте другие переменные по аналогии как я Вам показал.
Я же не могу знать какие поля Вы используете еще.

[lil_koi]

я переписал код, но почему-то in_array не работает

<?php 
if (isset($data['do_changes'])){ 
    session_start(); 
    $user = R::findOne('users', 'id = ?', [ $_SESSION['logged_user']['id'] ] );
    if (in_array("new_name", $data)){ 
    $user->name = $data['new_name'];} 
    if (in_array("new_email", $data)){
    $user->email = $data['new_email'];}
    if (in_array("new_phone", $data)){ 
    $user->phone = $data['new_phone'];} 
    R::store($user); 
}
?>

[qpPeW]

lil_koi, а причем здесь in_array ?
там должна быть проверка значения, а не проверка существует ли данное значение в массиве.

if (isset($data['do_changes'])){ 
    session_start(); 
    $user = R::findOne('users', 'id = ?', [ $_SESSION['logged_user']['id'] ] );
    if (!empty($data['new_name']) && $data['new_name'] != $usr->name){ 
    $user->name = $data['new_name'];} 
    if (!empty($data['new_email']) && $data['new_email'] != $usr->name){
    $user->email = $data['new_email'];}
    if (!empty($data['new_phone']) && $data['new_phone'] != $usr->name){ 
    $user->phone = $data['new_phone'];} 
    R::store($user); 
}

[lil_koi]

дай боженька тебе здоровья. Твой код работает, как мне надо :)

[qpPeW]

lil_koi, рад был помочь, не забывайте отмечать вопрос решенным.

Answer 2

[Vitsliputsli]

Если особо ничего не менять, то так:

<?php 
if (isset($_POST['do_changes'])){ 
    session_start(); 
    $user = R::findOne('users', 'id = ?', [ $_SESSION['logged_user']['id'] ] ); 
    $user->name = $_POST['new_name']; 
    $user->email = $_POST['new_email']; 
    $user->phone = $_POST['new_phone']; 
    R::store($user); 
}

Comments

[lil_koi]

если вписано что-то одно, а другое нет; например, только имя, то в бд всё остальное становится пустым

[Vitsliputsli]

Нужно заполнять значениями форму. Ну или как-то так напишите $user->email = $_POST['new_email'] ?? $user->email