Discussion :

[bndd24]

Salut a tous,

Voila, je souhaite insérer en bdd le nom du fichier (image) associé a l'utilisateur connecté, fichier que je stock dans un dossier public/uploads.

Je ne sais pas comment je dois faire exactement.

Voici mon controller :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
    /**
     * @Route("/edit", name="user_edit", methods={"GET","POST"})
     * 
     * @IsGranted("ROLE_USER")
     */
    public function edit(Request $request, FileUploader $fileUploader): Response
    {
        $form = $this->createForm(UserType::class, $this->getUser());
        $form->handleRequest($request);
 
        if ($form->isSubmitted() && $form->isValid()) {
 
            /** @var UploadedFile $uploadFile */
            $uploadFile = $form['avatar']->getData();
            if ($uploadFile) {
                $uploadFileName = $fileUploader->upload($uploadFile);
                //$user->setAvatar($uploadFileName); // Comment je fais pour insérer le nom du fichier dans ma table ?
            }
 
            $this->getDoctrine()->getManager()->flush();
 
            $this->addFlash(
                'info',
                'Votre compte vient d\'être modifié avec succès.'
            );
 
            return $this->redirectToRoute('user_edit');
        }
 
        return $this->render('user/edit.html.twig', [
            'form' => $form->createView(),
        ]);
    }

J'ai bien essayé de passer User $user dans les paramètres de la méthode mais j'ai une erreur :

Unable to guess how to get a Doctrine instance from the request information for parameter "user".

Voici mon formulaire au cas ou...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
<?php
 
namespace App\Form;
 
use App\Entity\User;
use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\Validator\Constraints\File;
use Symfony\Component\OptionsResolver\OptionsResolver;
use Symfony\Component\Form\Extension\Core\Type\FileType;
 
class UserType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('firstName')
            ->add('lastName')
            ->add('email')
            ->add('avatar', FileType::class, [
                'mapped' => false,
                'constraints' => [
                    new File([
                        'maxSize' => '1024k',
                        'mimeTypes' => [
                            'image/jpeg',
                            'image/png',
                        ],
                        'mimeTypesMessage' => 'Veuillez télécharger une photo au format jpeg ou png',
                    ])
                ],
                'attr' => [
                    'onchange' => 'previewFile()'
                ]
            ])
        ;
    }
 
    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults([
            'data_class' => User::class,
        ]);
    }
}

Je vous remercie

[Alexandre T]

Je n'ai pas bien compris ce qui plantait, désolé.

Déjà, peux-tu nous dire quel est ce service que tu nommes le FileUploader ?

Pour User, il semble que c'est une page pour que l'utilisateur change son propre avatar. Dans ce cas, je dirais qu'il suffit de faire cela :

• Ajouter le service SluggerInterface pour être sûr qu'on te balance pas des caractères à la ... dans ton nom de fichier,
• Choper l'utilisateur avec $user = $this->getUser();
• L'entité User doit une propriété $avatar de type string,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
    /**
     * @Route("/edit", name="user_edit", methods={"GET","POST"})
     * 
     * @IsGranted("ROLE_USER")
     */
    public function edit(Request $request, EntityManagerInterface $entityManager, SluggerInterface $slugger): Response
    {
        $user = $this->getUser();
        $form = $this->createForm(UserType::class, $user);
        $form->handleRequest($request);
 
        if ($form->isSubmitted() && $form->isValid()) {
 
            /** @var UploadedFile $uploadFile */
            $uploadFile = $form['avatar']->getData();
            if ($uploadFile) {
                $originalFilename = pathinfo($uploadFile->getClientOriginalName(), PATHINFO_FILENAME);
                //On évite les cochoncetés dans le nom de fichier
                $safeFilename = $slugger->slug($originalFilename);
                $newFilename = $safeFilename.'-'.uniqid().'.'.$uploadFile->guessExtension();
 
                // On déplace le fichier
                try {
                    $uploadFile->move(
                        $this->getParameter('avatar_directory'),//ça tu le configure
                        $newFilename
                    );
                } catch (FileException $e) {
                    // ... On n'a pas pu déplacer le fichier 
                }
 
                // Mettre à jour l'utilisateur
                $user->setAvatar($newFilename);
            }
            // Ouh ça marche, mais c'est moche !
            //$this->getDoctrine()->getManager()->flush();
            $entityManager->flush;
            $this->addFlash(
               'info',
               'Votre compte vient d\'être modifié avec succès.'
            );
 
            return $this->redirectToRoute('user_edit');
        }

Symfony a fait une excellente documentation sur ce sujet

Enfin si tu upload plein d'autres fichiers, tu peux te tourner vers le VichUploaderBundle. Mais pour un vava c'est un peu l'artillerie lourde.

[bndd24]

excellent Alex tu as résolu le problème, peut être même sans le savoir

En résumé, je voulais injecter la valeur du nouveau nom de fichier dans la table user mais je ne savais pas que je pouvais faire un setter avec le "getUser();" comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$user = $this->getUser();
// ...
$user->setAvatar($uploadFileName);

en même temps c'est trompeur car c'est écrit GETuser....... je pensais pouvoir seulement récupérer et non injecter, je ne sais pas si tu vois ou je veux en venir, enfin bref problème résolu !

pour répondre a ta question concernant mon "FileUploader" c'est un Service que voici (justement sur la base de la doc!) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
<?php
 
namespace App\Service;
 
use Symfony\Component\HttpFoundation\File\Exception\FileException;
use Symfony\Component\HttpFoundation\File\UploadedFile;
 
class FileUploader
{
    private $targetDirectory;
 
    public function __construct($targetDirectory)
    {
        $this->targetDirectory = $targetDirectory;
    }
 
    public function upload(UploadedFile $file)
    {
        $fileName = uniqid() . '.' . $file->guessExtension();
 
        try {
            $file->move($this->getTargetDirectory(), $fileName);
        } catch (FileException $e) {
            // ... handle exception if something happens during file upload
        }
 
        return $fileName;
    }
 
    public function getTargetDirectory()
    {
        return $this->targetDirectory;
    }
}

QUESTION : J'ai bien lu ton commentaire a propos du SluggerInterface : J'ai volontairement supprimé le SluggerInterface car je renomme le fichier de façon aléatoire, est-ce toujours dangereux ?

Je te remercie pour ton aide précieuse !

[Alexandre T]

uniqid te garantit que tu es peinard pour les chaines de caractères. Pour des vavas publics, ça va très bien. Si tu stockais des fichiers importants, je te recommanderais d'utiliser une autre fonction. Un utilisateur reconnaissant la signature de la fonction uniq_id pour ses documents pourrait s'amuser à scanner les fichiers. A parti de symfony 5.1, j'utiliserai cela : https://symfony.com/doc/5.1/components/uid.html

[bndd24]

ok je ne connaissais pas le UID, c'est intéressant, peut-être la future "norme" a adopter.. pour pour l'info. Je passe le post en résolu.