feat(nsis): Pass --update flag to uninstaller when auto updating an application

develar · develar · commit 505a63d742f9 · 2017-01-29T17:18:33.000+01:00
Close #1162
diff --git a/packages/electron-builder-util/package.json b/packages/electron-builder-util/package.json
@@ -19,7 +19,7 @@
     "debug": "2.6.0",
     "node-emoji": "^1.5.1",
     "electron-builder-http": "~0.0.0-semantic-release",
-    "source-map-support": "^0.4.10",
+    "source-map-support": "^0.4.11",
     "7zip-bin": "^2.0.4"
   },
   "typings": "./out/electron-builder-util.d.ts"
diff --git a/packages/electron-builder/templates/nsis/installSection.nsh b/packages/electron-builder/templates/nsis/installSection.nsh
@@ -30,15 +30,20 @@
       ${endif}
 
       ClearErrors
-      ${GetParameters} $R4
-      ${GetOptions} $R4 "--delete-app-data" $R5
+      ${GetParameters} $R0
+      ${GetOptions} $R0 "--update" $R2
       ${ifNot} ${Errors}
-        StrCpy $1 "--delete-app-data"
-      ${else}
-        StrCpy $1 ""
+        StrCpy $0 "$0 --update"
+      ${endif}
+
+      ClearErrors
+      ${GetParameters} $R0
+      ${GetOptions} $R0 "--delete-app-data" $R2
+      ${ifNot} ${Errors}
+        StrCpy $0 "$0 --delete-app-data"
       ${endif}
 
-      ExecWait '"$PLUGINSDIR\old-uninstaller.exe" /S /KEEP_APP_DATA $0 $1 _?=$R1'
+      ExecWait '"$PLUGINSDIR\old-uninstaller.exe" /S /KEEP_APP_DATA $0 _?=$R1'
     ${endif}
   ${endif}
 !macroend
diff --git a/packages/electron-builder/templates/nsis/multiUserUi.nsh b/packages/electron-builder/templates/nsis/multiUserUi.nsh
@@ -198,7 +198,7 @@ Var RadioButtonLabel1
 					StrCpy $7 "There is a per-user installation. ($perUserInstallationFolder)$\r$\nWill uninstall."
 				!endif
 			${else}
-				StrCpy $7 "Fresh install for current user only"
+				StrCpy $7 "Fresh install for current user only."
 			${endif}
 			SendMessage $0 ${BCM_SETSHIELD} 0 0 ; hide SHIELD
 		${else} ; all users
diff --git a/packages/electron-builder/templates/nsis/oneClick.nsh b/packages/electron-builder/templates/nsis/oneClick.nsh
@@ -7,9 +7,9 @@
       !else
         ${GetParameters} $R0
         ${GetOptions} $R0 "--update" $R1
-        ${IfNot} ${Errors}
+        ${ifNot} ${Errors}
           ExecShell "" "$SMPROGRAMS\${PRODUCT_FILENAME}.lnk" "--updated"
-        ${Else}
+        ${else}
           ExecShell "" "$SMPROGRAMS\${PRODUCT_FILENAME}.lnk"
         ${endif}
       !endif
diff --git a/packages/electron-updater/package.json b/packages/electron-updater/package.json
@@ -16,7 +16,7 @@
     "fs-extra-p": "^3.1.0",
     "js-yaml": "^3.7.0",
     "semver": "^5.3.0",
-    "source-map-support": "^0.4.10",
+    "source-map-support": "^0.4.11",
     "electron-builder-http": "0.0.0-semantic-release"
   },
   "typings": "./out/electron-updater.d.ts"
diff --git a/packages/electron-updater/src/AppUpdater.ts b/packages/electron-updater/src/AppUpdater.ts
@@ -151,15 +151,16 @@ export abstract class AppUpdater extends EventEmitter {
       throw new Error(`Latest version (from update server) is not valid semver version: "${latestVersion}`)
     }
 
