stadline / linkdata-client Goto Github PK

Nous avons actuellement aucun tests sur ce que nous développons sur le client.
Il faut que ça change :-/

Je pense que ce serait plus judicieux de passer directement ce qui nous interesse de la config en param plutot que de passer un array fourre tout.
Ca permettrait l'autowiring, des tests bien plus simple, une meilleure compréhension et plus de rigidité dans le process.

Ca éviterait les lignes plus bas du style :

 if (!\array_key_exists('authorization', $args)) {
            throw new AuthenticationException('Authorization token not found.');
        }

Actuellement on a mis des string partout pour simplifier.

J'ai vu qu'on pouvait ajouter des listener au moment de la désérialization pour qu'il génère bien les \DateTime.

Comme dans un autre ticket ici la configuration doit être passée en paramètres du construct pour l'autowiring etc.

Encore un parse de la config qui ne devrait pas être là.

Je ne comprend pas pourquoi il y a un catch ici ... si on renvoie ce qu'on vient juste de catch, ça ne sert à rien

Lorsqu'on a 0 ou 1 résultat sur un Get, le paginator nous retourne un array. A partir de 2 résultats, le paginator nous retourne un objet Paginator.

On re-génére un nouveau proxymanager à chaque fois que l'on appelle la méthode, il vaudrait mieux le garder de côté.

Je ne comprend pas pourquoi cette méthode existe. Elle est utilisée dans le proxyObject pour récupérer le proxyManager, mais dans ce cas autant passer directement le proxyManager en param du proxyObject, c'est bien plus simple non ?

J'ai vu voir ici que nous n'avons pas d'interface pour l'adapteur ... pourquoi ?
Pouvez-vous m'expliquer le role exact de l'adapteur et de ses méthodes ?