Skip to content

Commit

Permalink
Fix wrong approve time in expense report
Browse files Browse the repository at this point in the history
The approve time showed was 8 hours more than the real time.
Checking database turned out that it had the wrong data(+8 hours).

After checking every timezone related settings were correct,
I found that it do twice idate() in setApproved().
Since we have idate() in $sql, we don't need the idate()
at the line 1249.
  • Loading branch information
ggg44473 committed Mar 29, 2018
1 parent 125085a commit 3dd9d47
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion htdocs/expensereport/class/expensereport.class.php
Expand Up @@ -1246,7 +1246,7 @@ function setApproved($fuser, $notrigger=0)
$error = 0;

// date approval
$this->date_approve = $this->db->idate($now);
$this->date_approve = $now;
if ($this->fk_statut != 5)
{
$this->db->begin();
Expand Down

0 comments on commit 3dd9d47

Please sign in to comment.