Project

General

Profile

Anomalie #559

Galette v0.7.4dev. Auto-inscription bloquée au niveau du titre.

Added by Roland Telle about 8 years ago. Updated about 8 years ago.

Status:
Fermé
Priority:
Haut
Category:
-
Target version:
-
Start date:
02/20/2013
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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 !)

#1

Updated by Roland Telle about 8 years ago

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 ...).

#2

Updated by Roland Telle about 8 years ago

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 ?

#3

Updated by Roland Telle about 8 years ago

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 ? ;(

#4

Updated by Johan Cwiklinski about 8 years ago

  • Assignee set to Johan Cwiklinski

Je retiens qu'il se prend les pieds dans le tapis :D

#5

Updated by Johan Cwiklinski about 8 years ago

À 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

#6

Updated by Johan Cwiklinski about 8 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100
#7

Updated by Johan Cwiklinski about 8 years ago

  • Status changed from Résolu to Fermé
#8

Updated by Roland Telle about 8 years ago

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).

#9

Updated by Johan Cwiklinski about 8 years ago

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) ;)

#10

Updated by Roland Telle about 8 years ago

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 ??

#11

Updated by Johan Cwiklinski about 8 years ago

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...

#12

Updated by Roland Telle about 8 years ago

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).

#13

Updated by Johan Cwiklinski about 8 years ago

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é.

#14

Updated by Johan Cwiklinski about 8 years ago

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.

Also available in: Atom PDF