Moodle

core_course_external_testcase::test_create_courses => start date timestamp is out of range

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.3.1, 2.4
  • Fix Version/s: 2.3.2
  • Component/s: Web Services
  • Labels:

Description

core_course_external_testcase::test_create_courses => start date timestamp is out of range

Issue Links

blocks

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34894 core_course_external_testcase::test_duplicate_course fails on Windows

  • Critical - Crashes server, loss of data, severe memory leak
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.
has been marked as being related by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34908 External functions: timestamp should be set to a new PARAM_TIMESTAMP?

  • Minor - Minor loss of function where workaround is possible
  • Open - The issue is open and ready for the assignee to start work on it.
is duplicated by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34282 core_course_external_testcase::test_create_courses failing on oracle

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jérôme Mouneyrac added a comment -

Submitting for integration without peer-review as it's simple and as it breaks PHPUnit test on 32bits system.

Show
Jérôme Mouneyrac added a comment - Submitting for integration without peer-review as it's simple and as it breaks PHPUnit test on 32bits system.
Hide
Permalink
Tim Hunt added a comment -

Since I was the one who originally reported the problem, and can reproduce it, I have tested this fix, and can confirm that it works. So, integrators, you may wish to move this directly to the Tested state.

I have also peer-reviewed the change, and I give it my +1.

Show
Tim Hunt added a comment - Since I was the one who originally reported the problem, and can reproduce it, I have tested this fix, and can confirm that it works. So, integrators, you may wish to move this directly to the Tested state. I have also peer-reviewed the change, and I give it my +1.
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Integrated, thanks (23 & master).

Show
Eloy Lafuente (stronk7) added a comment - Integrated, thanks (23 & master).
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Passing based on Tim's tests, thanks!

Show
Eloy Lafuente (stronk7) added a comment - Passing based on Tim's tests, thanks!
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

YEAR!*

CAF*, TOT!*

  • Your effort amazingly resulted. (unbelievable :-P)
  • Closing as fixed.
  • Tons of thanks.
Show
Eloy Lafuente (stronk7) added a comment - YEAR!* CAF*, TOT!* Your effort amazingly resulted. (unbelievable :-P) Closing as fixed. Tons of thanks.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: