Bien charter l'adhésion personne physique

[agallou]

La page d'adhésion des personnes physiques ressemble à cela (https://afup.org/admin/register) :

La page d'adhésion des personnes morales ressemble elle à cela (https://afup.org/association/adherer/entreprise) :

C'est historique, cette page n'est pas chartée. Il faudrait la charter comme le reste du front (pour garder le contexte d'enête/footer notamment).

L'inscription est gérée ici :

web/sources/AppBundle/Controller/LegacyController.php

Line 73 in baea202

public function registerAction(Request $request)

Elle utilise smarty.

L'adhésion entreprise est géré elle ici :

web/sources/AppBundle/Controller/MemberShipController.php

Line 54 in e76b355

public function companyAction(Request $request)

Il faudrait passer le formulaire sur un form symfony afin de pouvoir basculer sur un template twig et pouvoir hériter du template :admin/association/membership:_base.html.twig.

[vinceAmstoutz]

@agallou que veut-tu dire par charter ?

[agallou]

@vinceAmstoutz par charter je veux dire que la page respecte la charte, le design d'afup.org. Qu'il y a ait entre autres, en haut les deux menus bleus, le footer et que le formulaire ressemble à celui des entreprises.

[Zarlokh]

Est-ce qu'on peut changer aussi l'url ou il faut pour x ou y raisons absolument conserver l'url actuelle ?

[agallou]

@Zarlokh on peux changer l'url, c'est même mieux,
pour les adhésions entreprises c'est https://afup.org/association/adherer/entreprise on peux donc partir sur https://afup.org/association/adherer/particulier pour les particuliers (et on mettra en place une redirection coté apache, dans un htaccess).

[Zarlokh]

@agallou J'ai réussi à trouver le temps d'avancer sur le sujet
J'ai vu qu'il y a pas mal de valeur par défaut qui sont défini à la soumission du formulaire via des input hidden que j'ai normalement repris (faut que je regarde si les tests en place vérifie ça ou si il faut en faire)
Dans tous les cas, je regardais pour bien intégrer dans la charte et je me rends compte que lorsqu'un champs est invalide, on a un décalage de l'interface, champs pas en rouge, erreur pas en rouge etc...

Quand j'ai regardé sur le formulaire d'adhésion pour une entreprise, j'ai vu que c'est la même chose en réalité

Je me demandais si je laisse comme ça (semble être un form_theme a mettre de manière global pour l'ensemble du site sous sf donc peut-être une autre issue ?) ou si je corrige ce problème de manière unitaire sur ma page ?

J'ai également vu qu'il n'y a pas de vérification du format du numéro de téléphone
Est-ce que je regarde pour intégrer un libphonenumber afin de valider les num de tel ou encore une fois, c'est pas le but de l'issue donc je laisse pour une autre issue ?

[agallou]

@Zarlokh pour la vérification du numéro de téléphone j'aurais tendance à le faire dans une autre PR pour qu'on déploie ces changements au fur et à mesure
pour le souci de décalage, si tu arrives à le corriger dans cette PR sans que ça ne soit trop embêtant/ne prennes trop de temps, oui autant le faire maintenant, sinon on le fera plus tard vu que le problème est déjà présent

[agallou]

Traité dans la PR #1516