Events do not fire when Nova UI is used.

[greggh]

This package is great, does everything we need, except in Laravel Nova. When I use the Attach or Detach buttons on the Nova UI for any of my belongsToMany relationships, nothing fires, no events.

The same relationships with no code changes work great outside of Nova, and I get my events in my observer class for. In jobs running on the schedule, and even down in tinker on the command line. The events fire.

public function belongsToManyAttached($relation, Website $website, $ids)

and

public function belongsToManyDetached($relation, Website $website, $ids)

Website is the class I am observing.

[chelout]

I'd like to help you, but i don't have Nova license 😔

[JT501]

Hi, I got the same issues when using laravel-admin which is a CMS framework similar to Nova. Although I don't have a Nova license, I want to share my experience to fix this issues in laravel-admin and hope this may help you to find out the problem in Nova.

After few hours digging in laravel-admin source code, I found out the function which update relations and in this function, there is a switch() closure as below:

.
.
.
switch (get_class($relation)) {
                case Relations\BelongsToMany::class:
                case Relations\MorphToMany::class:
                    if (isset($prepared[$name])) {
                        $relation->sync($prepared[$name]);
                    }
                    break;
                case Relations\HasOne::class:
.
.
.

The problem is get_class($relation). When using laravel-relationship-events package, it will return Chelout\RelationshipEvents\BelongsToMany instead of Relations\BelongsToMany, although Chelout\RelationshipEvents\BelongsToMany inherits Relations\BelongsToMany, it still can't match any of the cases, thus no any update (sync in my case) and event fire.

So I change the logic a little bit, and it works prefectly. And I created a pull request to laravel-admin.

.
.
.
 switch (true) {
                case $relation instanceof Relations\BelongsToMany:
                case $relation instanceof Relations\MorphToMany:
                    if (isset($prepared[$name])) {
                        $relation->sync($prepared[$name]);
                    }
                    break;
                case $relation instanceof Relations\HasOne:
.
.
.

At this moment, I don't have any idea without changing the source code of laravel-admin.
If @chelout find out any workaround, it would be great ! 😄

[chelout]

Thanks for share @hallelujahbaby!
But without Nova code i can't fix the problem 😢
Maybe @themsaid can help with a dev license? 😬

[greggh]

@themsaid I'm sure you get requests all the time, but this would be amazing. None of the existing packages I can find actually fire these events when Nova is used. If @chelout had a license it could be fixed in this one. Would be a huge help.

[f-liva]

I see Nova using a Illuminate\Database\Eloquent\Relations\Pivot instance to do the pivot rescue. I guess that's why events don't get launched.

However, the events at the pivot update don't work that way either:

 Location::find(20993)->products()->first()->save();

Why is the event belongsToManyUpdatingExistingPivot not raised in this case?

[arall]

If it helps, this is what the Nova controller does when attaching a model into a morphedByMany relationship.

($pivot = $relationship->newPivot())->forceFill([
    $relationship->getForeignPivotKeyName() => $request->resourceId,
    $relationship->getRelatedPivotKeyName() => $request->input($request->relatedResource),
    $relationship->getMorphType() => $request->findModelOrFail()->{$request->viaRelationship}()->getMorphClass(),
        ]);

[f-liva]

Any news here?

[chelout]

As i can see from comments above Nova works with pivot models and doesn't support normal relations, so many-to-many relations can't be supported by this package. Ofcourse we can change basic behavior of laravel's relations, but i don't want to miss miss compatibility.

[f-liva]

How would you do that?

[arall]

Maybe we can implement an adaptor or a Nova component for doing that without messing with this repo? But I have no much clue of how to do it...

[chelout]

@fede91it I guess, it is possible to override laravel's methods like attach, detach, etc to use pivot models.
@arall yeah, i guess we can try to write a component for Nova, but as i mentioned above i don't have license or even Nova code to do it.

[arall]

@chelout I might be able to provide you a license. If you're interested, could you reach me on Telegram? I'm using the same username as in here. Or email (my full name @ gmail.com).

[f-liva]

If necessary, I can sponsor the development of this additional feature.