Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-52029

MDL-43839 regression: installer not styled if php shows errors

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      You will need windows to test this.

      1. Prepare moodle files for install and remove or move your config.php
      2. Setup PHP: error_reportng = E_ALL, display_errors = true. in your apache php.ini
      3. Browse to your moodle directory in www
      4. Use the browser inspector and locate install/css.php.
      5. It SHOULD NOT start with php warning.
      Show
      You will need windows to test this. Prepare moodle files for install and remove or move your config.php Setup PHP: error_reportng = E_ALL, display_errors = true. in your apache php.ini Browse to your moodle directory in www Use the browser inspector and locate install/css.php. It SHOULD NOT start with php warning.
    • Affected Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      moodle/install/css.php contains error code, which tries to open unexistent file. If display_errors and error_reporting is on, error info breaks css. In other conditions error is not seen.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              vadimon Vadim Dvorovenko
              Reporter:
              vadimon Vadim Dvorovenko
              Peer reviewer:
              Simey Lameze
              Integrator:
              David Monllaó
              Tester:
              Rajesh Taneja
              Participants:
              Component watchers:
              Matteo Scaramuccia, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                21/Dec/15

                  Time Tracking

                  Estimated:
                  Original Estimate - 1 hour
                  1h
                  Remaining:
                  0m
                  Logged:
                  Time Not Required
                  Not Specified