[FO - Formulaire police] BatId en cas d'adresse forcee#5704
[FO - Formulaire police] BatId en cas d'adresse forcee#5704hmeneuvrier wants to merge 3 commits intodevelopfrom
Conversation
hmeneuvrier
changed the title
|
| $hasViolation = true; | ||
| } | ||
|
|
||
| if ($value instanceof FormServiceSecoursStep2) { |
There was a problem hiding this comment.
si on fait ça, il faut vérifier que l'adresse est manuelle, sinon ça bloque pour les cas simples
There was a problem hiding this comment.
En effet ca alerte y compris en adresse automatique, je pense cependant qu'il ne faut pas du tout mettre de validation à ce niveau. Vu le peu d'élément obligatoire je ne vois pas pourquoi le rnbId le serait
| ], Response::HTTP_BAD_REQUEST); | ||
| } | ||
|
|
||
| if (!$rnbId) { |
There was a problem hiding this comment.
la partie gestion d'erreur pourrait être concaténée dans un service, non ? on a 3 fois les mêmes tests pour ça
| {{ form_widget(form.step2.rnbId) }} | ||
| {{ form_widget(form.step2.lat) }} | ||
| {{ form_widget(form.step2.lng) }} | ||
| <button |
There was a problem hiding this comment.
il manque le picto sur le bouton (il faut inclure des css dsfr, il me semble)
| {% if current_step == 'step2' %} | ||
| <div data-ajax-form> | ||
| <form action="{{ sign_url(path('service_secours_complete_localisation', { | ||
| 'slug': 'ppp', |
There was a problem hiding this comment.
Ah ben ça devait être chez toi, il faudrait que tu modifies, ça ne fonctionne pas chez moi
numew
left a comment
numew
left a comment
There was a problem hiding this comment.
Le sélecteur de bâtiment ne fonctionne pas.
En se basant sans retouche sur le code de pick-localisation.js ce ne pourras pas etre ok, il intègre un système de cache car il est basé sur une adresse "fixe" enregistré sur la fiche BO, dans notre cas il faudra qu'il puisse se mettre a jour si l'adresse a changé après une seconde ouverture. (je me demande d’ailleurs si une modale était le plus approprié pour ce formulaire secours prévu pour les mobiles, mais j'ai vu que c'est les spec d'arnaud !)
| ]); | ||
| } | ||
|
|
||
| #[Route( |
There was a problem hiding this comment.
Je ne comprends pas l'utilité de cette route.
L'information qu'il nous faut stoker c'est le rnbId et on le récupère directement coté JS, si cette données doit être utilisé pour faire un traitement il faut déplacer cela dans la factory.
Il me semble que ca permettrais de simplifier je met des autres com sur les élément à supprimer si on cette option.
| attacheAutocompleteAddressEvent, | ||
| initComponentAddress, | ||
| } from '../../services/component/component_search_address'; | ||
| import { jsonResponseHandler } from '../../services/component/component_json_response_handler'; |
There was a problem hiding this comment.
Plus nécessaire si on supprime la route service_secours_complete_localisation
|
|
||
| updatePickButton(); | ||
|
|
||
| const submitBtn = document.getElementById('fr-modal-pick-localisation-submit'); |
There was a problem hiding this comment.
Plus nécessaire si on supprime la route service_secours_complete_localisation
|
|
||
| public ?string $rnbId = null; | ||
|
|
||
| public ?float $lat = null; |
There was a problem hiding this comment.
Plus nécessaire si on supprime la route service_secours_complete_localisation
|
|
||
| public ?float $lat = null; | ||
|
|
||
| public ?float $lng = null; |
There was a problem hiding this comment.
Plus nécessaire si on supprime la route service_secours_complete_localisation
| ->add('rnbId', HiddenType::class, [ | ||
| 'required' => false, | ||
| ]) | ||
| ->add('lat', HiddenType::class, [ |
There was a problem hiding this comment.
Plus nécessaire si on supprime la route service_secours_complete_localisation
| ->add('lat', HiddenType::class, [ | ||
| 'required' => false, | ||
| ]) | ||
| ->add('lng', HiddenType::class, [ |
There was a problem hiding this comment.
Plus nécessaire si on supprime la route service_secours_complete_localisation
| $hasViolation = true; | ||
| } | ||
|
|
||
| if ($value instanceof FormServiceSecoursStep2) { |
There was a problem hiding this comment.
En effet ca alerte y compris en adresse automatique, je pense cependant qu'il ne faut pas du tout mettre de validation à ce niveau. Vu le peu d'élément obligatoire je ne vois pas pourquoi le rnbId le serait
|
|
||
| {% block body %} | ||
| <main class="fr-container fr-mb-5w" id="content"> | ||
| {% include "flash-messages.html.twig" %} |
There was a problem hiding this comment.
Plus nécessaire si on supprime la route service_secours_complete_localisation
3 participants
Ticket
#5627
Description
Dans le cas de signalements usagers avec une adresse forcée, afficher la modale de selection de batiment à l'usager
Changements apportés
Pré-requis
Tests