Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BO - Edition de photos] Ajouter limite de caractères pour la description #2453

Conversation

hmeneuvrier
Copy link
Collaborator

Ticket

#2438

Description

Limiter la taille du champ description pour les fichiers

Changements apportés

  • Migration + limitation dans l'entité
  • Limitation dans les champs lors de l'ajout et de l'édition

Pré-requis

Tests

  • Ajouter une photo de visite, vérifier qu'on ne peut pas mettre une description de plus de 250 caractères
  • Editer une photo de visite et vérifier aussi la limitation

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Questions d'utilisation :
J'ai bien la limite, pas de soucis
1/ Par contre, la description que je saisis, elle doit s'afficher où ? Ca sert à quoi ? :)
2/ Et aussi, si je réédite le même doc, ce que j'avais mis en description précédemment ne s'affiche pas dans la modale.
3/ C'est normal que dans form_upload_document.js on utilise un input, et dans edit-file.html.twig on utilise un textarea ?

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deux petites remarque de code, et deux autres ci-dessous :

  • On met un text-area mais à l'affichage de la photo on ne reprend pas les saut de ligne (mais c'est peut être une bonne chose pour éviter des soucis de design)
  • Si je cherche la petite bête en remplissant le champ de "é" la soumission crash (mais si ca se trouve c'est un comportement en local uniquement)

src/Entity/File.php Show resolved Hide resolved
src/Entity/File.php Show resolved Hide resolved
@hmeneuvrier
Copy link
Collaborator Author

hmeneuvrier commented Apr 16, 2024

@emilschn

1/ Par contre, la description que je saisis, elle doit s'afficher où ? Ca sert à quoi ? :)

Elle s'affiche dans l'album photo, cela sert à décrire la photo de visite.

2/ Et aussi, si je réédite le même doc, ce que j'avais mis en description précédemment ne s'affiche pas dans la modale.

je ne reproduis pas, par contre il y avait un souci pour l'édition d'une photo de visite, c'est peut-être ça ?

3/ C'est normal que dans form_upload_document.js on utilise un input, et dans edit-file.html.twig on utilise un textarea ?

Moui, c'est un peu nul, mais dans la modale d'upload on n'a pas vraiment la place de mettre un textarea. Mathilde a validé comme ça, mais a demandé à mettre un textarea si possible dans la modale d'édition...

@numew

On met un text-area mais à l'affichage de la photo on ne reprend pas les saut de ligne (mais c'est peut être une bonne chose pour éviter des soucis de design)

j'ai ajouté nl2br, mais effectivement je pense que ce n'est pas une super idée car on ne voit plus le nom de la personne si on met trop de retours à la ligne. Je l'enlève

Si je cherche la petite bête en remplissant le champ de "é" la soumission crash (mais si ca se trouve c'est un comportement en local uniquement)

je ne reproduis pas, par contre il y avait un souci pour l'édition d'une photo de visite, c'est peut-être ça ?

Copy link

sonarcloud bot commented Apr 16, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@numew numew merged commit f9af85e into develop Apr 16, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the bugfix/2438-bo-edition-de-photos-ajouter-limite-de-caracteres-pour-la-description branch April 26, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants