Laravel Eloquentのリレーションのid名

Question

疑問点

Laravelでfactoryを使ってseedingをしようと試みているのですが、
下記のようなエラーが発生してしまいました。
（このプログラムでは、usersテーブルとschedulesテーブルを利用しています。）

Unknown column 'schedule_scheduleId'とあるように、model名+primary keyの名前のprimary keyを生成しているようにみえます。

この命名のルールのようなものを変更することはできますか？
それとも、このルールに従って、usersテーブルのscheduleのid名を付けるべきでしょうか？

vagrant@homestead:~/code/scheduler$ php artisan db:seed
Seeding: ScheduleSeeder

   Illuminate\Database\QueryException 

  SQLSTATE[42S22]: Column not found: 1054 Unknown column 'schedule_scheduleId' in 'field list' (SQL: insert into `users` (`userName`, `schedule_scheduleId`) values (Keegan, 1))

  at vendor/laravel/framework/src/Illuminate/Database/Connection.php:671
    667|         // If an exception occurs when attempting to run a query, we'll format the error
    668|         // message to include the bindings with SQL, which will make this exception a
    669|         // lot more helpful to the developer instead of just the database's errors.
    670|         catch (Exception $e) {
  > 671|             throw new QueryException(
    672|                 $query, $this->prepareBindings($bindings), $e
    673|             );
    674|         }
    675| 

  • A column was not found: You might have forgotten to run your migrations. You can run your migrations using `php artisan migrate`. 
    https://laravel.com/docs/master/migrations#running-migrations

      +13 vendor frames 
  14  database/seeds/ScheduleSeeder.php:16
      Illuminate\Database\Eloquent\Relations\HasOneOrMany::save()

      +8 vendor frames 
  23  database/seeds/DatabaseSeeder.php:15
      Illuminate\Database\Seeder::call()

app/Schedule.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Schedule extends Model
{
    protected $primaryKey = 'scheduleId';
    public $timestamps = false;
    //
    public function users() {
        return $this->hasMany('App\User');
    }

}

2020_06_30_000013_create_schedules_table.php

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateSchedulesTable extends Migration
{
    public function up()
    {
        Schema::create('schedules', function (Blueprint $table) {
            $table->increments('scheduleId');
            $table->string('scheduleName');
            $table->string('scheduleUuid');
        });
    }

    public function down()
    {
        Schema::dropIfExists('schedules');
    }
}

2020_06_29_235942_create_users_table.php

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateUsersTable extends Migration
{
    public function up()
    {
        Schema::create('users', function (Blueprint $table) {
            $table->increments('userId');
            $table->string('userName');
            $table->integer('scheduleId');
        });
    }

    public function down()
    {
        Schema::dropIfExists('users');
    }
}

ScheduleSeeder.php

<?php

use Illuminate\Database\Seeder;

class ScheduleSeeder extends Seeder
{

    public function run()
    {
        //
        factory(App\Schedule::class)->create()->users()->save(factory(App\User::class)->make());
    }
}

Answer 1

リレーション部分の書き方が違います。

ドキュメント引用

hasOneメソッドと同様に、外部キーとローカルキーをhasManyメソッドに追加の引数として渡すことでオーバーライドできます。

return $this->hasMany('App\Comment', 'foreign_key');

return $this->hasMany('App\Comment', 'foreign_key', 'local_key');

第二、第三引数をせっていしなかった場合、モデル名と主キーをもとに自動で外部キー名が生成されます。

Comments

[退会済みユーザー]

回答ありがとうございます。
Factoryではなく、そもそもEloquentのリレーションを理解できてなかったためですね。

この度はありがとうございました。