web-dev-qa-db-ja.com

フォーム値が存在する場合にのみフィールドを更新します

フォームモデルバインディングをそのまま使用し、fill()メソッドとsave()メソッドを使用してDBを更新しています。

{{ 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に設定されます。そのため、フォームから新しいパスワード値が指定された場合にのみ、パスワードフィールドを更新する必要があります。

私は次のことができることを知っています:

// 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();

しかし、これを処理するためのよりクリーンな方法があるかどうか疑問に思っていますか? Laravel/Eloquent内のUpdateOnlyIfValueExists()メソッドのように。

12

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());

12
acairns

ベースモデルを作成し、次のような更新関数をオーバーライドします

/**
 * @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
mrsum

これをチェックすると、パスワードが入力に存在するかどうかを検証し、一括割り当てから除外できます。 Input :: exceptとInput :: onlyをこの目的にのみ使用できます

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

    //return something
}
2
IAmJulianAcosta
$data = $request->password ? $request->all():$request->except('password');
$user->update($data);

これは、パスワードがnullでない場合にのみパスワードを更新します

2
Faran Ali

私はあなたの後者の例に固執します。もう1つのオプションは、 mutator を使用することです。これは、そこで値をチェックし、値が空の場合は更新しません。しかし、私の意見では、Eloquentはそれを行う責任を負わないはずです。

また、fill()でALL入力を使用することは避けたいと思います。必要なものだけを選択してください。

1
Aken Roberts

Noman Ur Rehmanが上で述べたように、ミューテーターを使用するのが最善のアプローチですが、彼はコードに誤りがありました。正しいものは次のとおりです。

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

よりクリーンなアプローチは、 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でなく、文字列が空の場合にのみパスワード属性を設定します。また、保存する前にパスワードをハッシュするため、コントローラーアクションや他のアプリケーションでパスワードを設定する必要はありません。

0
Noman Ur Rehman

これはLaravel(および他のフレームワーク)のかなりくだらない一般的な問題です。私のソリューションは以前のいくつかに似ています...

Update/storeメソッドの開始時に、常にフォームデータInput :: all()が変数に格納されています。通常、少なくとも2回(検証と作成/更新)必要になるため、これは良い習慣のようです。次に、それを使用して、他のことを行う前に、update()でパスワードの存在を確認します。次のようになります。

$aFormData = Input::all();

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

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

そしてそれはそれです、それが役立つことを願っています!

0
rei