Laravel属性の一意の検証ルールでモデルを更新
laravel Userモデルがあり、usernameおよびemailに一意の検証ルールがあります。リポジトリでは、モデルを更新するときに、必要なルール検証に問題がないようにフィールドを再検証します。
public function update($id, $data) {
$user = $this->findById($id);
$user->fill($data);
$this->validate($user->toArray());
$user->save();
return $user;
}
これはテストで失敗します
ValidationException: {"username":["The username has already been taken."],"email":["The email has already been taken."]}
これをエレガントに修正する方法はありますか？
現在更新中のインスタンスのidをバリデーターに追加します。
インスタンスの
idを渡して、一意のバリデーターを無視します。バリデーターで、パラメーターを使用して、リソースが更新かcreatingかを検出します。
更新する場合、特定のIDを無視するように一意のルールを強制します。
//rules
'email' => 'unique:users,email_address,' . $userId,
作成する場合は、通常どおり続行します。
//rules
'email' => 'unique:users,email_address',
別のエレガントな方法...
モデルで、静的関数を作成します。
public static function rules ($id=0, $merge=[]) {
return array_merge(
[
'username' => 'required|min:3|max:12|unique:users,username' . ($id ? ",$id" : ''),
'email' => 'required|email|unique:member'. ($id ? ",id,$id" : ''),
'firstname' => 'required|min:2',
'lastname' => 'required|min:2',
...
],
$merge);
}
作成時の検証：
$validator = Validator::make($input, User::rules());
更新時の検証：
$validator = Validator::make($input, User::rules($id));
更新時の検証、いくつかの追加ルール：
$extend_rules = [
'password' => 'required|min:6|same:password_again',
'password_again' => 'required'
];
$validator = Validator::make($input, User::rules($id, $extend_rules));
いいね.
私の質問内での作業：
public function update($id, $data) {
$user = $this->findById($id);
$user->fill($data);
$this->validate($user->toArray(), $id);
$user->save();
return $user;
}
public function validate($data, $id=null) {
$rules = User::$rules;
if ($id !== null) {
$rules['username'] .= ",$id";
$rules['email'] .= ",$id";
}
$validation = Validator::make($data, $rules);
if ($validation->fails()) {
throw new ValidationException($validation);
}
return true;
}
上記の受け入れられた答えに基づいて、私がやったことです。
編集：フォームリクエストにより、すべてがシンプルになります：
<?php namespace App\Http\Requests;
class UpdateUserRequest extends Request
{
/**
* Determine if the user is authorized to make this request.
*
* @return bool
*/
public function authorize()
{
return true;
}
/**
* Get the validation rules that apply to the request.
*
* @return array
*/
public function rules()
{
return [
'name' => 'required|unique:users,username,'.$this->id,
'email' => 'required|unique:users,email,'.$this->id,
];
}
}
更新メソッドにUpdateUserRequestを渡すだけで、モデルIDをPOSTにする必要があります。
Laravel 5と互換性のある一般的な方法：
私はちょうど同じ問題を抱えていて、一般的な方法でそれを解決しました。アイテムを作成する場合、デフォルトのルールを使用します。アイテムを更新する場合、ルールの:uniqueを確認し、必要に応じて自動的に除外を挿入します。
BaseModelクラスを作成し、すべてのモデルにそのクラスを継承させます。
<?php namespace App;
use Illuminate\Database\Eloquent\Model;
class BaseModel extends Model {
/**
* The validation rules for this model
*
* @var array
*/
protected static $rules = [];
/**
* Return model validation rules
*
* @return array
*/
public static function getRules() {
return static::$rules;
}
/**
* Return model validation rules for an update
* Add exception to :unique validations where necessary
* That means: enforce unique if a unique field changed.
* But relax unique if a unique field did not change
*
* @return array;
*/
public function getUpdateRules() {
$updateRules = [];
foreach(self::getRules() as $field => $rule) {
$newRule = [];
// Split rule up into parts
$ruleParts = explode('|',$rule);
// Check each part for unique
foreach($ruleParts as $part) {
if(strpos($part,'unique:') === 0) {
// Check if field was unchanged
if ( ! $this->isDirty($field)) {
// Field did not change, make exception for this model
$part = $part . ',' . $field . ',' . $this->getAttribute($field) . ',' . $field;
}
}
// All other go directly back to the newRule Array
$newRule[] = $part;
}
// Add newRule to updateRules
$updateRules[$field] = join('|', $newRule);
}
return $updateRules;
}
}
これで、次のことに慣れているように、モデルにルールを定義します。
protected static $rules = [
'name' => 'required|alpha|unique:roles',
'displayName' => 'required|alpha_dash',
'permissions' => 'array',
];
そして、それらをコントローラーで検証します。モデルが検証されない場合、対応する検証エラーを含むフォームに自動的にリダイレクトされます。検証エラーが発生しなかった場合、その後のコードの実行が継続されます。
public function postCreate(Request $request)
{
// Validate
$this->validate($request, Role::getRules());
// Validation successful -> create role
Role::create($request->all());
return redirect()->route('admin.role.index');
}
public function postEdit(Request $request, Role $role)
{
// Validate
$this->validate($request, $role->getUpdateRules());
// Validation successful -> update role
$role->update($request->input());
return redirect()->route('admin.role.index');
}
それでおしまい！ :)作成時にはRole::getRules()を呼び出し、編集時には$role->getUpdateRules()を呼び出します。
Laravelの異なる列IDを使用した一意の検証
'UserEmail'=>"required|email|unique:users,UserEmail,$userID,UserID"
またはフォームリクエストでできることは（Laravel 5.3+の場合）
public function rules()
{
return [
'email' => 'required|email|unique:users,email,'.$this->user, //here user is users/{user} from resource's route url
];
}
Laravel 5.6でそれをやったが、うまくいった。
ロール更新の簡単な例
// model/User.php
class User extends Eloquent
{
public static function rolesUpdate($id)
{
return array(
'username' => 'required|alpha_dash|unique:users,username,' . $id,
'email' => 'required|email|unique:users,email,'. $id,
'password' => 'between:4,11',
);
}
}
。
// controllers/UsersControllers.php
class UsersController extends Controller
{
public function update($id)
{
$user = User::find($id);
$validation = Validator::make($input, User::rolesUpdate($user->id));
if ($validation->passes())
{
$user->update($input);
return Redirect::route('admin.user.show', $id);
}
return Redirect::route('admin.user.edit', $id)->withInput()->withErrors($validation);
}
}
BaseModelクラスがあるので、もっと汎用的なものが必要でした。
//app/BaseModel.php
public function rules()
{
return $rules = [];
}
public function isValid($id = '')
{
$validation = Validator::make($this->attributes, $this->rules($id));
if($validation->passes()) return true;
$this->errors = $validation->messages();
return false;
}
ユーザークラスで、検証するのに必要なのはメールと名前だけだとしましょう。
//app/User.php
//User extends BaseModel
public function rules($id = '')
{
$rules = [
'name' => 'required|min:3',
'email' => 'required|email|unique:users,email',
'password' => 'required|alpha_num|between:6,12',
'password_confirmation' => 'same:password|required|alpha_num|between:6,12',
];
if(!empty($id))
{
$rules['email'].= ",$id";
unset($rules['password']);
unset($rules['password_confirmation']);
}
return $rules;
}
これをphpunitでテストし、正常に動作します。
//tests/models/UserTest.php
public function testUpdateExistingUser()
{
$user = User::find(1);
$result = $user->id;
$this->assertEquals(true, $result);
$user->name = 'test update';
$user->email = '[email protected]';
$user->save();
$this->assertTrue($user->isValid($user->id), 'Expected to pass');
}
より良いアイデアを得るためであっても、私は誰かを助けることを願っています。あなたのものも共有してくれてありがとう。 （Laravel 5.0でテスト済み）
StoreとUpdateに対して異なる検証クラスを呼び出しています。私の場合、すべてのフィールドを更新する必要はありません。そのため、作成と編集の共通フィールドにbaseRulesがあります。それぞれに追加の検証クラスを追加します。私の例が役立つことを願っています。 Laravelを使用しています4。
型：
public static $baseRules = array(
'first_name' => 'required',
'last_name' => 'required',
'description' => 'required',
'description2' => 'required',
'phone' => 'required | numeric',
'video_link' => 'required | url',
'video_title' => 'required | max:87',
'video_description' => 'required',
'sex' => 'in:M,F,B',
'title' => 'required'
);
public static function validate($data)
{
$createRule = static::$baseRules;
$createRule['email'] = 'required | email | unique:musicians';
$createRule['band'] = 'required | unique:musicians';
$createRule['style'] = 'required';
$createRule['instrument'] = 'required';
$createRule['myFile'] = 'required | image';
return Validator::make($data, $createRule);
}
public static function validateUpdate($data, $id)
{
$updateRule = static::$baseRules;
$updateRule['email'] = 'required | email | unique:musicians,email,' . $id;
$updateRule['band'] = 'required | unique:musicians,band,' . $id;
return Validator::make($data, $updateRule);
}
コントローラー：保管方法：
public function store()
{
$myInput = Input::all();
$validation = Musician::validate($myInput);
if($validation->fails())
{
$key = "errorMusician";
return Redirect::to('musician/create')
->withErrors($validation, 'musicain')
->withInput();
}
}
更新方法：
public function update($id)
{
$myInput = Input::all();
$validation = Musician::validateUpdate($myInput, $id);
if($validation->fails())
{
$key = "error";
$message = $validation->messages();
return Redirect::to('musician/' . $id)
->withErrors($validation, 'musicain')
->withInput();
}
}
以下のコードを試すことができます
return [
'email' => 'required|email|max:255|unique:users,email,' .$this->get('id'),
'username' => 'required|alpha_dash|max:50|unique:users,username,'.$this->get('id'),
'password' => 'required|min:6',
'confirm-password' => 'required|same:password',
];
'email' => [
'required',
Rule::exists('staff')->where(function ($query) {
$query->where('account_id', 1);
}),
],
'email' => [
'required',
Rule::unique('users')->ignore($user->id)->where(function ($query) {
$query->where('account_id', 1);
})
],
外部キーまたはインデックスとして使用されている別の列がある場合、このようなルールでも同様に指定する必要があります。
'phone' => [
"required",
"phone",
Rule::unique('shops')->ignore($shopId, 'id')->where(function ($query) {
$query->where('user_id', Auth::id());
}),
],
public static function custom_validation()
{
$rules = array('title' => 'required ','description' => 'required','status' => 'required',);
$messages = array('title.required' => 'The Title must be required','status.required' => 'The Status must be required','description.required' => 'The Description must be required',);
$validation = Validator::make(Input::all(), $rules, $messages);
return $validation;
}
同じ問題がありました。私がやったこと：ビューのIDをモデルのIDでビューに追加し、ビューでIDを取得した場合にのみ、バリデーターで一意をチェックします。
$this->validate(
$request,
[
'index' => implode('|', ['required', $request->input('id') ? '' : 'unique:members']),
'name' => 'required',
'surname' => 'required',
]
);
カスタムFormRequestおよびLaravel 5.7+の場合、次のように更新されたモデルのIDを取得できます。
public function rules()
{
return [
'name' => 'required|min:5|max:255|unique:schools,name,'.\Request::instance()->id
];
}
Laravel 5.8シンプルで簡単
これをすべてフォームリクエストで非常にうまく行うことができます。 。 。
最初に、通常の編集フォームでID（非表示）を渡すことができるフィールドを作成します。すなわち、
<div class="form-group d-none">
<input class="form-control" name="id" type="text" value="{{ $example->id }}" >
</div>
...次に、次のようにフォームリクエストにRuleクラスを追加してください。
use Illuminate\Validation\Rule;
...次のように、現在のIDを無視する一意のルールを追加します。
public function rules()
{
return [
'example_field_1' => ['required', Rule::unique('example_table')->ignore($this->id)],
'example_field_2' => 'required',
];
...最後に、次のように、storeメソッドと同じようにupdateメソッドでフォームリクエストを入力します。
public function update(ExampleValidation $request, Examle $example)
{
$example->example_field_1 = $request->example_field_1;
...
$example->save();
$message = "The aircraft was successully updated";
return back()->with('status', $message);
}
この方法では、不必要にコードを繰り返すことはありません:-)