nilportugues / symfony-jsonapi

JSON API Transformer Bundle for Symfony 2 and Symfony 3
http://nilportugues.com
MIT License
115 stars 19 forks source link

Fix some namespaces on JsonApiResponseTrait #15

Closed obokaman-com closed 8 years ago

obokaman-com commented 8 years ago

Bones @nilportugues! Estem mirant d'utilitzar el Bundle en un projecte amb l'Albert Lombarte (crec que ja t'ha comentat alguna cosa, pel repo d'exemple que he vist por ahí :D), però ens trobem que el JsonApiResponseTrait crida a alguns namespaces que semblen no existir ja a la llibreria "json-api".

He mirat de trobar les "correspondències" actualitzades, i en algun cas simplement ha estat corregir una part del namespace, d'altres he vist que canviava la manera de construïr els errors (fes-hi una ullada a veure si tenen sentit els canvis que apunto més avall), però em queden 3 casos sense resoldre. Si em pots orientar una mica amb el tema puc mirar de fer les modificacions necessàries.

Gràcies per l'ajuda i per la feina feta amb aquestes llibreries. ;-)

nilportugues commented 8 years ago

@obokaman-com el problema es que em vaig colar. Ara pujo un fix amb els objectes que toquen

nilportugues commented 8 years ago

@obokaman-com 1.3.1 ho hauria d'arreglar tot.

No dono a l'abast amb la popularitat d'aquesta llibreria :sweat_smile:

Merci! Intentaré contestar més ràpid :+1:

obokaman-com commented 8 years ago

Gràcies @nilportugues

Sembla que està quasi resolt! Només fallen encara els mètodes d'error. Els errors que extenen de AbstractErrorResponse, como ara el ResourceNotFound, que es crida des del mètode resourceNotFoundResponse del trait, esperen rebre un ErrorBag al constructor, i en canvi el trait actualment els hi passa un string json.

Precisament d'aqui venia el canvi que feia a https://github.com/nilportugues/symfony-jsonapi/pull/15/files#r58279025

Fins ara i merci de nou per la feina feta i l'ajuda. ;-)

nilportugues commented 8 years ago

@obokaman-com miro com ho tinc al package de laravel (vaig unificar dependències) i ho arreglo. Merci per la paciència :+1:

nilportugues commented 8 years ago

@obokaman-com https://github.com/nilportugues/symfony-jsonapi/releases/tag/1.3.2