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 plus d'un an. Mis à jour il y a plus d'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%

Temps estimé:
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

Révisions associées

Révision 749d3cc5 (diff)
Ajouté par Johan Cwiklinski il y a plus d'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 plus d'un an

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

#2 Mis à jour par Johan Cwiklinski il y a plus d'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 plus d'un an

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

#4 Mis à jour par Johan Cwiklinski il y a plus d'un an

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF