Add required headers and parameter checks

[neomerx]

For example media types requirements

The easiest way to meet the spec is to replicate the trait

i think to use the middleware instead of trait method because this project is special to jsonapi on phalcon framework.

[neomerx]

It requires the app to send raw objects rather than Responses. Would it be natural to phalcon users?

Does the library send PHP object?
i thought it sends raw JSON

[neomerx]

Middleware is something between the app and the user. The following image shows them as yellow circles

If data conversion is placed in middleware it will require the app to send objects/arrays/iterators and the middleware will convert them to Responses with json. Are you planning to make such middleware?

The phalcon it self returning raw Phalcon\Http\Response object see the docs and my latest code [line#44](https://github.com/sumeko/phalcon-jsonapi/blob/master/routes.php #L44).

anyway, is there any better way?

[neomerx]

@sumeko If you plan to implement this approach

return $app->encoder->encode(Comment::find());

it might be design issue. Users actually send desired format with Accept header and it could be like this

text/plain; q=0.5, text/html, text/x-dvi; q=0.8, text/x-c

When you return just text (json, xml, html) and then in middleware add headers you can't know actual data type.

Headers needs to be analyzed and when you convert data you need to know the requested type.

Also have a look at Location header requirement. With this approach it can't be implemented as you lose this info after returning json text.

[neomerx]

shortly it's better to return Response with correct body, code and headers and it should have got requested format

Oh i see,
Phalcon has before middleware event to analyze the request headers, so i think it still ok to use middleware.
but i will implement the trait method also

[neomerx]

IMHO it could be trait or controller or some helper class. Choose what is natural for phalcon community.
My most concern is filling correct Response.

@neomerx
How to just return array instead of json with the lib?

[neomerx]

inherit/extend Encoder and override encodeToJson method

[neomerx]

how is it going? is anything to look at? any help needed?