PHP 7.2 + Laravel 5.8: Call to parent method defined in Trait overwritten by a call to parent method in a Class that uses this Trait

Question

It may be hard to explain this case via title so let me put it here.

I had to use Laravel Boilerplate. It uses UUIDs for models. It has a trait that when User model is being created, it adds UUID.

protected static function boot() // this is in the trait
{
    parent::boot();

    static::creating(function ($model) {
        $model->{$model->getUuidName()} = PackageUuid::generate(4)->string;
    });
}

My Eloquent User uses this trait. Everything was working but one day I had to add global scope to User so I defined in the User class:

protected static function boot()
{
    parent::boot();

    static::addGlobalScope(new UserScopeSalesContactAdmin());
}

I totally missed that. Now my global scope works but UUID is ignored. I obviously know why this happens. I'm not the author of the Boilerplate and I would architecture this in a different way but it's a nice educational case hence my question:

How can I change my code the way that both my global scope works as well as UUID? I could simply move my addGlobalScope to the trait but it's obviously stupid idea as scope has nothing to do with UUID hence it violates the OOP rule.

edit:

I did other way around: I copied:

static::creating(function ($model) {
     $model->{$model->getUuidName()} = PackageUuid::generate(4)->string;
});

to the boot method in the User but still I consider this as a duplication…

Namoshek · Accepted Answer · 2019-06-12 04:22:08Z

2

You can write your own trait. But instead of boot() define boot<TraitName>():

trait HasUuid
{
    protected static function bootHasUuid()
    {
        static::creating(function ($model) {
            $model->{$model->getUuidName()} = PackageUuid::generate(4)->string; 
        });
    }
}

This way you don't need to override boot().

Laravel will call the special boot method of traits based on the naming scheme described above. It's just not well documented functionality...

answered Jun 12, 2019 at 4:22

Namoshek

6,5562 gold badges21 silver badges34 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Thungdemo · Accepted Answer · 2019-06-12 04:37:19Z

0

You can try this assuming the name of the trait is BoilerplateTrait. Call the BoilerplateTrait boot function from inside the boot function of you User model

protected static function boot()
{
    //parent::boot();
    BoilerplateTrait::boot();
    static::addGlobalScope(new UserScopeSalesContactAdmin());
}

answered Jun 12, 2019 at 4:37

Thungdemo

1362 silver badges6 bronze badges

Collectives™ on Stack Overflow

PHP 7.2 + Laravel 5.8: Call to parent method defined in Trait overwritten by a call to parent method in a Class that uses this Trait

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related