Error with assignRole when I pass a string.

[DesKevinMendez]

Hi.

Today have been migrating from id autoincrement to uuid, but I have this error when I run seeds.
SQLSTATE[23000]: Integrity constraint violation: 1452 Cannot add or update a child row: a foreign key constraint fails ('coffe'.'model_has_roles', CONSTRAINT 'model_has_roles_role_id_foreign' FOREIGN KEY ('role_id') REFERENCES 'roles' ('id') ON DELETE CASCADE) (SQL: insert into 'model_has_roles' ('model_id', 'model_type', 'role_id') values (98768b53-584c-4d9f-8508-ec712ee49321, App\Models\User, 98768))

The code for my seed is

        $superadmin = User::factory()->create([
            'first_name' => 'Kevin',
            'last_name' => 'Méndez',
            'email' => 'kevin@coffeapp.com'
        ]);
        $superadmin->assignRole('superadmin');

and yes, I already have the superadmin role.

My User model has the HasRoles trait, and my migration for permissions is

        $tableNames = config('permission.table_names');
        $columnNames = config('permission.column_names');
        $teams = config('permission.teams');

        if (empty($tableNames)) {
            throw new \Exception('Error: config/permission.php not loaded. Run [php artisan config:clear] and try again.');
        }
        if ($teams && empty($columnNames['team_foreign_key'] ?? null)) {
            throw new \Exception('Error: team_foreign_key on config/permission.php not loaded. Run [php artisan config:clear] and try again.');
        }

        Schema::create($tableNames['permissions'], function (Blueprint $table) {
            $table->uuid('id')->primary(); // permission id
            $table->string('name');       // For MySQL 8.0 use string('name', 125);
            $table->string('guard_name'); // For MySQL 8.0 use string('guard_name', 125);
            $table->timestamps();
            $table->softDeletes();

            $table->unique(['name', 'guard_name']);
        });

        Schema::create($tableNames['roles'], function (Blueprint $table) use ($teams, $columnNames) {
            $table->uuid('id')->primary(); // role id
            if ($teams || config('permission.testing')) { // permission.testing is a fix for sqlite testing
                $table->uuid($columnNames['team_foreign_key'])->nullable();
                $table->index($columnNames['team_foreign_key'], 'roles_team_foreign_key_index');
            }
            $table->string('name');       // For MySQL 8.0 use string('name', 125);
            $table->string('guard_name'); // For MySQL 8.0 use string('guard_name', 125);
            $table->timestamps();
            $table->softDeletes();

            if ($teams || config('permission.testing')) {
                $table->unique([$columnNames['team_foreign_key'], 'name', 'guard_name']);
            } else {
                $table->unique(['name', 'guard_name']);
            }
        });

        Schema::create($tableNames['model_has_permissions'], function (Blueprint $table) use ($tableNames, $columnNames, $teams) {
            $table->uuid(PermissionRegistrar::$pivotPermission);

            $table->string('model_type');
            $table->uuid($columnNames['model_morph_key']);
            $table->index([$columnNames['model_morph_key'], 'model_type'], 'model_has_permissions_model_id_model_type_index');

            $table->foreign(PermissionRegistrar::$pivotPermission)
                ->references('id') // permission id
                ->on($tableNames['permissions'])
                ->onDelete('cascade');
            if ($teams) {
                $table->uuid($columnNames['team_foreign_key']);
                $table->index($columnNames['team_foreign_key'], 'model_has_permissions_team_foreign_key_index');

                $table->primary([$columnNames['team_foreign_key'], PermissionRegistrar::$pivotPermission, $columnNames['model_morph_key'], 'model_type'],
                    'model_has_permissions_permission_model_type_primary');
            } else {
                $table->primary([PermissionRegistrar::$pivotPermission, $columnNames['model_morph_key'], 'model_type'],
                    'model_has_permissions_permission_model_type_primary');
            }

        });

        Schema::create($tableNames['model_has_roles'], function (Blueprint $table) use ($tableNames, $columnNames, $teams) {
            $table->uuid(PermissionRegistrar::$pivotRole);

            $table->string('model_type');
            $table->uuid($columnNames['model_morph_key']);
            $table->index([$columnNames['model_morph_key'], 'model_type'], 'model_has_roles_model_id_model_type_index');

            $table->foreign(PermissionRegistrar::$pivotRole)
                ->references('id') // role id
                ->on($tableNames['roles'])
                ->onDelete('cascade');
            if ($teams) {
                $table->uuid($columnNames['team_foreign_key']);
                $table->index($columnNames['team_foreign_key'], 'model_has_roles_team_foreign_key_index');

                $table->primary([$columnNames['team_foreign_key'], PermissionRegistrar::$pivotRole, $columnNames['model_morph_key'], 'model_type'],
                    'model_has_roles_role_model_type_primary');
            } else {
                $table->primary([PermissionRegistrar::$pivotRole, $columnNames['model_morph_key'], 'model_type'],
                    'model_has_roles_role_model_type_primary');
            }
        });

        Schema::create($tableNames['role_has_permissions'], function (Blueprint $table) use ($tableNames) {
            $table->uuid(PermissionRegistrar::$pivotPermission);
            $table->uuid(PermissionRegistrar::$pivotRole);

            $table->foreign(PermissionRegistrar::$pivotPermission)
                ->references('id') // permission id
                ->on($tableNames['permissions'])
                ->onDelete('cascade');

            $table->foreign(PermissionRegistrar::$pivotRole)
                ->references('id') // role id
                ->on($tableNames['roles'])
                ->onDelete('cascade');

            $table->primary([PermissionRegistrar::$pivotPermission, PermissionRegistrar::$pivotRole], 'role_has_permissions_permission_id_role_id_primary');
        });

        app('cache')
            ->store(config('permission.cache.store') != 'default' ? config('permission.cache.store') : null)
            ->forget(config('permission.cache.key'));
    

