Closed (fixed)
Project:
Backup and Migrate
Version:
8.x-4.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Feb 2018 at 10:13 UTC
Updated:
8 Apr 2019 at 14:41 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
philyComment #3
bryrock commentedI will add a patch for this momentarily.
Comment #4
bryrock commentedHere's a patch to get the timestamp from local time instead of GMT.
Comment #5
bryrock commentedComment #7
bryrock commentedThe patch passes when tested with the correct test (the default test for this module is set to unsupported PHP version for Drupal 8).
The module is using php function "gmdate" which always results in greenwich time. According to php manual, "Identical to the date() function except that the time returned is Greenwich Mean Time (GMT)."
The only change needed to get the timestamp to run on local time is:
- $name .= '-' . gmdate($this->confGet('timestamp_format'));
+ $name .= '-' . date($this->confGet('timestamp_format'));
In other words, change gmdate() to date(). That's it.
(It also works)
Comment #8
alex andrascu commentedComment #9
damienmckennaComment #10
damienmckennaCommitted. Thanks.
Comment #13
philyHaving been away for a while, I just want to thank you for fixing: it works!