Project

General

Profile

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

Added by Titouan Mesot over 3 years ago. Updated about 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
Base de données
Target version:
-
Start date:
09/07/2016
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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


Files

Associated revisions

Revision 749d3cc5 (diff)
Added by Johan Cwiklinski about 3 years ago

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

History

#1

Updated by Titouan Mesot over 3 years ago

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

#2

Updated by Johan Cwiklinski about 3 years ago

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

Updated by Johan Cwiklinski about 3 years ago

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

Updated by Johan Cwiklinski about 3 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF