Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Champs "Masqué par défaut, sauf si" n'apparait jamais dans le ticket #880

Closed
jc-j opened this issue Jan 4, 2018 · 30 comments · Fixed by #901
Closed

Champs "Masqué par défaut, sauf si" n'apparait jamais dans le ticket #880

jc-j opened this issue Jan 4, 2018 · 30 comments · Fixed by #901
Assignees
Labels
Milestone

Comments

@jc-j
Copy link

jc-j commented Jan 4, 2018

Bonjour,
Un champ ayant la propriété "Masqué par défaut, sauf si" ne s'affichera jamais dans le ticket même si le "sauf si" est valide.
Je précise que tout cela fonctionnait "avant" avec Formulaires 2.4.0, par contre je ne me rappelle plus exactement quelle version de glpi j'avais. Les mises à jours se sont passées sans problèmes apparents.
Merci

@btry
Copy link
Collaborator

btry commented Jan 4, 2018

Bonjour

Vous observez ce comportement avec des types de champs particuliers et/ou des valeurs particulières, ou bien pour vous cela ne fonctionne pas du tout ?

@jc-j
Copy link
Author

jc-j commented Jan 4, 2018

Bonjour
Cela ne fonctionne pas du tout.
Je décris : J'ai un premier champ (bouton radio avec 4 possibilités). Suivant ce qui est coché, cela fait apparaitre les champs suivants. Pour cela, les champs suivants ont la propriété "Masqué par défaut, sauf si" et la condition dépend du premier champ (bouton radio) et de ce qui est coché.
Merci

@jc-j jc-j closed this as completed Jan 4, 2018
@btry btry reopened this Jan 4, 2018
@btry
Copy link
Collaborator

btry commented Jan 9, 2018

bonjour

Je viens de faire un essai avec un formularie simple: des boutons radio avec les choix a b c et d et un champ de texte. Pour moi ça fonctinne avec la version 2.6.1 + mes premiers patchs.

Soit le bug est déjà résolu, soit les valeurs de vos boutons radio ont des particularités (apostrophes, caractères accentués).

Pouvez-vous me montrer les valeurs de vous boutons radio, pour que je tente de reproduire avec les elles ?

@btry btry self-assigned this Jan 9, 2018
@andreseliseo
Copy link

Hi,
We have the same issue with GLPI 9.2 and forms 2.6.1 last commit. This kind of questions (with option "hidden unless") does not get a number in description of ticket ##FULLFORM##. The only questions getting numbers 1), 2), 3) are those with option "Always displayed".
This didn't happen on version 2.5.2 and GLPI 9.1

@andreseliseo
Copy link

Hi,
Just a little more information about this issue:
commit d723a47: This issue is not present. Every question the user answers in his form are getting enummerated as 1), 2), 3)...

commit 5f87e97: We observe the issue. Questions that has "Hidden unless" option enabled, are filled in form but are not appearing in ticket description. The ennumerated questions are only those with "allways displayed" option. There is no gap between this ennumerated questions.

@btry
Copy link
Collaborator

btry commented Jan 22, 2018

Hi

Thanks, I think this will be helpful to find what happens.

@eckorion
Copy link

Bonjour,

Je reproduit un dysfonctionnement similaire voilà comment:
question 1 choix : 1,2,3
question 2 choix : 1,2,3
question 3 choix : 1,2,3 masquée sauf si q2 = 3
question 4 masquée sauf si q1 = 1 ou q2 = 1 ou q3 =1

En fait la question 4 n'est visible que lorsque la question 3 est visible et = 1 même si q1 ou q2 sont = 1

Voilà, j'espère que ça peut vous aider.

Bonne journée

@flobidon
Copy link

Bonjour,

Même problème pour moi depuis que je suis passé sous la v2.6.1 (GLPI 9.2.1).
Tous mes formulaires qui comportent une zone de texte qui sont masqué sauf si la réponse à une question est définie, ne renvoient plus les réponses dans le fullform.
Si je passe cette zone en 'toujours visible', plus de problème.
Assez contraignant sachant que pour certains formulaire, la zone de texte est le descriptif du problème utilisateur, donc rien n’apparaît pour les techniciens.

Dispo pour test

@btry
Copy link
Collaborator

btry commented Feb 1, 2018

Hi

Please test the above patch and feedback

@flobidon
Copy link

flobidon commented Feb 1, 2018

Hi,

I confirm, the patch fix the problem.

Thanks

@eckorion
Copy link

eckorion commented Feb 1, 2018

