Moodle

improvements to display of results from environment checks

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.6.6, 1.7.3, 1.8.2, 1.9
  • Fix Version/s: 1.7.2, 1.8.2, 1.9
  • Component/s: Administration
  • Labels:
    None
  • Affected Branches:
    MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

Description

Improvements to display of results from environment checks.

Activity

Hide
Jamie Pratt added a comment - - edited

Attached a patch to :

  • seperate 'custom checks' into a seperate table.
  • sort environment checks so that errors appear at the top of both tables, then warnings, then OK messages.
  • put feedback message inside the table cell of the environment check that created them, rather than in a seperate list at the bottom of the page.
  • add css for colour and added green 'ok' colour to html.
Show
Jamie Pratt added a comment - - edited Attached a patch to :
  • seperate 'custom checks' into a seperate table.
  • sort environment checks so that errors appear at the top of both tables, then warnings, then OK messages.
  • put feedback message inside the table cell of the environment check that created them, rather than in a seperate list at the bottom of the page.
  • add css for colour and added green 'ok' colour to html.
Hide
Jamie Pratt added a comment - - edited

Ignore the last screen shot file ScreenShot001.bmp. Attached file ScreenShot002.jpg is a shot of the new layout. The second table is only shown if the custom check function returns a non null value.

Show
Jamie Pratt added a comment - - edited Ignore the last screen shot file ScreenShot001.bmp. Attached file ScreenShot002.jpg is a shot of the new layout. The second table is only shown if the custom check function returns a non null value.
Hide
Jamie Pratt added a comment -

I've separated results from custom checks in environment xml into a seperate table assuming that custom environment check will check things other than server setup.

Show
Jamie Pratt added a comment - I've separated results from custom checks in environment xml into a seperate table assuming that custom environment check will check things other than server setup.
Hide
Martin Dougiamas added a comment -

Yep looks good to me! I'm sure we can keep refining it but this is a good step

Show
Martin Dougiamas added a comment - Yep looks good to me! I'm sure we can keep refining it but this is a good step
Hide
Jamie Pratt added a comment -

Added links to docs as requested.

Show
Jamie Pratt added a comment - Added links to docs as requested.
Hide
Jamie Pratt added a comment -

Applied this to 1.7, 1.8 and 1.9

Show
Jamie Pratt added a comment - Applied this to 1.7, 1.8 and 1.9

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: