[MDL-47395] improve visual cues for cli errors - Moodle Tracker

XML

Word

Printable

Affected Branches:
MOODLE_28_STABLE
Pull from Repository:
git://github.com/nebgor/moodle.git
Pull Master Branch:
MDL-47395
Pull Master Diff URL:
https://github.com/nebgor/moodle/compare/moodle:master...MDL-47395
Workaround:

Hide

read the cli text output in detail every time as a meticulous, often overloaded, moodle systems administrator.

Show
read the cli text output in detail every time as a meticulous, often overloaded, moodle systems administrator.
Testing Instructions:

Hide

Test: use cli tools. try to get an error and observe that a biggish unicode 'X' , in red on some systems, appears to indicate to admin that cli has output an error

example of cli test run using cli_error() call which puts in the visual cue:
php admin/cli/upgrade.php
❌ No upgrade needed for the installed version 2.8dev (Build: 20140918) (2014091800). Thanks for coming anyway!

Show
Test: use cli tools. try to get an error and observe that a biggish unicode 'X' , in red on some systems, appears to indicate to admin that cli has output an error example of cli test run using cli_error() call which puts in the visual cue: php admin/cli/upgrade.php ❌ No upgrade needed for the installed version 2.8dev (Build: 20140918) (2014091800). Thanks for coming anyway!

often cli errors are just text which administrators can easily miss. this aims to put a visual cue into cli_error.

note: there is no cli api call for cli_ok() or so...

ps: fwrite(STDERR, "\xF0\x9F\x91\x8E ". $text);

has been marked as being related by

MDL-50631 Display Moodle ASCII logo in CLI installer

Participants:: Aparup Banerjee, CiBoT, Damyon Wiese, Dan Poltawski, Sam Hemelryk

Component watchers:: David Woloszyn, Huong Nguyen, Jake Dallimore, Michael Hawkins, Stevani Andolo