So sense no one else made a pr (with working tests) i went ahead and created the exclude functionallity at the same time i updated the fractal package itself.

• Fix paginator test by just moving out the links array from the meta data.
• Added exclude functionallity similar to include functionality.
• upgraded to fractal 0.14.0

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 8.x.

Before merging, you need to:

• Checkout the l8-compatibility branch
• Review all comments for additional changes
• Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

I was trying to use Fractal's JsonApiSerializer with this package, and it wasn't playing nicely with the 1.0 spec. Realized Fractal was updated 3 days ago, so I updated Fractal and the newly failing tests.

Let me know if I need to change anything or do anything differently to contribute!

As per this issue: https://github.com/spatie/fractalistic/issues/7 I am making a PR with proposed changes.

I am open to suggestions :)

There may be a benefit in removing the \Spatie\Fractal\Response and using the Illuminate Response instead. We could also try to extend Illuminate Response by Fractal Response to preserve Laravel's native API, yet add our own methods.

P.S. These have been my first unit tests ever. Please provide some feedback if something is not done 'the right' way. P.P.S. I am indeed terrible at naming things. Currently, public methods are:

statusCode(); // get a status code
code();       // set a status code
getHeaders(); // get all headers
header();     // set one header
headers();    // set multiple headers

I would also suggest adding those aliases: (some of them ofc)

// adding status code
responseCode();
withStatusCode();
withCode();
addCode();
setCode();
setStatusCode();

// adding header/s
setHeader();
addHeader();
withHeader();

setHeaders();
addHeaders();
withHeaders();

This updates the file to make it more consistent with other config files that are built-in with the framework. The comment itself also now features a laravelish Staircase-Like Effect™

This PR implements auto includes feature based on parseIncludes() method from the spatie/fractalistic package:

• include includes "/books?include=author,publishers.somethingelse"
• exclude includes "/books?exclude=author,publishers"
• include parametrs "/books?include=comments:limit(5|1):order(created_at|desc)"

parseIncludes() and parseExcludes() methods are well tested, no test included.

The PR for auto includes only targeted 5.5 branch. This simply backports the same PR to 5.4 target.

Original PR https://github.com/spatie/laravel-fractal/pull/114/files

I got:

ErrorException in Fractal.php line 163:
array_merge(): Argument #2 is not an array

after doing:

Fractal::collection($data, new $transformerClass, $transformerClass::TYPE)->parseIncludes(Input::get('expand'))->toJson();

I think that this is so common that the method should account for null or anything other than array and string.

About

This PR will add ^6.0 to the list of compatible versions of Laravel.

By preemtively adding the new version, it will allow us to give this fractalistic package a spin, before the final release of Laravel 6.0.

In the case that you don't feel comfortable merging/tagging this change before the official release, I'll just continue working off of my fork. :relaxed: :v:

When building up APIs with fractal I found myself repeating the same code to include relations on my Eloquent models:

// BookTransformer.php

protected $availableIncludes = [
    'author',
    'characters'
];

public function includeAuthor(Book $book) {
     return $this->item($book->author, new AuthorTransformer);
}

public function includeCharacters(Book $book) {
    return $this->collection($book->characters, new CharacterTransformer);
}

This PR enables you to express the above code more succinctly as:

// BookTransformer.php

protected $itemIncludes = [
    'author' => AuthorTransformer::class,
];

protected $collectionIncludes = [
    'characters' => CharacterTransformer::class,
];

Tests and docs included.

As proposed in #129 ;) The default value of fractal_class is Spatie\Fractal\Fractal but if you prefer to have it set to null the code will keep working using the default Spatie\Fractal\Fractal class

PS: sorry for the horrible commit message