Модель оновлення Laravel з унікальним правилом перевірки атрибуту

У мене є Userмодель laravel, яка має унікальне правило перевірки на 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екземпляр, який зараз оновлюється, до валідатора.

1. Передайте idсвій екземпляр, щоб проігнорувати унікальний валідатор.

2. У валідаторі використовуйте параметр, щоб визначити, чи ви оновлюєте чи створюєте ресурс.

Під час оновлення змусіть унікальне правило ігнорувати заданий ідентифікатор:

//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 своєму методу оновлення та обов’язково розмістити ідентифікатор моделі.

Унікальна перевірка за допомогою різних ідентифікаторів стовпців у Laravel

'UserEmail'=>"required|email|unique:users,UserEmail,$userID,UserID"

Сумісний із 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 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, і це спрацювало.

У мене є клас 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 = 'ddd@test.si';
    $user->save();

    $this->assertTrue($user->isValid($user->id), 'Expected to pass');

}

Я сподіваюся, комусь допоможу, навіть якщо я хочу отримати кращу ідею. Дякуємо, що поділились і своїм. (протестовано на Laravel 5.0)

Простий приклад оновлення ролей

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

}

Я називаю різні класи перевірки для Store and 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();
    }
}

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;
}

У мене була та сама проблема. Що я зробив: додайте у моєму перегляді приховане поле з ідентифікатором моделі та у валідаторі перевірте унікальний, лише якщо я отримав якийсь ідентифікатор з подання.

$this->validate(
        $request,
        [
            'index'       => implode('|', ['required', $request->input('id') ? '' : 'unique:members']),
            'name'        => 'required',
            'surname'     => 'required',
        ]
);

'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);
    })
],

Ви можете спробувати код нижче

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',
];

Якщо у вас є інший стовпець, який використовується як зовнішній ключ або індекс, то ви повинні вказати це також у такому правилі.

'phone' => [
                "required",
                "phone",
                Rule::unique('shops')->ignore($shopId, 'id')->where(function ($query) {
                    $query->where('user_id', Auth::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;

... Додайте унікальне правило, ігноруючи поточний ідентифікатор так:

public function rules()
{
    return [
          'example_field_1'  => ['required', Rule::unique('example_table')->ignore($this->id)],
          'example_field_2'  => 'required',

    ];

... Нарешті введіть натяк на запит форми в методі оновлення так само, як і метод зберігання, приблизно так:

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


}

Таким чином, ви не будете повторювати код без потреби :-)

public function rules()
{
    if ($this->method() == 'PUT') {
        $post_id = $this->segment(3);
        $rules = [
            'post_title' => 'required|unique:posts,post_title,' . $post_id
        ];
    } else {
        $rules = [
            'post_title' => 'required|unique:posts,post_title'
        ];
    }
    return $rules;
}

Для власного FormRequest та Laravel 5.7+ ви можете отримати ідентифікатор оновленої моделі таким чином:

public function rules()
    {
        return [
            'name' => 'required|min:5|max:255|unique:schools,name,'.\Request::instance()->id
        ];
    }

Для тих, хто використовує запит на форму

У моєму випадку я спробував усі наступні дії, жоден з них не працював:

$this->id, $this->user->id, $this->user.

Це було тому, що я не міг отримати доступ $idні до моделі, ні $idбезпосередньо.

Отже, я отримав $idзапит із використанням того самого uniqueполя, яке я намагаюся перевірити:

    /**
 * Get the validation rules that apply to the request.
 *
 * @return array
 */
public function rules()
{
    $id = YourModel::where('unique_field',$this->request->get('unique_field'))->value('id');
    return [
        'unique_field' => ['rule1','rule2',Rule::unique('yourTable')->ignore($id)],
    ];
}