Proxy initializer function bug about rest-client-sdk HOT 6 CLOSED

Hi,

It looks to me that everything is working fine :
When you call a relation, the real object is not fetched but a proxy object is created containing only the ID and every other property is null

You have a initialization Tracker property saying that it is not initialized too.

You can found more about this on the ocramius/proxy-manager package.

If you call ->getNumero() it should work too

from rest-client-sdk.

It looks to me that everything is working fine

it is working fine, but when you call getNumero from the controller the condition if(!$isAllowedMethod) becomes false.
I investigated this again and i found this that the condition needs parenthesis in the first part like this:

vendor/mapado/rest-client-sdk/src/SdkClient.php:209
$isAllowedMethod = ('jsonSerialize' === $method || '__set' === $method || '__isset' === $method) && 'id' === $parameters['name'];
and whit that parenthesis works.........i don't know why hahaha
maybe something about php logic or something.

from rest-client-sdk.

The isset needs to be coupled with the $id so putting the parenthesis does make other call to bug.
I will have a look at this tomorrow because it's not easy to debug on my phone 😁

Can you copy your entity Metadata to reproduce?

Thanks

from rest-client-sdk.

OK So I reviewed briefly the code.
First of all, you referenced line 209 for $isAllowedMethod, so you might not be on the latest version of the package.

Secondly, the $isAllowedMethod is possibly not a great name for the variable, but you can see it as $isAllowedMethod = !$willInstanciateProxy
ie. if $isAllowedMethod is false is the normal case when calling any method that need to initialize object.
The only calls you can do without initializing the object is calling isset($obj->getId()), call json_encode($obj) or calling the magic setter.

Can you make dump the return of $agenteRepository->find(20)->getOficina()->getNumero() ?
Theoretically, it should :

1. make a call to /agentes/20
2. make a call to /oficinas/xxx
3. return the office number.

from rest-client-sdk.

@rubiodamian is this bug still opened ?

from rest-client-sdk.

Closing due to inactivity. Feel free to re-open and provide more info if it is still relevant.

from rest-client-sdk.