with not being called in single record request about laravel-json-api

[lindyhopchris]

Hi @tylergets

I've looked into this and it's going to involve some interface changes. As I'm probably going to have to make interface changes when I sort out all the relationships stuff (see #60) I'll include it in that change.

I'm going to do a release of the 1.0 changes I've already made on #60, then will probably move on to the relationship stuff next.

[lindyhopchris]

Hi. The with is only called on index calls at the moment. I can change this but wasn't sure what the point of calling it on a read call is? Because N+1 would only occur if you are loading multiple Auction objects, which you wouldn't on a read request (because it is for a single record).

Unless I'm missing something?! Happy to discuss as if there's a good reason it's needed for the with request this can be added.

[tylergets]

@lindyhopchris In my case, I am including the nested relationships "items,items.bids,items.bids.bidder"

I need to eager load the Bidder object which is N+1ed on a read. Unless I am looking at this wrong?

[lindyhopchris]

ah, ok that makes sense - couldn't see that in your adapter's with method.

it makes sense for the change to be implemented. I'll take a look at what it involves - I suspect I'd need to include it in the next release because it'll be breaking (I'll have to modify the AdapterInterface so that the find method receives the encoding parameters from the client).

[lindyhopchris]

This change is now included on the develop (1.0) branch.

The Eloquent adapter now converts JSON API include paths into Eloquent relationship paths for eager loading e.g. comments.created-by is converted to comments.createdBy for eager loading. If you need to map these paths differently, you can specify this using the $includePaths property of your adapter. See https://github.com/cloudcreativity/laravel-json-api/blob/develop/src/Eloquent/Concerns/IncludesModels.php#L18-L47

It also continues to support default paths for eager loading on the $defaultWith property.

In addition, eager loading is now also supported when reading and updating specific resources.