Map Upload files into request properties

[brendt]

class MyRequest implements Request
{
    use IsRequest;
    
    public Upload $avatar;
}

[yassiNebeL]

@brendt I figure it out, could I send a PR or you will fix it? this #381 was blocking the server-side request ServerRequestInterface.

this is what I did :

final readonly class RequestToObjectMapper implements Mapper
    ...
    public function map(mixed $from, mixed $to): array|object
    {
        $data = array_merge($from->getBody(), $from->getFiles());
        /** @var Request $from */
        return map($data)->to($to);
    } 

I look forward to your response

[brendt]

Are you sure this works? Because $from->getFiles() is an array without named keys?

[yassiNebeL]

Yes, we need a named keys, any idea how we can resolve that? when I tried to map the request to a model object with ArrayToObjectMapper because we couldn't find the property name with the array keys.

[yassiNebeL]

We need to add the uploadedFiles as well, now getFiles returns an array with named keys.

final readonly class RequestToPsrRequestMapper implements Mapper
{
   ...

    public function map(mixed $from, mixed $to): PsrRequest
    {
        /** @var Request $from */

        return new ServerRequest(
            uri: $from->getUri(),
            method: $from->getMethod()->value,
            headers: $from->getHeaders(),
            cookieParams: $from->getCookies(),
            queryParams: $from->getQuery(),
            parsedBody: $from->getBody(),
            uploadedFiles: $from->getUploadedFiles()
        );
    }
}

Continuing with the Book example after adding coverPicture property the request along with the Book model this is what I got:

INSERT INTO `Book` (title, coverPicture) VALUES (:title, :coverPicture);

bindings: {
    "title": "titre 1",
    "coverPicture": null
}

Could you tell me which value should I store in the DB?

[yassiNebeL]

I'm pretty sure that the mapping is properly working, I'm waiting for your response Brendt in order to know which value we should save the path or the name, that's why I would implement a new caster.

[brendt]

I think the easiest path forward is to submit a PR that I can review ;)