Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

Cloture d'exercice, gestion des reliquats et traitement des jours fériés français. Fix #418 #806

Merged
merged 22 commits into from
Jul 24, 2019

Conversation

wouldsmina
Copy link
Member

@wouldsmina wouldsmina commented Mar 21, 2019

Nouveau code, nouvelle logique métier! La clôture d'exercice permet de reporter le solde/an sur le solde courant et le solde courant sur les reliquats (si activé). Le nouveau code fonctionne sur ce même principe, mais avec quelques particularités :

  • seul les employés activés sont traités
  • tous les employés actif doivent avoir déposé au moins un congés de chaque type durant l'exercice courant, sinon la clôture est annulé (c'est peut être un peu fort, a voir durant le test).
  • la date limite des reliquats n'est plus calculé automatiquement, il faut saisir l'année avant de valider le formulaire, le mois et la date sont paramétré dans la configuration globale, il est possible de le consulter en positionnant la souris sur le (?).
  • les reliquats ne sont plus inclus dans le solde.
  • les jours fériés français sont paramétrables automatiquement, pour l'année suivante, en cochant la case idoine. Cela permet, au passage, de corriger un bug d'affichage sur la gestion des jours fériés.

Comment tester :

Prérequis :

  • option report des reliquats activée
  • option jour et mois limite des reliquats paramétré
  • des utilisateurs avec des soldes variables
  • make update
  • copier le contenu de la table conges_appli et conges_solde_user pour comparaison ultérieur (et toute la bdd pour recommencer en cas de besoin ;) )
  1. Changement d'exercice :

Valider un changement d'exercice en paramétrant l'année du report des reliquats (le jour et le mois sont visibles en passant la souris sur le ?) à l'aide du datepicker. Cocher la case pour remplir automatiquement les jours fériés.

Résultat attendu (comparaison bdd):

  • champ conges_appli.date_limite_reliquats, l'année doit correspondre à la saisie dans le formulaire, le jour et le mois à la date limite dans les options
  • champ conges_appli.num_exercice incrémenté.
  • table conges_solde_user:
  • ancien su_solde reporté dans nouveau su_reliquat
  • su_nb_an = su_solde
  • les jours fériés français pour l'année suivante sont paramétrés
  1. dépôt et traitement d'une demande :

Avec la date limite des reliquats (dans conges_appli.date_limite_reliquats) dans le futur, déposer un congés en tant qu'employé, puis en tant que responsable, valider la demande.

Résultat attendu : le solde de l'employé reste le même, le reliquat et décrémenté du nombre de jour du congé validé.

Avec une date dans le passé (à modifier directement en bdd), effectuer la même manipulation.

Résultat attendu : le solde est décrémenté, le reliquat est à zéro.

Je n'ai, intentionnellement, pas implémenté la fermeture par groupe et par utilisateur. si le besoin existe, je le ferai.

@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@libertempo libertempo deleted a comment Mar 21, 2019
@prytoegrian
Copy link
Member

Cool ! C'est toujours ça en moins.

@wouldsmina
Copy link
Member Author

wouldsmina commented May 22, 2019

Cool ! C'est toujours ça en moins.

Ouaip!

Par contre, j'admets que le test ne sera pas une partie de plaisir 😒 . Désolé pour celui qui va s'y coller 😥

@wouldsmina
Copy link
Member Author

wouldsmina commented Jul 6, 2019

tous les employés actif doivent avoir déposé au moins un congés de chaque type durant l'exercice courant, sinon la clôture est annulé (c'est peut être un peu fort, a voir durant le test).

en relisant et testant ce PR je me rend compte que ce n'est pas le cas. Si l'utilisateur n'a pas déposé de congés, il n'est pas traité.
Cela m'arrange, je préfère assouplir cette règle pour le moment.

@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@libertempo libertempo deleted a comment Jul 6, 2019
@wouldsmina wouldsmina merged commit 7c9fff6 into develop Jul 24, 2019
@wouldsmina wouldsmina deleted the ws/ClotureExe branch July 24, 2019 20:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants