fco-veragua / tweetfony

0 stars 0 forks source link

Implementar GET /tweets y GET/users #3

Closed sambrista closed 2 years ago

fco-veragua commented 2 years ago

Completado.

sambrista commented 2 years ago

Los enlaces que das están mal. Si te fijas, cuando haces por ejemplo la lista de tweets estás haciendo una lista de enlaces a la ruta "api_get_tweets", cuando debería ser "api_get_tweet". El enlace que creas es por ejemplo "https://localhost:8000/api/tweets?id=4", y debería ser "https://localhost:8000/api/users/4".

fco-veragua commented 2 years ago

Corregido.

sambrista commented 2 years ago

Comprobado.

Si no hay tweets o no hay usuarios devuelves un error 404. Esto es correcto, pero quizás habría sido más elegante inicializar $result como un array vacío y no controlar si hay tweets o no:

    function getTweets()
    {
        $entityManager = $this->getDoctrine();

        $tweets = $entityManager->getRepository(Tweet::class)->findAll();

        $result = array();

        foreach ($tweets as $tweet) {
            $result[] = $this->generateUrl('api_get_tweet', [
                'id' => $tweet->getId(),
            ], UrlGeneratorInterface::ABSOLUTE_URL);
        }
        return new JsonResponse($result);
    }

De esta manera, si hay tweets devolveremos la lista con los tweets y si no hay devolveremos la lista vacía.