From 1f7e4c886b4f85579a53f96f3039ec38097d673d Mon Sep 17 00:00:00 2001
From: Alexis Svinartchouk <alexis@resin.io>
Date: Mon, 22 Apr 2019 18:25:26 +0200
Subject: [PATCH] Don't show a dialog when the write fails.

There is already an error modal and the error detail will be shown in the console.

Changelog-entry: Don't show a dialog when the write fails.
---
 lib/gui/app/pages/main/controllers/flash.js | 8 +++++---
 lib/shared/messages.js                      | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/lib/gui/app/pages/main/controllers/flash.js b/lib/gui/app/pages/main/controllers/flash.js
index 64003f0c..d10bb065 100644
--- a/lib/gui/app/pages/main/controllers/flash.js
+++ b/lib/gui/app/pages/main/controllers/flash.js
@@ -22,7 +22,7 @@ const flashState = require('../../../models/flash-state')
 const driveScanner = require('../../../modules/drive-scanner')
 const progressStatus = require('../../../modules/progress-status')
 const notification = require('../../../os/notification')
-const exceptionReporter = require('../../../modules/exception-reporter')
+const analytics = require('../../../modules/analytics')
 const imageWriter = require('../../../modules/image-writer')
 const path = require('path')
 const store = require('../../../models/store')
@@ -145,12 +145,13 @@ module.exports = function (
     driveScanner.stop()
 
     const iconPath = '../../../assets/icon.png'
+    const basename = path.basename(image.path)
     try {
       await imageWriter.flash(image.path, drives)
       if (!flashState.wasLastFlashCancelled()) {
         const flashResults = flashState.getFlashResults()
         notification.send('Flash complete!', {
-          body: messages.info.flashComplete(path.basename(image.path), drives, flashResults.results.devices),
+          body: messages.info.flashComplete(basename, drives, flashResults.results.devices),
           icon: iconPath
         })
         $state.go('success')
@@ -181,7 +182,8 @@ module.exports = function (
         FlashErrorModalService.show(messages.error.childWriterDied())
       } else {
         FlashErrorModalService.show(messages.error.genericFlashError())
-        exceptionReporter.report(error)
+        error.image = basename
+        analytics.logException(error)
       }
     } finally {
       availableDrives.setDrives([])
diff --git a/lib/shared/messages.js b/lib/shared/messages.js
index 2112777d..6f0733b3 100644
--- a/lib/shared/messages.js
+++ b/lib/shared/messages.js
@@ -171,7 +171,7 @@ module.exports = {
     },
 
     genericFlashError: () => {
-      return 'Oops, seems something went wrong.'
+      return 'Something went wrong. If it is a compressed image, please check that the archive is not corrupted.'
     },
 
     validation: () => {