Только поле обновления, если существует значение формы

Question

Я использую привязку формы модели как таковой и обновляю свою БД с помощью методов fill() и save().

{{ Form::model($account) }} 
    {{ Form::text('name', null, array('class'=>'class')) }} 
    {{ Form::text('email', null, array('class'=>'class')) }} 
    {{ Form::password('password', array('class'=>'class')) }} 
    {{ Form::password('password_confirmation', array('class'=>'class')) }} 
{{ Form::close() }} 

, который выстреливает мой метод контроллера editAccount:

$rules = array(
    'name' => array('required'), 
    'email' => array('required'), 
    'password' => array('confirmed') 
); 

$validator = Validator::make(Input::all(), $rules); 

if ($validator->fails()) 
{ 
// Redirect 
} 

// Save to DB 
$account->fill(Input::all()); 
$account->save(); 

Который работает отлично, но если пароль не был поставлен (потому что пользователь не хочет, чтобы обновить/изменить его), то поле пароля установите значение null в db. Поэтому я хочу, чтобы поле пароля обновлялось, если новое значение пароля передается через форму.

Я знаю, что я могу сделать следующее:

// Set the fields manually 
$account->name = Input::get('name'); 
$account->email = Input::get('email'); 

// Only update the password field if a value is supplied 
if (Input::get('password')) { 
    $account->password = Input::get('password'); 
} 
$account->save(); 

Однако мне интересно, если есть более экологически чистые способы справиться с этим? Как метод UpdateOnlyIfValueExists() в Laravel/Eloquent.

Answer 1

Используя Input::only('foo', 'bar'), вы получите только значения, необходимые для заполнения запроса, вместо использования Input::all().

Однако, если «Foo» или «бар» не существует в пределах входа, ключ будет существовать со значением null:

$input = Input::only('foo', 'bar'); 
var_dump($input); 

// Outputs 
array (size=2) 
    'foo' => null 
    'bar' => null 

Для фильтрации в чистом виде, любые значения с микросхемой null значение:

$input = array_filter($input, 'strlen'); 

В вашем примере, это было бы заменить: $account->fill(Input::all());

Answer 2

Я бы придерживался вашего последнего примера. Другим вариантом было бы использовать mutator, который проверяет значение там и не обновляет, если значение пусто. Но, на мой взгляд, Eloquent не должен нести ответственность за это.

Я бы также не использовал ВСЕ вход с fill(). Выберите только то, что вы хотите.

Answer 3

Создание модели базы и функции обновления переопределение как

/** 
* @param array $attributes 
* @return mixed 
*/ 
public function update(Array $attributes = array()){ 
    foreach($attributes as $key => $value){ 
     if(!is_null($value)) $this->{$key} = $value; 
    } 
    return $this->save(); 
} 

После использования:

$model = Model::find($id); 
$model->update(Input::only('param1', 'param2', 'param3')); 

Answer 4

Это довольно хреново и общая проблема с Laravel (и другими структурами). Мое решение напоминает некоторые из предыдущих ...

У меня всегда есть данные формы Input :: all(), хранящиеся в переменной в начале методов обновления/хранения. Поскольку вам обычно требуется как минимум два раза (проверка и создание/обновление), это кажется хорошей практикой. Тогда с этим и прежде чем делать что-нибудь еще проверить в обновлении() на наличие пароля, что-то вроде этого:

$aFormData = Input::all(); 

if (!$aFormData['password']) 
    unset($aFormData['password']); 

... the rest of your code here using $aFormData ;) ... 

И это все, надеюсь, что это помогает!

Answer 5

Проверьте это, вы можете проверить, если парол ord присутствует во входном сигнале и исключает его из массового присвоения.Вы можете использовать вход :: исключением и Input :: только для этой цели

public function update ($id) { 
    $user = User::findOrFail ($id); 
    if (Input::get ('password') == '') { 
     $user->update (Input::except ('password')); 
    } 
    else { 
     $user->update (Input::all()); 
    } 

    //return something 
} 

Answer 6

$data = $request->password ? $request->all():$request->except('password'); 
$user->update($data); 

Это будет только обновить пароль, если это не пустой

Answer 7

Гораздо чище подход будет использовать Eloquent Mutators

Ни при каких обстоятельствах вы не допустили бы null или пустую строку в качестве пароля, чтобы вы могли безопасно определить следующий мутатор в своей модели Account.

// Only accept a valid password and 
// hash a password before saving 
public function setPasswordAttribute($password) 
{ 
    if ($password !== null & $password === '') 
    { 
     $this->attributes['password'] = bcrypt($password); 
    } 
} 

выше мутатор будет только установить атрибут пароля, если он не null и пустая строка. Он также хэширует пароль перед сохранением, поэтому вам не нужно делать это в действии или приложении контроллера в другом месте.