使用唯一字段更新 Laravel 模型

Question

如果模型具有唯一字段，我将无法更新它。我收到消息“该名称已被占用。”

我的控制器：

/**
 * Update the specified Category in storage.
 *
 * @param  int              $id
 * @param UpdateCategoryRequest $request
 *
 * @return Response
 */
public function update($id, UpdateCategoryRequest $request)
{
    $category = $this->categoryRepository->findWithoutFail($id);

    if (empty($category)) {
        Flash::error('Category not found');

        return redirect(route('categories.index'));
    }

    $category = $this->categoryRepository->update($request->all(), $id);

    Flash::success('Category updated successfully.');

    return redirect(route('categories.index'));
}

更新类别请求

 /**
 * Get the validation rules that apply to the request.
 *
 * @return array
 */
public function rules()
{
    return Category::$rules;
}

类别模型

    /**
 * Validation rules
 *
 * @var array
 */
public static $rules = [
    'name' => 'required|unique:categories,name'
];

我已尝试将以下内容附加到我的规则中：

$this->id
$id

@include('adminlte-templates::common.errors')
       <div class="box box-primary">
           <div class="box-body">
               <div class="row">
                   {!! Form::model($category, ['route' => ['categories.update', $category->id], 'method' => 'patch']) !!}

                        @include('categories.fields')

                   {!! Form::close() !!}
               </div>
           </div>
       </div>

Answer 1

要从检查中排除当前模型，请将 id 作为3rd 列传递。

'name' => 'required|unique:categories,name,'. $this->id

Answer 2

在您的请求中，只需附加到您从模型中获得的“名称”规则。

public function rules()
{
    $rules = Category::$rules;
    $rules['name'] .= ','. $this->route('id');
    return $rules;
}

Answer 3

好的，这对我有用：

在我的 UpdateCategoryRequest 中：

public function rules()
    {
        $rules = Category::$rules;
        if ($this->isMethod('patch'))
        {
            $id = $this->categories;
            $rules['name'] = $rules['name'].',name,'.$id;
        }
        return $rules;
    }

Answer 4

这样做。

您需要排除当前模型。由于您要求表的唯一性，因此不排除您当前的列或说 id。

请编辑您的请求

public function rules()
{
    switch($this->method())
    {
        case 'GET':
        case 'DELETE':
        {
            return [];
        }
        case 'POST':
        {
            return [
                'name' => 'required|unique:categories',

            ];
        }
        case 'PUT':
        case 'PATCH':
        {
            return [
                'name' => 'required|unique:categories,name,'.$this->getSegmentFromEnd().',id'

        }
        default:break;
    }
}

Answer 5

我认为您无法通过在不进行任何更改的情况下静态地从模型中检索您的规则来解决此问题，因为您必须将模型 ID 作为第三个参数动态附加到 unique 规则中。

要获取模型 ID，您需要访问请求，该请求仅在您的 FormRequest 类中具有，在您的情况下使用 $this->route('id')。或者，如果您使用的是route model binding，并且您已经定义了类似my/route/{category} 的路由路径，那么：

public function rules()
{
    $category = $this->route('category');

    $rules = Category::$rules;
    $rules['name'] .= ",{$category->id}";

    return $rules;
}

当然，除非您花哨并在模型中声明一个 getter 方法，该方法将接受 id 作为参数并在那里构建规则。

Answer 6

public function update($id, UpdateCategoryRequest $request)
{
    $category = $this->categoryRepository->findWithoutFail($id);

    if (empty($category)) {
        Flash::error('Category not found');

        return redirect(route('categories.index'));
    }

    $category = $category->update($request->all());

    Flash::success('Category updated successfully.');

    return redirect(route('categories.index'));
}