Anomalie #559
fermé
Galette v0.7.4dev. Auto-inscription bloquée au niveau du titre.
Ajouté par Roland Telle il y a environ 11 ans.
Mis à jour il y a environ 11 ans.
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 !)
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 ...).
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 ?
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 ? ;(
- Assigné à mis à Johan Cwiklinski
Je retiens qu'il se prend les pieds dans le tapis :D
À 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
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
- Statut changé de Résolu à Fermé
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).
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) ;)
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 ??
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...
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).
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é.
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.
Formats disponibles : Atom
PDF
Mis à jour par 
Mis à jour par