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

Fix the legacy coding style in the admin/settings/server.php file

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.10
    • 3.9.1
    • Administration
    • MOODLE_39_STABLE
    • MOODLE_310_STABLE
    • MDL-69632-master-adminserversettings
    • Hide

      Testing requirements

      • Ability to run CLI scripts
      • Ability to confirm that two files are identical

      Anti-regression test

      1. Put the attached dumpadminroot.php to the root of the recent unpatched Moodle.
      2. Run

      php dumpadminroot.php > before.txt
      

      to save the structure of the admin tree before the patch.
      3. Change over to the patched version of Moodle with this issue present and have the same dumpadminroot.php file there, too.
      4. Run

      php dumpadminroot.php > after.txt
      

      to save the structure of the admin tree before the patch.
      5. TEST Make sure (using a utility such as diff or similar that the before.txt and after.txt files are identical.

      Show
      Testing requirements Ability to run CLI scripts Ability to confirm that two files are identical Anti-regression test 1. Put the attached dumpadminroot.php to the root of the recent unpatched Moodle. 2. Run php dumpadminroot.php > before.txt to save the structure of the admin tree before the patch. 3. Change over to the patched version of Moodle with this issue present and have the same dumpadminroot.php file there, too. 4. Run php dumpadminroot.php > after.txt to save the structure of the admin tree before the patch. 5. TEST Make sure (using a utility such as diff or similar that the before.txt and after.txt files are identical.

      While working on MDL-67748 I started with fixing the ancient coding style in the file admin/settings/server.php so that I could start performing more changes there in a clean way.

      It turned out it would be easier to get this coding style fix integrated first so that we can focus on the actual changes in the follow up commits.

      The patch should only fix the coding style, with no other functional or visual change to the settings.

            mudrd8mz David Mudrák (@mudrd8mz)
            mudrd8mz David Mudrák (@mudrd8mz)
            Luca Bösch Luca Bösch
            Sara Arjona (@sarjona) Sara Arjona (@sarjona)
            Janelle Barcega Janelle Barcega
            Votes:
            1 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 3 hours, 30 minutes
                3h 30m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.