hasOneJson?

[Matt Kingshott]

Hi Jonas,

Amazing work as always!

Just wondered if it would be possible to do a hasOneJson instead of hasManyJson? Is there maybe a simple way to achieve this by chaining to hasManyJson?

Thanks!

[Matt Kingshott]

So based on my experimentation, it would appear that if you extend HasManyJson and replace ->get() within the getResults() method with ->first(), it works as expected.

e.g.

class HasOneJson extends HasManyJson
{
    /**
     * Retrieve the results of the relationship.
     *
     */
    public function getResults() : mixed
    {
        return ! is_null($this->getParentKey()) ? $this->first() : $this->related->newCollection();
    }
}

If this seems acceptable, it would be great if this could be merged in. Thanks! 🙏

[Jonas Staudenmeir]

Hi @mattkingshott,
I'll look into it. There also need to be some adjustments for eager loading, but it shouldn't be complicated.

[Matt Kingshott]

Awesome!

[Jonas Staudenmeir]

I released a new version with the HasOneJson relation:
https://github.com/staudenmeir/eloquent-json-relations?tab=readme-ov-file#hasonejson

[Matt Kingshott]

Great stuff! Thank you 🙏🏻