Skip to content

Correction de l'issue 240

DAVID Titouan requested to merge TDavid/framadate:master into master

Résumé de l'issue #240 (closed) :

Lors de l'ajout d'un mot de passe (après la création du sondage), on a une option permettant de laisser les résultats du sondage visible ou non lorsque la personne arrivant sur la page du sondage n'a pas encore renseigner le mot de passe pour pouvoir voter.

Ceci n'est donc fonctionnel que si il y a un mot de passe, or nous pouvons cocher l'option sans pour autant renseigner de mot de passe.

De plus, il y a une autre option,indépendante du mot de passe, qui permet de rendre ou non visible les résultats du sondages. Dans le cas où :

  1. on a un mot de passe
  2. on a choisi d'avoir les résultats visibles sans mot de passe
  3. on a choisi de cacher les résultats du sondage => on se retrouve, lorsque l'on a pas encore renseigné le mot de passe, avec seulement les valeurs disponibles du sondage sans avoir le résultat. On devrait s'attendre à ne rien voir à la place.

On a 2 options contradictoires.

Changement fonctionnel concernant l'issue #240 (closed) :

  1. J'ai choisi que l'option de cacher les résultats devait être prioritaire sur celle ne s'appliquant que sur les mots de passe. C'est pourquoi lorsque la première option est activé, la deuxième doit être rendu désactivé et inactivable.

  2. L'option permettant d'avoir les résultats lorsque le mot de passe n'est pas renseigné ne doit pas être activable lorsqu'il n'y a pas/plus de mot de passe renseigné.

  3. L'option permettant d'avoir les résultats pour les mots de passe est renommé en "only votes are protected"

Changements techniques concernant l'issue #240 (closed) :

  1. dans le template tpl/part/poll_info.tpl : ajout d'une condition sur l'option "hidden" pour afficher le statut de l'option "resultsPubliclyVisible" (activé ou désactivé) ou rendre inactivable l'option (désactivé dans tout les cas) dans le php adminstuds.php : ajout d'une condition sur l'option "hidden" pour conserver le statut de l'option "resultsPubliclyVisible" (activé ou désactivé) ou écraser l'option (désactivé dans tout les cas)

  2. dans le template tpl/part/poll_info.tpl : ajout d'une condition sur le mot de passe pour afficher le statut de l'option "resultsPubliclyVisible" (activé ou désactivé) ou rendre inactivable l'option (désactivé dans tout les cas) dans le php adminstuds.php : ajout d'une condition sur le mot de passe pour conserver/modifier le statut de l'option "resultsPubliclyVisible" (activé ou désactivé) ou écraser l'option (désactivé dans tout les cas) dans le javascript js/app/studs.js : ajout d'une mise à jour dynamique de l'activabilité ou non de l'option "resultsPubliclyVisible" lorsque le mot de passe est renseigné ou non lors de l'ajout de mot de passe.

  3. ajout d'un texte spécial pour l'option (il partageait le même libellé "result are visible") dans les fichiers "locale" pour les traductions, et modification du libellé dans le template tpl/part/poll_info.tpl

Amélioration possible :

  • Modifier la base SQL pour faire la vérification que l'on ne peut pas avoir simultanément les résultats cachés et avoir uniquement les votes protégés par mot de passe

  • Modifier le js de la création de sondage pour dynamiquement rendre activable ou non l'option (actuellement : on peut cocher l'option sans renseigner de mot de passe, génère erreur lors de la validation du formulaire du fait de la non présence de mot de passe, et si on désactive l'ajout d'un mot de passe, l'option n'est pas conservé)

-modifier le nom de la variable

avant : https://framapic.org/gallery#DhkD1fe6KSz4/71UOmdk6gs5N.png,uoOGUk9Z16Fi/utZ9hS5UYjfU.png,IvdJLY2585JG/tEAN85B5KIzP.png,Ljcx1SJ2zkl8/imeBTLsxdEdI.png,tXVetezHmqgg/rEucKtzrixXn.png,tLgOiXNlGaJ0/Y2qQ9cjPujUQ.png après : https://framapic.org/gallery#4qGBM8LCjY8j/hDxZBEyzqemC.png,Q5deRBkPlBpf/qcIua4EpUsug.png,BH4wwcpBnJS7/8LPKhD0Raz42.png (image disponible jusqu'au 18 janvier)

Contact : titouan.david@etu.utc.fr

Edited by Thomas

Merge request reports