Projet

Général

Profil

Anomalie #958

Impossible d'ajouter un nouvel adhérant si les champs Apparaître dans la liste des membres, Compte, Administrateur Galette ou Exempt de cotisation sont non-obligatoire et non-visible

Ajouté par Titouan Mesot il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Base de données
Version cible:
-
Début:
07/09/2016
Echéance:
% réalisé:

100%

Version utilisée:

Description

Hello,
Toujours dans le même sujet que le patch précédant, il est impossible d'ajouter des adhérant si les champs suivant sont invisible et non obligatoire sur une base de données postgres SQL (en tout cas) :

Apparaître dans la liste des membres,
Compte,
Administrateur Galette
Exempt de cotisation

Cela semble être lié au bugs #18899 et #19354 dont un fix a été effectué mais celui-ci ne prends pas en compte tous les champs boolean du formulaire, ni le fait que leurs valeurs peut être vide car ceux-ci ne sont pas visible.

Je propose un patch en annexe par contre, je n'ai pas fait de test sous MySQL.

Salutations,
Titouan Mesot

fix_empty_boolean_value_for_postgressql.patch Magnifier (827 octets) Titouan Mesot, 07/09/2016 22:30

Révisions associées

Révision 749d3cc5
Ajouté par Johan Cwiklinski il y a environ un an

Do not display hidden elements with empty value, let's PHP objects handle this; fixes #958

Historique

#1 Mis à jour par Titouan Mesot il y a environ un an

Oups, l'assignation n'était pas voulue ... désolé

#2 Mis à jour par Johan Cwiklinski il y a environ un an

Merci pour le patch, mais je vois plutôt la correction côté template, si le correctif fonctionne pour postgres - qui enregistrera un vrai "false" en base, on ne sait pas trop ce que fera mysql ; alors que le comportement actuel est le bon.

En gros, une simple histoire d'affichage d'un champ ne doit pas à mon avis amener la modification de la logique derrière.

#3 Mis à jour par Johan Cwiklinski il y a environ un an

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#4 Mis à jour par Johan Cwiklinski il y a environ un an

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF