[MDL-70583] Implement a renderer for progress_bar in cli output - Moodle Tracker

XML

Word

Printable

Details

Type: Improvement
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.11
Fix Version/s: 3.11
Component/s: Administration
Labels:
- CatalystIT
- ci
- cli
- partner
- release_notes

Affected Branches:
MOODLE_311_STABLE
Fixed Branches:
MOODLE_311_STABLE
Pull from Repository:
https://github.com/brendanheywood/moodle
Pull 3.11 Branch:
~~MDL-70583~~-cli-progress-MOODLE_311_STABLE
Pull 3.11 Diff URL:
https://github.com/moodle/moodle/compare/MOODLE_311_STABLE...brendanheywood:MDL-70583-cli-progress-MOODLE_311_STABLE
Pull Main Branch:
~~MDL-70583~~-cli-progress
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/master...brendanheywood:MDL-70583-cli-progress
Testing Instructions:

Hide

1) Run this test cli script:

php ~~MDL-70536~~-cli-progress.php

2) Confirm progress bars are blue while progressing and green when done

3) Confirm the estimate remaining is correct and counts down

4) Repeat but simulating a non tty: (or test it on windows)

php ~~MDL-70536~~-cli-progress.php | cat -v

5) Confirm you still get a simpler but dynamic progress bar with no ascii codes messing up the console

Show
1) Run this test cli script: php MDL-70536 -cli-progress.php 2) Confirm progress bars are blue while progressing and green when done 3) Confirm the estimate remaining is correct and counts down 4) Repeat but simulating a non tty: (or test it on windows) php MDL-70536 -cli-progress.php | cat -v 5) Confirm you still get a simpler but dynamic progress bar with no ascii codes messing up the console

Description

Should be the exact same as web output:

$progressbar = new progress_bar();

$progressbar->create();

And use ASCII cursors to draw and update a progress bar like:

Attachments

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

image-2021-01-09-14-52-19-213.png
11 kB
09/Jan/21 11:52 AM
MDL-70536-cli-progress.php
2 kB
27/Jan/21 9:52 PM
MDL-70583.jpg
76 kB
03/Feb/21 5:06 PM

Issue Links

Testing discovered

MDL-70709 cli_ansi_format() should detect absence of a tty

Open

will help resolve

MDL-70536 Create a CLI script to reset user dashboards

Closed

Activity

People

Assignee:: Brendan Heywood

Reporter:: Brendan Heywood

Integrator:: Andrew Lyons

Tester:: Anna Carissa Sadia

Participants:: Andrew Lyons, Anna Carissa Sadia, Brendan Heywood, CiBoT, Jordan Tomkinson, Peter Burnett

Votes:: 2 Vote for this issue

Watchers:: 7 Start watching this issue

Dates

Created:: 07/Jan/21 12:58 PM

Updated:: 07/May/21 6:02 PM

Resolved:: 04/Feb/21 10:20 PM

Fix Release Date:: 17/May/21

Time Tracking

Estimated:

0m

Remaining:

0m

Logged:

6h 10m