Laravel Eloquentリレーション先の結果によってデータを取得する。

Question

前提・実現したいこと

skillsテーブル

.id
.name
.status

blog_skillsテーブル(中間テーブル)

.skill_id
.blog_id

blogsテーブル

.content
.id

モデル、マイグレーション

[Blog_php]

public function blogSkills()
    {
        return $this->hasMany(BlogSkill::class, 'blog_id');
    }

[skill.php]

public function blogSkills()
    {
        return $this->hasMany(BlogSkill::class, 'skill_id');
    }

[Blogマイグレーション ]

public function up()
	{
		Schema::create('blogs', function(Blueprint $table)
		{
			$table->integer('id', true);
			$table->string('content', 200);
			$table->boolean('status');
			$table->dateTime('published_at')->nullable();
			$table->timestamps();
			$table->softDeletes();
		});
	}

[skillマイグレーション ]

public function up()
	{
		Schema::create('skills', function(Blueprint $table)
		{
			$table->integer('id', true);
			$table->string('skill_name', 50)->unique('skill_name');
			$table->boolean('status');
			$table->timestamps();
			$table->softDeletes();
		});
	}

内容

現在、Eloquentを利用してskillsテーブルのstatusが１のものだけを取得したいと考えています。
しかし下記のソースコードでは、対応するskills_idが複数あり、一つでもstatusが１であれば、その他のstatusが0だとしても弾かれることはなく取得してしまいます。
複数skill_idを持っていたとしても、statusが１のものだけを取得したいです。

リレーションを使いたいので、Eloquentで実装したいと考えます。

ご教授お願いいたします。

該当のソースコード

Blog::whereHas('blogSkills', function($query){
                $query->whereHas('skill', function($query2){
                    $query2->where('status', '=', 1);
                });
            })

Comments

[退会済みユーザー]

リレーションの定義が間違っていますよ。
多対多で定義すべきところ、HasMany, BelongsTo 使っていると思います。

[potter9610]

リレーションの定義が違うとは、どこのことを指しているのでしょうか。
勉強不足で申し訳ありません。

[退会済みユーザー]

では、Blog.php Skill.php を提示してください。

[退会済みユーザー]

あと、migrationファイルもお願いします。

[potter9610]

質問文に追記いたしました。

Answer 1

各テーブル、skills, blog_skills, blogs の関係、blog_skillsは中間テーブルであると、ご自身も書かれているので、設計をまず見直しましょう。

Laravel において、中間テーブルで連結するリレーションは、belongsToMany で定義します。
詳細は、こちらをご覧ください。

「多対多」
https://readouble.com/laravel/6.x/ja/eloquent-relationships.html#many-to-many

例示されているように、中間テーブルは「単数形_単数形」でテーブル名を定義します。
ですので、「blog_skills」ではなく、「blog_skill」となります。_の前後はアルファベット順になります。

そうすると、

Blog　クラスは

class Blog extend Model
{
    public function skills(): BelongsToMany
    {
        return $this->belongsToMany(Skill::class);
    }
}

Skill クラスは

class Skill extend Model
{
    public function blogs(): BelongsToMany
    {
        return $this->belongsToMany(Blog::class);
    }
}

上記のように定義するのが正しい「リレーション」となります。

$blogs = Blog::query()
            ->with(['skills' => function ($query) {
                $query->where('status', 1)
            }])
            ->whereIn('id', function ($query) {
                $query->from('blog_skill')
                    ->select('blog_id')
                    ->where('status', 1);
            })
            ->get();

Comments

[退会済みユーザー]

せめて理解できたかどうかのリアクションは欲しい…