WhereHas Constraints Not Working about compoships HOT 7 OPEN

@MannikJ Which version of the package are you using?

from compoships.

Sorry, v2.1 it was, I guess

Update: Just updated to the latest v2.1.1 but the issue is still there..
Using it in Laravel v8.38.0 with PHP v7.4.16 and MySQL v8.0.20 by the way

@topclaudy Do you have an idea?

from compoships.

Sorry, v2.1 it was, I guess

Update: Just updated to the latest v2.1.1 but the issue is still there..
Using it in Laravel v8.38.0 with PHP v7.4.16 and MySQL v8.0.20 by the way

@topclaudy Do you have an idea?

Sorry for the delay. Can you try 2.0.x?

from compoships.

I am facing the same problem.
And downgrading to 2.0.x does not work as that version was not yet compatible with laravel/framework 9.x

from compoships.

And as was also in the original authors code snippet. I too am trying to have a relationship onto the same class.
With another class it works.

So:

class A extends Model
{
    use compoships;

    public function linkedA()
    {
        return $this->belongsTo(
            A::class,
            ['linked_a_id', 'other_id'],
            ['id', 'other_id']
        );
    }
}

A::whereHas('linkedA');

does not work.

If we add

class B extends Model
{
    use compoships;

    public function a()
    {
        return $this->belongsTo(
            B::class,
            ['a_id', 'other_id'],
            ['id', 'other_id']
        );
    }
}

B::whereHas('a');

it works.

from compoships.

I can confirm I have the same setup as TheFehr and the issue only presents when the relation is on the same model e.g. a parent/child relationship.

from compoships.

As this was a big problem for me I removed this package and added calculated id columns for the pair of id's I needed to match.
It leads to pretty big id's but I now can correctly use laravel relations again.

For anyone curious I used the cantor pairing function.

If you use this approach don't forget to always use the same order for your two id's otherwise it won't work. And don't forget to add indexes for the new calculated id column.

from compoships.