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

improve visual cues for cli errors

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Reopened
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.8
    • Fix Version/s: None
    • Component/s: Administration
    • Labels:
    • 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!
    • 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.
    • Affected Branches:
      MOODLE_28_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

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

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              nebgor Aparup Banerjee
              Peer reviewer:
              Damyon Wiese
              Integrator:
              Sam Hemelryk
              Tester:
              Dan Poltawski
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated: