Project

General

Profile

Bug #4171

Dans la liste des absences de groupe changer la colonne 'Tous les services' en 'Services'

Added by Mikaël Ates almost 10 years ago. Updated over 9 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
Start date:
20 December 2013
Due date:
% Done:

50%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Il faudrait ainsi lister dans cette colonne la liste des services auxquels s'appliquent l'absence, en mettant Tous dans ce cas, plutôt qu'une coche si c'est tous les services.


Files

Associated revisions

Revision 9090c9dd (diff)
Added by Serghei Mihai over 9 years ago

agenda: display the services list or "Tous" for group holidays instead of check icon

Closes #4171

Revision 0d4bf0b4 (diff)
Added by Serghei Mihai over 9 years ago

personnes: fixing the group holidays with empty services list

Closes #4171

History

#1

Updated by Mikaël Ates over 9 years ago

  • Target version set to 2.0
#2

Updated by Mikaël Ates over 9 years ago

  • Target version changed from 2.0 to 1.2 Autres

A l'APS, les absences de groupe sont toujours pour tous les services. Enlever le
choix du service. Corriger les congés de groupe existant qui ne seraient
pas sur tous les services. Dans l'agenda pas besoin de gérer le grisé et
les pastilles.

Cependant, il pourrait y avaoir d'autres cas où les absences de groupe seraitent par service.

Peut-on qu'on peut gérer les deux cas et mettre une option dans les settings.
Pour l'options 'Absences de groupes pour tous les services', on masque le choix du service, l'affichage du service sur la page des absences et dans l'agenda et on positionne en automatiquement l'absence dans tous les services.

#3

Updated by Serghei Mihai over 9 years ago

  • Assignee set to Serghei Mihai
  • Patch proposed set to No
#4

Updated by Serghei Mihai over 9 years ago

Mikaël Ates a écrit :

A l'APS, les absences de groupe sont toujours pour tous les services. Enlever le
choix du service. Corriger les congés de groupe existant qui ne seraient
pas sur tous les services.

Actuellement un congé est consideré pour tous les services si aucun ou tous les services sont séléctionnés.
Je penche pour la deuxième option. Je ferai un script pour ça.

Dans l'agenda pas besoin de gérer le grisé et
les pastilles.

Cependant, il pourrait y avaoir d'autres cas où les absences de groupe seraitent par service.

Dans ce cas autant de garder le fonctionnement actuel, en cochant tous les services par défaut lors de la création d'une absence?

Peut-on qu'on peut gérer les deux cas et mettre une option dans les settings.
Pour l'options 'Absences de groupes pour tous les services', on masque le choix du service, l'affichage du service sur la page des absences et dans l'agenda et on positionne en automatiquement l'absence dans tous les services.

#6

Updated by Serghei Mihai over 9 years ago

Script de management pour ajouter tous les services aux congés sans service

#7

Updated by Benjamin Dauvergne over 9 years ago

Relu, rien qui me choque.

#8

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 50 to 100

Appliqué par commit calebasse|commit:9090c9dd99bdf3941fe08ae980c4da58317041bf.

#9

Updated by Serghei Mihai over 9 years ago

Appliqué par commit calebasse|commit:0d4bf0b4a97f02cee0a83f67548bc4408a20e524.

#10

Updated by Mikaël Ates over 9 years ago

  • Status changed from Résolu (à déployer) to En cours
  • % Done changed from 100 to 50

Il faut soit ajouter la commande à la procédure de mise à jour de la prod soit ne pas faire une commande mais une migration de données south.

Ce script sert à assurer une cohérence dans l'affichage du message 'Tous' avec le fait que tous les services sont bien enregistrés pour l'absence de groupe. Cependant pour aller plus loin dans la cohérence il faudrait que lorsque l'on ajoute un congés de groupe sans aucun service, soit que l'on enregistre automatiquement tous les services. soit que l'on affiche un message d'erreur 'au moins un service est requis'.

#11

Updated by Serghei Mihai over 9 years ago

J'ai fait tourner le script de vérification des congés sur une copie de la base de prod et il n'avait aucun congés de groupe qui n'était associé à aucun service.
Je vais include le lancement du script dans la procédure de màj.

Quand on ajoute une absence de groupe sans service de coché, il y a déjà un message d'erreur dans le formulaire disant que le champ est obligatoire.

#12

Updated by Jérôme Schneider over 9 years ago

  • Status changed from En cours to Information nécessaire
#13

Updated by Mikaël Ates over 9 years ago

  • Status changed from Information nécessaire to En cours

Je viens de tester l'ajout de congés, les logs console indique la création mais ils n'apparaissent pas dans la liste des congés de groupe.

#14

Updated by Mikaël Ates over 9 years ago

s/congés/absences

#15

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Information nécessaire

Je viens de re-vérifier, en mettant un commentaire pour pouvoir retrouver l'absence dans la liste et cela fonctionne.

#16

Updated by Jérôme Schneider over 9 years ago

  • Status changed from Information nécessaire to En cours

Je viens de tester et j'ai effectivement le même soucis que Mikaël.

#17

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Information nécessaire

En fait l'ajout fonctionne correctement, mais le listing des congés n'affiche que les congés à venir, donc si on rajoute un congé dans le passé, il ne sera pas visible.

#18

Updated by Mikaël Ates over 9 years ago

  • Status changed from Information nécessaire to Fermé

Ok

Also available in: Atom PDF