Skip to content

Fixed Reschedule function

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

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

lorsque l'on clique sur le bouton pour copier les horaires du premier jour vers les autres jours, les horaires copiés sont seulement ceux pour lequelles il existe un champ disponible

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

  • Ajout de champ "horaire" pour les dates pour lesquelles il en manque avant de copier les horaires

Changement technique concernant l'issue #203 (closed) :

  • dans le javascript js/app/studs.js :
    • ajout d'une fonction addHour(last_hour,add_button) avec
      • last_hour est un selecteur jquery sur le champ horaire à copier et à adapter (de n à n+1)
      • add_button est un selecteur jquery sur le bouton d'ajout d'horaire du jour (afin de le désactiver si la limite est atteinte) fonction inspirée de l'existant $(document).on('click', '.add-an-hour', function () {...})
    • ajout de l'appel fonction addHour(last_hour,add_button) dans $(document).on('click', '.add-an-hour', function () {...}) et remplace le code actuel
    • ajout de l'appel fonction addHour(last_hour,add_button) dans $('#copyhours').on('click', function () {...}) avec une boucle sur les jours pour avoir au moins autant d'horaire que le premier jour

Evolution possible :

  • Supprimer les champs en trop pour avoir exactement autant de champs

Visuel de la Solution :

  • état initial : etat initial
  • avant la correction : avant la correction
  • après la correction : après la correction

(désolé pour les nombreux commit, je n'ai pas su n'envoyé qu'une parti de mes modifications, seul celui nommé "correction issue 203" (ou "update date_poll.js") est à prendre en compte)

Edited by Thomas

Merge request reports