Response NS

[tuupola]

Response NS

[tuupola]

You are correct it is a bug. It seems I have assumed Headers object is immutable. Thanks for the heads up!

[B3B8]

Mika can I ask you a tip to reduce the serialize lines and respective response ie

$app->post("/todos", function ($request, $response, $arguments) {
 [...]
    /* Serialize the response data. */
    $fractal = new Manager();
    $fractal->setSerializer(new DataArraySerializer);
    $resource = new Collection($todos, new TodoTransformer);
    $data = $fractal->createData($resource)->toArray();

    return $response->withStatus(200)
        ->withHeader("Content-Type", "application/json")
        ->write(json_encode($data, JSON_UNESCAPED_SLASHES | JSON_PRETTY_PRINT));

$app->get("/todos/{uid}", function ($request, $response, $arguments) {
 [...]
    /* Serialize the response data. */
    $fractal = new Manager();
    $fractal->setSerializer(new DataArraySerializer);
    $resource = new Item($todo, new TodoTransformer);
    $data = $fractal->createData($resource)->toArray();

    return $response->withStatus(200)
        ->withHeader("Content-Type", "application/json")
        ->write(json_encode($data, JSON_UNESCAPED_SLASHES | JSON_PRETTY_PRINT));

Start from todos routes methods, compliting my others API routes like todos with the same functions, I'll write the same serialize with the same response, so I think about "don't repeat yourself" principle and maybe with SLIM design pattern I can improve the code...Is it right use this schema or I'm just complicating myself? If it's worth, how can you rewrite it?
Thanks for your suggestion!

[tuupola]

You could extract serializing part to external facade class and it would save a couple of lines in the route. Calling the facade could look something like.

$data = (new SerializerFacade($todo, TodoTransformer::class))->toArray()

Downside is this would make code a bit harder to follow since you have to go see the source of the facade class to understand what is happening. It is always a fine line between optimising and readability.