goldspecdigital / laravel-eloquent-uuid Goto Github PK

This lib doesn't work with function 'with'
for example Dataset::with('keywords')->get();

return

SQLSTATE[42883]: Undefined function: 7 ERROR: operator does not exist: uuid = integer LINE 1: ..."keyword_id" where "dataset_keyword"."dataset_id" in (0, 61,... ^ HINT: No operator matches the given name and argument types. You might need to add explicit type casts. (SQL: select "keyword".*, "dataset_keyword"."dataset_id" as "pivot_dataset_id", "dataset_keyword"."keyword_id" as "pivot_keyword_id", "dataset_keyword"."order" as "pivot_order" from "keyword" inner join "dataset_keyword" on "keyword"."id" = "dataset_keyword"."keyword_id" where "dataset_keyword"."dataset_id" in (0, 61, 6, 0, 0) order by "order" asc)

I have relationship table that connects 2 other tables together (let say post _tags) this table only gets id of post and id of tags therefore this table does not have model (doesn't need one), now when I store my post tags with sync() method. It leaves id column empty which cause errors while storing my posts.

Any solution to that?

Composer refuses to install this with Laravel 7.

Hopefully you can fix it. Don't think anything has changed or needs to be fixed, just the dependency version check needs to be fixed. :)

String comparison of UUID is least recommended due to infamous cycles of comparison as compared to integer.
Support article - Optimised UUID Storage

Currently recommended repository Laravel Efficient UUID be referred more of to get an idea of implementation.

Although not many repositories yet support uuid alternative as key identification of data, hence above alternative repository seemed more helpful. (They also have a normal uuid repo too.)

I loved your ease of usage over their repo, hence suggesting their ideology and your accessibility, together, will be a big charm.

After changing my users table to use uuid when I try to register it says

419
Page Expired

Has anyone else experienced this? what should I do?

Schema::create('users', function (Blueprint $table) {
            $table->uuid('id')->primary(); <-- here
            $table->string('name');
            // rest of it...
}

use GoldSpecDigital\LaravelEloquentUUID\Database\Eloquent\Uuid; <-- here

class User extends Authenticatable
{
    use HasApiTokens;
    use HasFactory;
    use HasProfilePhoto;
    use Notifiable;
    use TwoFactorAuthenticatable;
    use Uuid; <-- here
    protected $keyType = 'string'; <-- here
    public $incrementing = false; <-- here
}

How can I create UUID in seeder file?

Sample

       DB::table('brands')->delete();

        $brands = array(
            array(
                'id' => '',  // how to create this?
                'name' => 'No Brand',
                'slug' => 'no-brand',
                'description' => null,
                'photo' => null,
                'created_at' => Carbon::now()->format('Y-m-d H:i:s'),
                'updated_at' => Carbon::now()->format('Y-m-d H:i:s'),
            ),
        );
        
        DB::table('brands')->insert($brands);

Following the proposal of #20, we should definitely add a trait as an alternative to extending the classes provided.

This flexibility allows this package to still be utilised in the event that the client code is already extending their models from a class provided in another package.

This will need to be documented in the README.md.

Is there anyway to default to other versions of uuid? Instead of creating uuid v4 create uuid v1?
This is just out of curiosity? Apache usergrid uuid default is v1 so if you would like to build something over that this would be great.

i'm not able to use model injection in my controllers

When running uuid:make:model in a Linux system, a FileNotFoundException is raised.

The code looks for a 'stubs' directory (with lowercase 's'), and the directory src/Console/Stubs (with uppercase 's') is not found. A simple solution would be changing this directory to a lowercase name.

The installation is not posible because the composer json is limited to 6.0 version.

Laravel now use sem versioning this package can be set to install on any 6.x laravel version

Hi there, i have this problem with sqlite:

SQLSTATE[HY000]: General error: 20 datatype mismatch (SQL: insert into "users" ("name", "role", "email", "password", "id", "updated_at", "created_at") values (Fern, superuser, Fern@d
ash.com, $2y$10$LjnVxgcMpuQZWrFErSKDmOeJND8yMLnMJ2vFQ7QfqQRABz8T22NQK, cf0de48c-3472-4ee2-be4a-2828f5c9b882, 2020-05-16 02:45:28, 2020-05-16 02:45:28))

I think that sqlite not support this ??

I’m using Postgres with the uuid-ossp extension which provides the following functionality: uuid_generate_v4() which I have created a macro on the schematic builder for:

Blueprint::macro('primaryUuid', function ($field = 'uuid') {
    $this->uuid($field)->default(\DB::raw('uuid_generate_v4()'))->primary();
});

This requires the $incrementing property on the model to be true (incrementing is a bad choice for this property on Laravel’s side, it basically means “the database handles it”)

Might I suggest that this model utilises the uuid_generate_v4 method to create the UUID for Postgres if the option is available?

Might it be better to create a macro for creating primary UUID columns and add it to the schema builder which sets a default value for a primary UUID?

If this is something you would want to try I could make a PR for it.

Hi, as of Laravel 8 is out, I can't upgrade my project because it's not compabitible with goldspecdigital/laravel-eloquent-uuid.
If possible, can you create a new release?

Hi there,

Am I right if I think the Uuid trait is not available for laravel 5.8.* (using v1.1.0)?
Would it be possible to backport this? Or are there specific reasons to not go there? :)

Thanks,
Glenn

Hello,

there is a security against duplicates ?

When using the laravel "MustVerifyEmail", with the "GoldSpecDigital\LaravelEloquentUUID\Foundation\Auth\User" the user is still able to log in even without confirming their emails.

Hi there,

I have tried the package, but this error not present in normal model insertion

SQLSTATE[23000]: Integrity constraint violation: 19 NOT NULL constraint failed: client_service.id (SQL: insert into "client_service" ("client_id", "detail", "service_id", "start", "user_id") values (e5cbdc0c-e23c-4df3-9908-a1a52de45311, VDSVDS, d50c2578-75d7-4b96-b8d7-c960e624734a, 2020-05-16 00:00:00, 3f7d6572-3135-4d62-8d63-a03e7a5a363b))

the code if u want see:

    /**
     * @param Request $request
     * @return \Illuminate\Http\RedirectResponse
     */
    public function create(Request $request)
    {
        $request->validate([
            'id' => 'max:255|exists:clients',
            'service_id' => 'max:255|required|exists:services,id',
            'start' => 'max:255',
            'detail' => 'max:255',
        ]);

        \App\Client::find($request->id)->services()
            ->attach($request->service_id, [
                'user_id' => auth()->id(),
                'start' => Carbon::parse(str_replace('/', '-', $request->start))->toDate(),
                'detail' => $request->detail ?? '',
            ]);

        return back()->with('success', __('Service has been created'));
    }

i have restored normal id for the moment.. tel me updates

Hi! On version ^8.0 I'm getting an error:

Type of GoldSpecDigital\LaravelEloquentUUID\Database\Eloquent\Model::$guarded must be array 
(as in class Illuminate\Database\Eloquent\Model)

Set up a feature test with the following (after setting up the uuid in the User model as per the documentation).

    /** @test */
    public function a_user_can_be_created()
    {
        Event::fake();

        $user = \App\User::create([
            'name' => 'J Smith',
            'email' => '[email protected]',
            'password' => 'some_pass',
        ]);

         dd($user->id);
    }

Will result in an SQL error:

PDOException: SQLSTATE[HY000]: General error: 1364 Field 'id' doesn't have a default value

Remove the Event::fake(); line and it works as expected, outputting the user's uuid.

Is there a way to use this and still use the event fakes in tests?

With the new version of Laravel being released, we need to release a new version of the package which supports ^v6.0.

Since Laravel has also switched to semantic versioning, the major version of the package should now match the version of Laravel it supports for simplicity.

Great package! I'm just missing some documentation on how the migrations table should look like.

I used to use:

$table->uuid('user_id');

But I am getting error:

SQLSTATE[01000]: Warning: 1265 Data truncated for column 'user_id' at row 1

When inserting data. I assume this is because of a wrong type?

I have an issue for when custom id create when booting method
"Declaration of App\Http\Core\Helper\Helpers::boot() must be compatible with GoldSpecDigital\LaravelEloquentUUID\Database\Eloquent\Model::boot(): void"
I thought that it is a feature and can implement for manually created for custom Id at the model class with Trait. I hope you add this feature to the next released version.

An artisan command should be provided which creates a model using this class. If possible, we should have this override the standard Laravel command - and it should be configurable so you can choose whether to override it or not.

I see that there is already a pull request to support Laravel 9 #47.
Any estimates on when this will be merged and published?

Thank you for your work!

hi,
is there a way of creating uuid on pivot?
on seeding, I'm getting General error: 1364 Field 'id' doesn't have a default value

my model is..

use GoldSpecDigital\LaravelEloquentUUID\Database\Eloquent\Model;
class BoardPost extends Model
{
    use Uuid;
    
    public $incrementing = false;
    protected $keyType = 'string';
    protected $primaryKey = 'id';
    
    public static function boot()
    {
        parent::boot();
        self::creating(function ($model) {
            $uuid = (string) Uuid::generate(4);
            $model->id = $uuid;
        });
    }
}

atm this packages doesn't allow me to upgrade to php8 :(