Moodle
  1. Moodle
  2. MDL-34941

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

        Details

        • Type: Bug Bug
        • Status: 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

            Gliffy Diagrams

              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
              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
              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

                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

                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    1 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: