Laravel5.5でのFormRequestを使った複合ユニークでの重複チェック

Question

実現したいこと

前提
タスクリストモデルとタスクモデルが存在し、タスクリスト(1)がタスク(多)のリレーションを設定している。

データ構造

mysql> show columns from task_lists;
+------------+------------------+------+-----+---------+----------------+
| Field      | Type             | Null | Key | Default | Extra          |
+------------+------------------+------+-----+---------+----------------+
| id         | int(10) unsigned | NO   | PRI | NULL    | auto_increment |
| list_name  | varchar(191)     | NO   |     | NULL    |                |
| created_at | timestamp        | YES  |     | NULL    |                |
| updated_at | timestamp        | YES  |     | NULL    |                |
+------------+------------------+------+-----+---------+----------------+

mysql> show columns from tasks;
+--------------+------------------+------+-----+---------+----------------+
| Field        | Type             | Null | Key | Default | Extra          |
+--------------+------------------+------+-----+---------+----------------+
| id           | int(10) unsigned | NO   | PRI | NULL    | auto_increment |
| task_list_id | int(10) unsigned | NO   | MUL | NULL    |                |
| task_name    | varchar(191)     | NO   |     | NULL    |                |
| limit        | date             | NO   |     | NULL    |                |
| done         | tinyint(1)       | NO   |     | 0       |                |
| created_at   | timestamp        | YES  |     | NULL    |                |
| updated_at   | timestamp        | YES  |     | NULL    |                |
+--------------+------------------+------+-----+---------+----------------+

モデル

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class TaskList extends Model
{
    protected $fillable = ['list_name'];

    public function tasks(){
        return $this->hasMany('App\Task');
    }
}

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Task extends Model
{
    protected $fillable = ['task_name', 'limit'];

    public function tasklist(){
        return $this->belongsTo('App/TaskList');
    }
}

viewのタスク追加フォーム

    <form method="post" action="{{ action('TaskController@store', $list) }}" >
      {{ csrf_field() }}
      <div class="form-group">
        <label for="nameInput">タスク名</label>
        <input type="text" class="form-control" id="nameInput" name="task_name">
        <input type="date" class="form-control" id="limitInput" name="limit">
      </div>
      <button type="submit" class="btn btn-primary">新規追加</button>
    </form>

実現したいこと

タスクリスト内のタスクを追加する際、タスクリスト内でのみタスク名の重複を禁止したい。
(MySQLでいう、複合ユニークキーの実現)

ex)
タスクリスト1内に、タスク1が存在し、タスク1を登録する→禁止
タスクリスト1内に、タスク1が存在する状況で、タスクリスト2内にタスク1を登録する->許可

発生している問題・エラーメッセージ

複合ユニークキーの設定方法がわかりません。
下記のバリデーションルールであれば、タスク名が重複する段階でバリデーションされてしまう

該当のソースコード

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;


class TaskRequest extends FormRequest
{
    /**
     * 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 [
            'task_name' => 'required|min:1|max:31|unique:tasks,task_name'. $this->id. ',id,task_list_id'. $this->task_list_id,
        ];
    }
}

試したこと

下記の記事等を参考にさせていただきましたが、実現できませんでした。
http://tdsen.blog.jp/archives/33449990.html

初歩的な質問なのかもしれませんが、有識者の方、何卒お力添えをよろしくお願いいたします。

###　補足情報
下記のバリデーションを設定した場合、エラーが発生しました。

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Validation\Rule;
use App\Http\Requests\TaskRequest;

class TaskRequest extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return true;
    }

    public function rules()
    {
       return [
            Validator::make($data, [
                'task_name' =>[
                    Rule::unique('tasks')->where(function ($query){
                        return $query->where('task_list_id', $data['task_list_id']);
                    })
                ],
            ]->validate();
        ];
    }

ReflectionException
Class App\Http\Requests\TaskRequest does not exist

Answer 1

この様な感じで如何でしょうか？
このサイトの「追加のWHERE節を付け加える」使えそうですが…

Validator::make($data, [
    'task_list_id2' => [
        Rule::unique('tests')->where(function($query) use($data){
            return $query->where("task_list_id",$data["task_list_id"]);
        })
    ],
])->validate();

追記

私の環境ではこのカスタムリクエストクラスで動かせました。

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Validation\Rule;
use Illuminate\Http\Request;

class TaskRequest extends FormRequest
{
    /**
     * 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(Request $request)
    {
        return [
            'task_list_id2' => 
                Rule::unique('tests')->where(function($query) use($request){
                    return $query->where("task_list_id",$request->task_list_id);
                })
            ];
    }
}

Comments

[sabamiso]

ご回答ありがとうございます。
回答の通りに、コードを組み込ませていただきましたところ、補足のようなエラーが表示されました。
どのような対応を行えばよろしいでしょうか。
基礎的な質問で申し訳ありません。

[motuo]

use App\Http\Requests\TaskRequest;

いらないんじゃないですか？

[sabamiso]

use App\Http\Requests\TaskRequest;
を削除しましても、エラーは変わらぬままの状況です。
お手数おかけして申し訳ありません。

[motuo]

エラー内容は
App\Http\Requests\TaskRequest does not exist
なので、これを読めない事がエラーの直接原因だと思いますよ。
requestではなく、controllerでエラーが起きているのではありませんか？

このあたりも参考になると思います。
https://qiita.com/niiyz/items/5b83ef5255a1ec64d9d6

これにてこずるようでしたら、まずはControllerで直接Validationかけてみては？

[sabamiso]

ご丁寧に自環境でも試していただき、ありがとうございます。
ぜひ、参考にさせていただきます。

Answer 2

とりあえず、「,」 が沢山足りていないようですが、

'task_name' => 
                                            ↓ここ　　　　　　　　　　　　　　↓ここ
'required|min:1|max:31|unique:tasks,task_name,'. $this->id. ',id,task_list_id,'. $this->task_list_id,

質問掲載時の、書き間違いでしょうか？？

---

追記します。

uniqueの後ろに指定できるのは、「ユニークチェック対象から除外する」条件です'task_name' =>
'required|min:1|max:31|unique:tasks,task_name,'. $this->id. ',id,task_list_id,'. $this->task_list_id
上記だと、・idが一致する・task_list_id が一致するのどちらかを満たす場合に除外されます。本来は、・idが一致する・task_list_id が一致しないのどちらかを満たす場合に、除外したいはず・・ですよね？なので、php'task_name' => //↓これでnot条件に'required|min:1|max:31|unique:tasks,task_name,'. $this->id. ',id,task_list_id,!'. $this->task_list_id,
が、設定すべき内容なのではないかと思われます。

書き直しますね・・・

uniqueの後ろに指定できるのは、「ユニークチェック」条件ですね！

1つめは条件一致で除外、2個目以降は条件一致で対象となります。

'task_name' => 
'required|min:1|max:31|unique:tasks,task_name,'. $this->id. ',id,task_list_id,'. $this->task_list_id

つまり

・idが一致しない
・$this->task_list_idが一致する

の双方を満たすものからユニークチェックを行います。

で、新規作成時の話ですが、

$this->idは何か入っていますか？？

-1とか0とか入っていれば大丈夫そうですが、nullの場合、想定の通りに動くのか謎です。
（やったことないので）

もし$this->id　がnullなのであれば、バリデーションを

'task_name' => 
                                            
'required|min:1|max:31|unique:tasks,tasks,task_name,'. intval($this->id). ',id,task_list_id,'. $this->task_list_id

のようにすれば、上手く行くかもです。

・・・何度も訂正を書いてしまったので、自分でも信頼性に欠ける回答だなと思っております。
すみません・・・

Comments

[sabamiso]

自分のケアレスミスでありました。
ご指摘の通り、修正させていただきました。
しかし、バリデーションチェックでの重複禁止が通らず、重複したデータを追加してしまいます。

[mix-peach]

回答追記後に、説明文が間違っていたことに気づきました。。すみません　＞＜; 　回答はまるごと訂正させていただきました・・

[sabamiso]

丁寧な説明ありがとうございました。
$this->idは、autoインクリメントであるため、nullになっている思うので、intvalを使用すると、予想通りの挙動をしてくれました！
ありがとうございます。
ちなみに、回答修正前の方法も試させていただきましたが、そちらも要望どおりの挙動をしてくれましたので、ご報告させていただきます。