I find it curious that the SQL query

insert into 'model_has_roles' ('model_id', 'model_type', 'role_id') values (98768b53-584c-4d9f-8508-ec712ee49321, App\Models\User, 98768))

role_id is only 98768 and not uuid completed

Do you have any idea about what is wrong with my code?

PD: if I pass an instance of Role, it works perfectly and role_id is a uuid complets.

        $superadmin = User::factory()->create([
            'first_name' => 'Kevin',
            'last_name' => 'Méndez',
            'email' => 'kevin@coffeapp.com'
        ]);
        $r = Roles::findByName('superadmin');
        $superadmin->assignRole($r);

PD2: the Role model has extended Roles

<?php

namespace App\Models;

use App\Traits\HasUuid;
use App\Traits\usePaginate;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\SoftDeletes;
use Spatie\Permission\Models\Role as SpattieRole;

class Roles extends SpattieRole
{
    use HasFactory, usePaginate, SoftDeletes, HasUuid;
}

[drbyte]

What version of the package are you using?
dev-main is currently v6, and recently had UUID support via #2089 added to it. That might affect whether you need your HasUuid trait.

/ping @erikn69

[DesKevinMendez]

My version is ^5.9

[erikn69]

the Role model has extended Roles

Did you set new model on permission config file?

[DesKevinMendez]

I did, but don't work yet

[erikn69]

is Trait HasUuid like this uuid?

<?php
  namespace App;

  use Facades\Str;

  trait UuidTrait
  {
      public static function bootUuidTrait()
      {
          static::creating(function ($model) {
              $model->keyType = 'string';
              $model->incrementing = false;

              $model->{$model->getKeyName()} = $model->{$model->getKeyName()} ?: (string) Str::orderedUuid();
          });
      }
      
      public function getIncrementing()
      {
          return false;
      }
      
      public function getKeyType()
      {
          return 'string';
      }
  }

[erikn69]

Maybe your role id on db has varchar length of 5

Today have been migrating from id autoincrement to uuid, but I have this error when I run seeds.

maybe you changed the datatype in db but you left autoincrement enabled

Just ideas to check out

[DesKevinMendez]

yes, my trait is equals to your example.

<?php

namespace App\Traits;

use Illuminate\Support\Str;

trait HasUuid
{
  public static function bootHasUuid()
  {
    static::creating(function ($model) {
      $model->keyType = 'string';
      $model->incrementing = false;

      $model->{$model->getKeyName()} = $model->{$model->getKeyName()} ?: (string) Str::orderedUuid();
    });
  }

  /**
   * Get the value indicating whether the IDs are incrementing.
   *
   * @return bool
   */
  public function getIncrementing()
  {
    return false;
  }
  /**
   * Get the auto-incrementing key type.
   *
   * @return string
   */
  public function getKeyType()
  {
    return 'string';
  }
}

[erikn69]

Maybe your role id on db has varchar length of 5
role_id is only 98768 and not uuid completed

somehow the id is getting cut, but I have no idea how it happens, maybe something in your implementation, because the tests are working, look tests/Role.php with UUID

laravel-permission/tests/HasRolesTest.php

Lines 39 to 50 in 0c49776

	public function it_can_assign_and_remove_a_role()
	{
	$this->assertFalse($this->testUser->hasRole('testRole'));
	$this->testUser->assignRole('testRole');
	$this->assertTrue($this->testUser->hasRole('testRole'));
	$this->testUser->removeRole('testRole');
	$this->assertFalse($this->testUser->hasRole('testRole'));
	}

[DesKevinMendez]

Let me run everything from scratch.

[DesKevinMendez]

For some reason, it works.

What did I do?

I use docker-compose.

• I removed all container
• I ran docker-compose with flag --no-cache
• I created the database from scratch.
• I ran migrations and it did not show me the error again.

Thanks for helping @erikn69 and @drbyte