-    const currentVersion = parseVersion(this.app.getVersion())
+    const currentVersionString = this.app.getVersion()
+    const currentVersion = parseVersion(currentVersionString)
     if (currentVersion == null) {
       throw new Error(`App version is not valid semver version: "${currentVersion}`)
     }
 
     if (!isVersionGreaterThan(latestVersion, currentVersion)) {
       this.updateAvailable = false
       if (this.logger != null) {
-        this.logger.info(`Update for version ${versionInfo.version} is not available`)
+        this.logger.info(`Update for version ${currentVersionString} is not available (latest version: ${versionInfo.version})`)
       }
       this.emit("update-not-available")
       return {
diff --git a/packages/electron-updater/src/GenericProvider.ts b/packages/electron-updater/src/GenericProvider.ts
@@ -22,13 +22,11 @@ export class GenericProvider extends Provider<UpdateInfo> {
         hostname: this.baseUrl.hostname,
         path: `${pathname}${this.baseUrl.search || ""}`,
         protocol: this.baseUrl.protocol,
+        headers: Object.assign({"Cache-Control": "no-cache, no-store, must-revalidate"}, this.requestHeaders)
       }
       if (this.baseUrl.port != null) {
         options.port = parseInt(this.baseUrl.port, 10)
       }
-      if (this.requestHeaders != null) {
-        options.headers = this.requestHeaders
-      }
       result = await request<UpdateInfo>(options)
     }
     catch (e) {
diff --git a/yarn.lock b/yarn.lock
@@ -1483,8 +1483,8 @@ istanbul-lib-source-maps@^1.1.0:
     source-map "^0.5.3"
 
 istanbul-reports@^1.0.0:
-  version "1.0.0"
-  resolved "https://registry.yarnpkg.com/istanbul-reports/-/istanbul-reports-1.0.0.tgz#24b4eb2b1d29d50f103b369bd422f6e640aa0777"
+  version "1.0.1"
+  resolved "https://registry.yarnpkg.com/istanbul-reports/-/istanbul-reports-1.0.1.tgz#9a17176bc4a6cbebdae52b2f15961d52fa623fbc"
   dependencies:
     handlebars "^4.0.3"
 

Original file line number	Diff line number	Diff line change
`@@ -151,15 +151,16 @@ export abstract class AppUpdater extends EventEmitter {`
`151`	`151`	throw new Error(`Latest version (from update server) is not valid semver version: "${latestVersion}`)
`152`	`152`	`}`
`153`	`153`
`154`		`- const currentVersion = parseVersion(this.app.getVersion())`
	`154`	`+ const currentVersionString = this.app.getVersion()`
	`155`	`+ const currentVersion = parseVersion(currentVersionString)`
`155`	`156`	`if (currentVersion == null) {`
`156`	`157`	throw new Error(`App version is not valid semver version: "${currentVersion}`)
`157`	`158`	`}`
`158`	`159`
`159`	`160`	`if (!isVersionGreaterThan(latestVersion, currentVersion)) {`
`160`	`161`	`this.updateAvailable = false`
`161`	`162`	`if (this.logger != null) {`
`162`		- this.logger.info(`Update for version ${versionInfo.version} is not available`)
	`163`	+ this.logger.info(`Update for version ${currentVersionString} is not available (latest version: ${versionInfo.version})`)
`163`	`164`	`}`
`164`	`165`	`this.emit("update-not-available")`
`165`	`166`	`return {`
Original file line number	Diff line number	Diff line change
`@@ -22,13 +22,11 @@ export class GenericProvider extends Provider<UpdateInfo> {`
`22`	`22`	`hostname: this.baseUrl.hostname,`
`23`	`23`	path: `${pathname}${this.baseUrl.search \|\| ""}`,
`24`	`24`	`protocol: this.baseUrl.protocol,`
	`25`	`+ headers: Object.assign({"Cache-Control": "no-cache, no-store, must-revalidate"}, this.requestHeaders)`
`25`	`26`	`}`
`26`	`27`	`if (this.baseUrl.port != null) {`
`27`	`28`	`options.port = parseInt(this.baseUrl.port, 10)`
`28`	`29`	`}`
`29`		`- if (this.requestHeaders != null) {`
`30`		`- options.headers = this.requestHeaders`
`31`		`- }`
`32`	`30`	`result = await request<UpdateInfo>(options)`
`33`	`31`	`}`
`34`	`32`	`catch (e) {`