search

nilportugues / php-json-api

JSON API transformer outputting valid (PSR-7) API Responses.
http://nilportugues.com
MIT License
71 stars 35 forks source link

Fixed case of empty include #53

Closed OskarD closed 8 years ago

OskarD commented 8 years ago

Fixed the problem explained here: https://github.com/nilportugues/laravel5-jsonapi/issues/61#issuecomment-217794244

nilportugues commented 8 years ago

Great addition @OskarD !

OskarD commented 8 years ago

Thanks @nilportugues! Glad you like it :) I think it could be optimized a little, but it does the job

nilportugues commented 8 years ago

Yeah there's always place for improvement. Specially since PHP7 is out I'm sure we could clean up some rough edges by doing typing and return types.

After all this was put together in a couple of days and they every one and then gets polished a bit more :)