Do not wrap errors from fetch & remove dead code 20/head
authorRaphael von der Grün <raphinesse@gmail.com>
Thu, 5 Jul 2018 10:48:42 +0000 (12:48 +0200)
committerRaphael von der Grün <raphinesse@gmail.com>
Thu, 5 Jul 2018 10:48:42 +0000 (12:48 +0200)
The errors from fetch were strongly emphasized (all red) for no obvious
reason. Moreover, everything after the first error event emission never
ran since the error event seems to cause an error to be thrown.

index.js

index 8f50d0a..0e3b565 100644 (file)
--- a/index.js
+++ b/index.js
@@ -137,16 +137,9 @@ function cordovaCreate (dest, opts = {}) {
             if (opts.template && isRemoteUri(opts.url)) {
                 var target = opts.url;
                 events.emit('verbose', 'Using cordova-fetch for ' + target);
-                return fetch(target, getSelfDestructingTempDir(), {})
-                    .catch(function (err) {
-                        events.emit('error', '\x1B[1m \x1B[31m Error from Cordova Fetch: ' + err.message);
-                        events.emit('error', 'The template you are trying to use is invalid.' +
-                        ' Make sure you follow the template guide found here https://cordova.apache.org/docs/en/latest/guide/cli/template.html.' +
-                        ' Templates now require a package.json.');
-                        throw err;
-                    });
-            // If assets are not online, resolve as a relative path on local computer
+                return fetch(target, getSelfDestructingTempDir(), {});
             } else {
+                // If assets are not online, resolve as a relative path on local computer
                 return path.resolve(opts.url);
             }
         })