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

Compte épargne : répare à nouveau l'erreur de début de cycle #1010

Merged
merged 3 commits into from
Sep 29, 2023

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Sep 27, 2023

Correction de la #990

Le timelog de début de cycle n'était toujours pas pris en compte

Modifications apportées :

  • ajouté un refresh sur les TimeLogs quand on refresh un Membership
  • corrigé le format de clone

@raphodn raphodn self-assigned this Sep 27, 2023
@raphodn raphodn linked an issue Sep 27, 2023 that may be closed by this pull request
@raphodn raphodn force-pushed the raphodn/time-log-saving-end-fix-2 branch from 53b412e to 418f570 Compare September 28, 2023 16:04
@@ -96,7 +96,9 @@ class TimeLog
*/
public function setCreatedAtValue()
{
$this->createdAt = new \DateTime();
if (!$this->createdAt) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

la ligne la plus importante de la PR

@raphodn raphodn merged commit 8c82c32 into master Sep 29, 2023
1 check failed
@raphodn raphodn deleted the raphodn/time-log-saving-end-fix-2 branch September 29, 2023 13:05
OursDesCavernes pushed a commit to Les400Coop/gestion-compte that referenced this pull request Jan 20, 2024
…grenoble#1010)

* Doctrine: refresh timelogs when refreshing member

* Fix TimeLog setCreatedAt prePersist

* Add date & time in other time logs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CET : Calcul de fin de cycle erroné
1 participant