Bonjour,
En ce qui me concerne, une partie du problème est toujours présente, il est nécessaire que toutes les questions entrant dans la condition soient "visibles". Si l'une d'entre elle ne l'est pas, le test ne fonctionne pas.
Cordialement

@btry
Copy link
Collaborator

btry commented Feb 1, 2018

Donc le patch fonctionne pour un cas simple (2 questions dont une est masquée selon l'autre), mais votre cas, plus complexe, échoue toujours. C'est cela ?

@flobidon
Copy link

flobidon commented Feb 1, 2018

effectivement de mon côté, je n'ai que des cas simple d'utilisation tel que décrit par btry.
Je vois si je peux faire un test.
@eckorion tu aurais un exemple?

@btry
Copy link
Collaborator

btry commented Feb 1, 2018

Si un scénario à 3 questions plante, il peut être utile. Moins il y a de questions plus ce sera facile à examiner.

@eckorion
Copy link

eckorion commented Feb 2, 2018

Bonjour,
Voici les questions testées:
image
image
image
Et le résultat lorsqu'on coche R3 dans Q1
image
Q3 n'apparait pas.

@eckorion
Copy link

eckorion commented Feb 2, 2018

Seule la réponse à Q2 permet d'afficher Q3
image

@btry
Copy link
Collaborator

btry commented Feb 2, 2018

Bonjour

Je reproduis le scénario. Je remarque que sélectionner la éponse R3 dans Q2 ne fait pas non plus apparaitre Q3.

Par contre, je remarque que Q3 apparait à partir du moment où les deux questions ont un élément sélectionné, et que Q1 ou Q2 vaut R3.

Il me semble que Q3 n'apparait pas car une des questions n'a aucune valeur sélectionnée.

@btry
Copy link
Collaborator

btry commented Feb 2, 2018

@eckorion

Essayez le patch #902; il devrait résoudre votre souci. N'oubliez pas de me faire un retour.

@eckorion
Copy link

eckorion commented Feb 2, 2018

Je viens de faire le test, j'ai toujours la même erreur.

@btry
Copy link
Collaborator

btry commented Feb 2, 2018

Etrange. J'ai fait ceci, et j'ai reproduit

image

image

image

image

A tout hasard, avez vous bien créé q3 après q1 et q2 ? J'ai remarqué fin 2017 que les conditions pouvaient ne pas fonctionner si une question Q2 dépendant de Q1 est créée avant Q1. Le fix n'est pas dans 2.6.x mais dans la future 2.7.0.

@btry
Copy link
Collaborator

btry commented Feb 2, 2018

Prévisu:

image

image

image

image

@eckorion
Copy link

eckorion commented Feb 2, 2018

Je pense que c'est parce que vous avez Q2 "toujours visible" s'il y a une condition sur Q2 ça ne fonctionne plus exemple Q2 "masqué par défaut sauf si Q1 = 2"

@btry
Copy link
Collaborator

btry commented Feb 2, 2018

ah, oui, j'ai raté ceci. Cela dit ca m'a fait cibler un autre souci.

@andreseliseo
Copy link

Hi all,
We test the first patch and, in our case, the patch fix the issue. But we only have simple cases. Let us know when the more complex scenario gets solved. And thanks for the patches.

Testing various scenarios, we discover another issue. New line in every "Textarea" box, appears as "n" characters and there is no newline at all. This does not occur in previous commits. Don't know exactly where the issue appears. Is there an open issue for this???

@btry
Copy link
Collaborator

btry commented Feb 4, 2018

Hi

New line in every "Textarea" box, appears as "n" characters and there is no newline at all.

I think there is already an issue. I remember someone talked about this one or two weeks ago.

I'll work on a other patch soon.

btry added a commit to btry/formcreator that referenced this issue Feb 5, 2018
@btry
Copy link
Collaborator

btry commented Feb 5, 2018

Hi

The PR #902 should fix all problems related to conditions reported in this issue.

Please feedback.

@eckorion
Copy link

eckorion commented Feb 5, 2018

Hi,
Great job, it works fine for me!
Best regards

@btry
Copy link
Collaborator

btry commented Feb 5, 2018

Great ! Waiting for other feedbacks, and once the PR merged I plan to release the version 2.6.2.

@flobidon
Copy link

flobidon commented Feb 6, 2018

J'ai tester le patch, cela fonctionne pour moi également.

@btry
Copy link
Collaborator

btry commented Feb 6, 2018

Good news ! Thank you for your feedback !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants