fixed #117

CHANGES.md

@@ -4,6 +4,7 @@
 
 * Added: `log()` convenience method the multibar to enable custom logging output on top of the progress bars during operation
 * Added: `gracefulExit` option (enabled by default) to stop the bars in case of `SIGINT` or `SIGTERM` - this restores most cursor settings before exiting
+* Bugfix: within a non-tty environment (e.g. CI/CD taskrunners) `multibar.create()` returns an undefined value in case `noTTYOutput` is not enabled #117
 
 ### 3.10.0 ###
 

examples/example-multibar.js

@@ -17,7 +17,8 @@ const multibar = new _progress.MultiBar({
     barCompleteChar: '\u2588',
     barIncompleteChar: '\u2591',
     clearOnComplete: true,
-    stopOnComplete: true
+    stopOnComplete: true,
+    noTTYOutput: true
 });
 
 console.log("Downloading files..\n");

examples/example-notty.php

@@ -1,7 +1,7 @@
 <?php
 
 // open 1-way read handle (stdout+stderr)
-$handle = popen('/usr/bin/node example.js 2>&1', 'r');
+$handle = popen('/usr/bin/node example-multibar.js 2>&1', 'r');
 
 // fetch content
 while (!feof($handle)) {

lib/multi-bar.js

@@ -42,17 +42,17 @@ module.exports = class MultiBar extends _EventEmitter{
 
     // add a new bar to the stack
     create(total, startValue, payload){
-        // progress updates are only visible in TTY mode!
-        if (this.options.noTTYOutput === false && this.terminal.isTTY() === false){
-            return;
-        }
-
         // create new bar element
         const bar = new _BarElement(this.options);
 
         // store bar
         this.bars.push(bar);
 
+        // progress updates are only visible in TTY mode!
+        if (this.options.noTTYOutput === false && this.terminal.isTTY() === false){
+            return bar;
+        }
+
         // multiprogress already active ?
         if (!this.isActive){
             // hide the cursor ?