sergisergio / Ads

src/Controller/AccountController.php

<?php

namespace App\Controller;

use App\Entity\Advert;
use App\Entity\Application;
use App\Entity\User;
use App\Form\AccountType;
use App\Form\CurriculumType;
use App\Repository\ApplicationRepository;
use App\Service\FileUploader;
use Doctrine\Common\Persistence\ObjectManager;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Symfony\Component\Routing\Annotation\Route;
use Symfony\Component\HttpFoundation\File\Exception\FileException;

class AccountController extends AbstractController
{
    /**
     * @Route("/account/cv/{id}", name="add_cv")
     */
    public function curriculum(Request $request, $id, FileUploader $fileUploader)
    {
        $em = $this->getDoctrine()->getManager();
        $user = $em->getRepository(User::class)->find($id);
        //dump($user);die();

        $form = $this->createForm(CurriculumType::class, $user);
        $form->handleRequest($request);


        if ($form->isSubmitted() && $form->isValid()) {
            // $file stores the uploaded PDF file
            /** @var Symfony\Component\HttpFoundation\File\UploadedFile $file */
            $file = $form->get('curriculum')->getData();
            $fileName = $fileUploader->upload($file);

            $user->setCurriculum($fileName);

            $em->persist($user);
            $em->flush();

            $this->addFlash('success', 'Votre CV a été téléchargé');
            return $this->redirectToRoute('account', ['id' => $user->getId()]);
        }

        return $this->render('account/cv.html.twig', [
            'cvForm' => $form->createView(),
        ]);
    }

    /**
     * @Route("/account/cv/edit/{id}", name="edit_cv")
     */
    public function editCurriculum(Request $request, $id, FileUploader $fileUploader)
    {
        $em = $this->getDoctrine()->getManager();
        $user = $em->getRepository(User::class)->find($id);
        //dump($user);die();

        $form = $this->createForm(CurriculumType::class, $user);
        $form->handleRequest($request);


        if ($form->isSubmitted() && $form->isValid()) {
            // $file stores the uploaded PDF file
            /** @var Symfony\Component\HttpFoundation\File\UploadedFile $file */
            $file = $form->get('curriculum')->getData();
            ;
            $fileName = $fileUploader->upload($file);

            $user->setCurriculum($fileName);

            $em->persist($user);
            $em->flush();

            $this->addFlash('success', 'Votre CV a été téléchargé');
            return $this->redirectToRoute('account', ['id' => $user->getId()]);
        }

        return $this->render('account/cv_edit.html.twig', [
            'cvForm' => $form->createView(),
        ]);
    }

    /**
     * @return string
     */
    private function generateUniqueFileName()

The method generateUniqueFileName() is not used, and could be removed.

    {
        // md5() reduces the similarity of the file names generated by
        // uniqid(), which is based on timestamps
        return md5(uniqid());
    }

    /**
     * @Route("/account/{id}", name="account")
     */
    public function view($id, Request $request, ObjectManager $manager)
    {
        $user = $this->getDoctrine()->getRepository(User::class)->find($id);
        if (null === $user) {
            throw new NotFoundHttpException("L'utilisateur ".$id." n'existe pas");
        }
        //dump($this->getUser()->getEmail());die();
        if ($user->getEmail() === $this->getUser()->getEmail()) {
            $form = $this->createForm(AccountType::class, $user);
            $form->handleRequest($request);
            if ($form->isSubmitted() && $form->isValid()) {
                $user->setEmail($this->getUser()->getEmail());
                $manager->persist($user);
                $manager->flush();
                $this->addFlash(
                    'notice',
                    'Votre compte a bien été modifié!'
                );
                dump($request);

The call to dump() has too few arguments starting with valid.

                die();

Using exit here is not recommended.

                return $this->redirectToRoute('account', ['id' => $user->getId()]);

return $this->redirectToRoute('account', array('id' => $user->getId())) is not reachable.

            }
            return $this->render('account/index.html.twig', [
                'user' => $user,
                'formAccount' => $form->createView()
            ]);
        } else {
            $this->addFlash('warning', 'Vous ne pouvez pas accéder à cette page !');
            return $this->redirectToRoute('home');
        }
    }

    /**
     * @Route("/account/myadverts/{id}", name="my_adverts")
     */
    public function myAdverts($id)

The parameter $id is not used and could be removed.

    {
        $em = $this->getDoctrine()->getManager();

The assignment to $em is dead and can be removed.

        $adverts = $this->getDoctrine()->getRepository(Advert::class)->findByAuthor($this->getUser());

The method findByAuthor() does not exist on App\Repository\AdvertRepository. Since you implemented __call, consider adding a @method annotation.

        return $this->render('account/myadverts.html.twig', [
            'adverts' => $adverts
        ]);
    }

    /**
     * @Route("/account/myadvert/applications/{id}", name="list_applications")
     */
    public function ApplicationsByAdvert($id)
    {
        $em = $this->getDoctrine()->getManager();
        $advert = $this->getDoctrine()->getRepository(Advert::class)->find($id);

        $listApplications = $em
            ->getRepository(Application::class)
            ->findBy(['advert' => $advert])
        ;

        return $this->render('account/applications.html.twig', [
            'advert' => $advert,
            'listApplications' => $listApplications
        ]);
    }
}