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

Я использую Binding Form Binding как таковой и обновляю свою БД с помощью методов 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.

Ответы

Ответ 1

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

Однако, если во входном файле не существует "foo" или "bar", ключ будет существовать со значением 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());

Ответ 2

Создать базовую модель и переопределить функцию обновления, например

/**
 * @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'));

Ответ 3

Проверьте это, вы можете проверить, присутствует ли пароль на входе, и исключить его из массового назначения. Вы можете использовать Input:: except и 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
}

Ответ 4

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

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

Ответ 5

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

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

Ответ 6

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

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

$aFormData = Input::all();

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

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

И это, надеюсь, это поможет!

Ответ 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 и пустой строкой. Он также хэширует пароль перед сохранением, поэтому вам не нужно делать это в действии или приложении контроллера в другом месте.

Ответ 8

Лучше всего использовать мутаторы, как сказал выше Noman Ur Rehman, но в его коде была ошибка. Право будет:

public function setPasswordAttribute($password){
   if ( $password !== null && $password !== '' )
      $this->attributes['password'] = Hash::make($password);
}