Relationship data generator is invoked more than once if data is empty

[lindyhopchris]

I've been getting this error:

Exception: Cannot traverse an already closed generator
src/Parser/RelationshipData/RelationshipDataIsCollection.php:132

It only occurs when relationship data is empty. The cause of the bug is these lines here:
https://github.com/neomerx/json-api/blob/master/src/Parser/RelationshipData/RelationshipDataIsCollection.php#L130-L131

As it does not assign an empty array to $this->parsedResources before iterating over the resource, the parsedResources property will remain null. Therefore when it is called a second time, there is no cache to use and the closed generator is used.

I'll submit a PR with a test reproducing the problem and a fix.

[lindyhopchris]

@neomerx see you haven't been active on Github since March last year. Hope everything is ok.

Hope you can tag the fix as 4.0.2. If you need help maintaining the package going forward, do let me know.

[mahagr]

Thanks! I also started my own fork as I need to add the missing pieces to the API, such as pagination and filtering.

[lindyhopchris]

@mahagr looks like I'm going to have to run a fork too, as I'm not sure that PR is going to get merged. My fork is here:
https://github.com/laravel-json-api/neomerx-json-api

[PabloKowalczyk]

Hey @neomerx, do you need some help maintain this repo?

[mahagr]

I am wondering the same. Otherwise, it would make sense to have an official fork instead of everyone doing their own.

[gfemorris]

Hey it seems this project is not maintained anymore. Did you agree which fork should be the "official" one?

[PabloKowalczyk]

I can create and maintain something like json-api-php/json-api.

[gfemorris]

@lindyhopchris created a fork and already has v5.0.0 in it maybe he wants to or you can adapt his changes?

[lindyhopchris]

Yeah I'm happy to maintain my fork because it's a critical part of Laravel JSON:API. I'm not up for doing any re-writing, more just keeping it up-to-date for PHP versions and with the spec if that does change (though it's taking them ages to get to 1.1 any way, so doubt we'll be talking about 1.2 any time soon).

[PabloKowalczyk]

Making it (fork) framework/library agnostic should be better for long term maintenance.

[lindyhopchris]

Yeah to be clear, even though it's in the laravel-json-api organisation, it'll always be left as framework agnostic. That's because I wrap it in another package to tie it into the Laravel JSON:API implementation - i.e. there's a laravel-json-api/encoder-neomerx package that does that wrapping.

So my fork will always remain framework agnostic.