Details
-
Bug
-
Status: Waiting for peer review
-
Minor
-
Resolution: Unresolved
-
3.9.9, 3.11.2, 3.11.11, 4.0, 4.0.5, 4.1
-
None
-
PHP 7.4 32-Bit version
-
MOODLE_311_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE, MOODLE_401_STABLE
-
MDL-72359-master-change-max-time
-
Description
defines a constant that is used in the user.php analysable here:
https://github.com/moodle/moodle/blob/036800d99debfe61dde45f61292bc5ad44ab7cb0/analytics/classes/user.php#L190-L192
This again is used in the time splitting periodic class here
https://github.com/moodle/moodle/blob/036800d99debfe61dde45f61292bc5ad44ab7cb0/analytics/classes/local/time_splitting/periodic.php#L82-L84
DateTimeImmutable needs an int timestamp to work. 32 Bit PHP can't handle the integer value 9999999999, so it converts it to float. This leads to the warning:
PHP Warning: DateTimeImmutable::setTimestamp() expects parameter 1 to be integer, float given in /moodle/application/htdocs/moodle/moodle_stacks/analytics/classes/local/time_splitting/periodic.php on line 83
Possible solution: Smaller value for timestamp?
Attachments
Issue Links
- has a non-specific relationship to
-
MDL-74905 Decide Moodle 4.2 requirements and push them to environment.xml (due date: 2022-12-26)
-
- Closed
-