From 50bbeb024567bf4e8b8f3179025a8ea928023bf5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ren=C3=A9e=20Kooi?= Date: Fri, 11 Oct 2019 16:42:40 +0200 Subject: [PATCH] core: provide default error message (#1880) We were already doing `if (file.error)` in places, now that is guaranteed to work even if the error message was empty or the error object is custom and doesn't have a `.message` property. --- packages/@uppy/core/src/index.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/@uppy/core/src/index.js b/packages/@uppy/core/src/index.js index 03c961760..a52ae0040 100644 --- a/packages/@uppy/core/src/index.js +++ b/packages/@uppy/core/src/index.js @@ -783,12 +783,12 @@ class Uppy { */ _addListeners () { this.on('error', (error) => { - this.setState({ error: error.message }) + this.setState({ error: error.message || 'Unknown error' }) }) this.on('upload-error', (file, error, response) => { this.setFileState(file.id, { - error: error.message, + error: error.message || 'Unknown error', response })