UUID column generates a reference when `use_constraints` config option is enabled

[tanthammar]

• Laravel Version: 8.22.1
• PHP Version: 8.0
• Blueprint Version: 1.20.2
• Platform: Mac

Issue:

If I name a UUID column to 'uuid' Blueprint generates a foreign key to a table that doesn't exist.

draft.yaml:

models:
  Vat:
    uuid: uuid

Expected outcome

        Schema::create('vats', function (Blueprint $table) {
            $table->id();
            $table->uuid('uuid');
            $table->timestamps();
        });

Blueprint generates a foreign key to a table that doesn't exist:

        Schema::create('vats', function (Blueprint $table) {
            $table->id();
            $table->uuid('uuid');
            $table->foreign('uuid')->references('uuid')->on('uuids')->onDelete('cascade');
            $table->timestamps();
        });

[ghostwriter]

Try this syntax:

models:
  Vat:
    Id
    uuid: uuid

See test fixture for example: tests/fixtures/drafts/uuid-shorthand-invalid-relationship.yaml

[tanthammar]

Thank you.
It still doesn't generate the correct migration.

Both this:

models:
  Vat:
    id: id
    uuid: uuid

And this:

models:
  Vat:
    Id
    uuid: uuid

Generates

        Schema::create('vats', function (Blueprint $table) {
            $table->id();
            $table->uuid('uuid');
            $table->foreign('uuid')->references('uuid')->on('uuids')->onDelete('cascade')->onUpdate('cascade');
            $table->timestamps();
        });

[ghostwriter]

You’re right, this should work:

models:
  Vat:
    id: id
    uuid: uuid unique

Blueprint converts uuid: uuid to id: uuid primary. It’s one of those custom YAML I mentioned in the previous issue.

[jasonmccreary]

Running Blueprint 1.20.2, the following works for me:

Draft

models:
  Vat:
    uuid: uuid

Migration

        Schema::create('vats', function (Blueprint $table) {
            $table->id();
            $table->uuid('uuid');
            $table->timestamps();
        });

Are you running the latest version of Blueprint?

[tanthammar]

I reinstalled Blueprint and double checked composer.lock, yes v1.20.2 :)

models:
  Vat:
    uuid: uuid

<?php

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

class CreateVatsTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::disableForeignKeyConstraints();

        Schema::create('vats', function (Blueprint $table) {
            $table->id();
            $table->uuid('uuid');
            $table->foreign('uuid')->references('uuid')->on('uuids')->onDelete('cascade');
            $table->timestamps();
        });

        Schema::enableForeignKeyConstraints();
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('vats');
    }
}

[tanthammar]

My config:

<?php

return [
    'namespace' => 'App',

    'models_namespace' => 'Models',
    'controllers_namespace' => 'Http\\Controllers',

    'app_path' => 'app',

    'generate_phpdocs' => true,


    'use_constraints' => true,

    'on_delete' => 'cascade',


    'fake_nullables' => true,


    'use_guarded' => true,

    'generate_fqcn_route' => true,

    'generators' => [
        'controller' => \Blueprint\Generators\ControllerGenerator::class,
        'factory' => \Blueprint\Generators\FactoryGenerator::class,
        'migration' => \Blueprint\Generators\MigrationGenerator::class,
        'model' => \Blueprint\Generators\ModelGenerator::class,
        'route' => \Blueprint\Generators\RouteGenerator::class,
        'seeder' => \Blueprint\Generators\SeederGenerator::class,
        'test' => \Blueprint\Generators\TestGenerator::class,
        'event' => \Blueprint\Generators\Statements\EventGenerator::class,
        'form_request' => \Blueprint\Generators\Statements\FormRequestGenerator::class,
        'job' => \Blueprint\Generators\Statements\JobGenerator::class,
        'mail' => \Blueprint\Generators\Statements\MailGenerator::class,
        'notification' => \Blueprint\Generators\Statements\NotificationGenerator::class,
        'resource' => \Blueprint\Generators\Statements\ResourceGenerator::class,
        'view' => \Blueprint\Generators\Statements\ViewGenerator::class,
    ],

];

[jasonmccreary]

Ah, it's the use_constraints that's throwing it off. It must be reading the uuid as a model reference.