better error handling about laravel5-jsonapi HOT 9 CLOSED

once I started debugging I saw the problem, looks like there's an endless loop, not sure why it happens yet. The following lines are repeated:

[Mon Sep 07 11:37:52 2015] PHP 119. NilPortugues\Serializer\Serializer->serializeData() /home/remo/jsonapi1/vendor\nilportugues/serializer/src/Serializer.php:469
[Mon Sep 07 11:37:52 2015] PHP 120. NilPortugues\Serializer\DeepCopySerializer->serializeObject() /home/remo/jsonapi1/vendor/nilportugues/serializer/src\Serializer.php:146
[Mon Sep 07 11:37:52 2015] PHP 121. NilPortugues\Serializer\Serializer->serializeInternalClass() /home/remo/jsonapi1/vendor/nilportugues/serializer/src\DeepCopySerializer.php:34

from laravel5-jsonapi.

Could you provide an example? I'll look into it...

from laravel5-jsonapi.

Seems to be failing because it cannot retrieve class properties from internal class... so question is, what's the internal class being used? If it's a special case, I should write its own serializer.

from laravel5-jsonapi.

I'm using the current laravel version and a super simple class:

namespace Ortic\Models;
class Post extends Model {
}

and in my controller:

        $posts = $this->post->all();
        //$posts = $this->post->find(1, ['id']);
        $transformer = $this->serializer->getTransformer();
        $transformer->setSelfUrl(route('posts.index', []));

        $response =  $this->response($this->serializer->serialize($posts));
        return $response;

and my config:

<?php
return [
    [
        'class' => 'OrticDb\Models\Post',
        'alias' => '',
        'aliased_properties' => [
        ],
        'hide_properties' => [
        ],
        'id_properties' => [
            'id',
        ],
        'urls' => [
            'self' => 'posts.index',
        ],
    ],
];

from laravel5-jsonapi.

Laravel Models out of the box use __get(), meaning properties are magic. I see the problem.

Should code a special case for theIlluminate\Database\Eloquent\Model class, indeed.

I'll treat this as a bug.

from laravel5-jsonapi.

Alright I've done some small testing and if nothing goes wrong this fix should up and ready today/tomorrow 👍

from laravel5-jsonapi.

@Remo you'll love this: https://github.com/nilportugues/laravel5-jsonapi-transformer/releases/tag/1.2.0

Thanks once again :) 👍

from laravel5-jsonapi.

You're fast, awesome! I'll continue my tests, thanks!

from laravel5-jsonapi.

@Remo let me know how it goes :)

from laravel5-jsonapi.