Anomalie #559
fermé
Galette v0.7.4dev. Auto-inscription bloquée au niveau du titre.
100%
Description
Il s'agit d'une Galette v0.7.4dev installée sous Xampp1.8.1 avec la nightly du 20 février matin.
L'auto-inscription déclenche une alerte sur le champ titre :
Logs :
::1 - 2013-02-20 05:36:00 - 4 - PHP Warning: trim() expects parameter 1 to be string, object given in D:\xampp\htdocs\galette074dev\lib\Galette\Entity\Adherent.php on line 1006 ::1 - 2013-02-20 05:37:17 - 4 - PHP Warning: trim() expects parameter 1 to be string, object given in D:\xampp\htdocs\galette074dev\lib\Galette\Entity\Adherent.php on line 1006 ::1 - 2013-02-20 05:37:59 - 4 - PHP Warning: trim() expects parameter 1 to be string, object given in D:\xampp\htdocs\galette074dev\lib\Galette\Entity\Adherent.php on line 1006
(bravo pour la bascule de machine et merci !)
Mis à jour par Roland Telle il y a environ 11 ans
En complément, je note que dans la Configuration des champs, le champ titre est par défaut rendu obligatoire bien que le popup ait l'air de dire le contraire :
mais qu'il est quand même possible de rendre le champ invisible (bien qu'obligatoire ...).
Mis à jour par Roland Telle il y a environ 11 ans
Je viens de regarder la table galette_fields_config ; le champ titre_adh a la valeur 0 dans required c'est-à-dire qu'il ne devrait pas ipso facto être obligatoire ? Non ?
Mis à jour par Roland Telle il y a environ 11 ans
Roland Telle a écrit :
Je viens de regarder la table galette_fields_config ; le champ titre_adh a la valeur 0 dans required c'est-à-dire qu'il ne devrait pas ipso facto être obligatoire ? Non ?
Ha, j'ai pigé !
Lorsque dans la page Configuration des champs, on choisit pour Visibilité la valeur Non alors que Obligatoire a la valeur Oui, le message indiquant une incohérence à traiter a été remplacé par une bascule de Obligatoire de Oui vers Non. Génial ça !!
Résumons :
1. l'anomalie signalée à l'origine (1e image) est confirmée,
2. le détail bizarre du popup (2e image) est confirmé (à moins que le champ ne puisse être marqué comme requis parce qu'il l'est déjà et/ou parce qu'il est impossible de changer l'option ... (mais c'est pas grave),
3. en revanche, même après avoir donné la valeur Non à Visibilité, le Titre apparaît toujours en auto-inscription et à Ajouter un adhérent (même après avoir détruit le cache on-sait-jamais) ; en revanche, lors de l'auto-inscription, il n'y a plus obligation à choisir une option dans Titre (anomalie n°1 disparue).
Hm ... Suis-je clair ? ;(
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Assigné à mis à Johan Cwiklinski
Je retiens qu'il se prend les pieds dans le tapis :D
Mis à jour par Johan Cwiklinski il y a environ 11 ans
À priori, c'est la configuration par défaut qui pose un souci ; le fait de simplement enregistrer la configuration des champs "résout" le problème
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit c15683b427e4ce3fe4d34eb558b540d9c4bfa60a.
Mis à jour par Roland Telle il y a environ 11 ans
L'affichage ou le non affichage du champ Titre est en effet résolu mais l'administrateur n'a plus la possibilité de rendre ce champ obligatoire, les options étant grisées et verrouillées sur Non
Si on rend ce champ obligatoire d'un coup de PhpMyAdmin [pour voir ;)], l'incident initial est reproduit (1e image).
Mis à jour par Johan Cwiklinski il y a environ 11 ans
Certains champs (comme le titre, ou le statut), ne peuvent être marqués comme obligatoires, car ça pose des soucis. D'autre part, si le titre est affiché, et comme il n'y a pas de valeur vide ; le champ devient de facto obligatoire (tout comme la liste des statuts en fait) ;)
Mis à jour par Roland Telle il y a environ 11 ans
Johan Cwiklinski a écrit :
Certains champs (comme le titre, ou le statut), ne peuvent être marqués comme obligatoires, car ça pose des soucis. D'autre part, si le titre est affiché, et comme il n'y a pas de valeur vide ; le champ devient de facto obligatoire (tout comme la liste des statuts en fait) ;)
Cela signifie-t-il que le 1er titre dans l'ordre d'affichage (Monsieur après l'installation) est enregistré pour toutes les fiches, par omission ??
Mis à jour par Johan Cwiklinski il y a environ 11 ans
Oui, mais il me semble qu'en fait c'était déjà comme ça avec l'ancienne méthode (les 3 boutons radio M/Mme/Mlle).
Il en va de même pour le statut, mais là on peut présumer d'un défaut, ce qui n'est absolument pas le cas pour le titre...
Mis à jour par Roland Telle il y a environ 11 ans
Johan Cwiklinski a écrit :
Oui, mais il me semble qu'en fait c'était déjà comme ça avec l'ancienne méthode (les 3 boutons radio M/Mme/Mlle) [...].
C'est exact. Mais le choix se faisait par bouton-radio avec une préselection bien visible sur Monsieur. Il faudrait, lorsque le champ Titre est obligatoire, avoir comme première ligne de menu un Veuillez choisir mais aussi que le pointeur de la souris mette en surbrillance le choix possible quand il balaie le menu déroulant (ce qui donnerait l'impression que le choix va bien être pris en compte). Comme ci-dessous :
avec un popup "Veuillez renseigner le champ obligatoire Civilité" en cas de non-exécution.
Si le champ Titre (que j'appelle Civilité dans l'exemple) n'était pas obligatoire, l'injonction "Veuillez choisir" n'apparaîtrait pas et l'absence de sélection se traduirait pas la valeur "Non défini" (dans la fiche de l'adhérent).
Mis à jour par Johan Cwiklinski il y a environ 11 ans
Oui, on est d'accord :)
Les derniers commits permettent de rendre ce champ requis. Dans tous les cas, une option "Non renseigné" a été ajoutée en début de liste ; et le message d'erreur (le faux positif) est maintenant corrigé.
Mis à jour par Johan Cwiklinski il y a environ 11 ans
Il faut modifier la base aussi et passer le champ titre_adh de la table galette_adherents de NOT NULL à NULL, et mettre NULL comme valeur par défaut.