From ac2dffd140c106a46681162dd0464788312ff621 Mon Sep 17 00:00:00 2001
From: Jason Tsai <jason@pews.dev>
Date: Thu, 31 Aug 2023 20:17:47 +0800
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: Amr Bashir <amr.bashir2015@gmail.com>
---
 core/tauri-utils/src/config.rs                |  6 +++++-
 tooling/bundler/src/bundle/settings.rs        |  4 ++--
 tooling/bundler/src/bundle/windows/nsis.rs    | 19 +++++++++----------
 .../bundle/windows/templates/installer.nsi    |  8 ++------
 4 files changed, 18 insertions(+), 19 deletions(-)

diff --git a/core/tauri-utils/src/config.rs b/core/tauri-utils/src/config.rs
index 8fc7691467f8..dbdb7133422e 100644
--- a/core/tauri-utils/src/config.rs
+++ b/core/tauri-utils/src/config.rs
@@ -439,6 +439,8 @@ pub struct WixConfig {
 }
 
 /// Compression algorithms used in the NSIS installer.
+///
+/// See <https://nsis.sourceforge.io/Reference/SetCompressor>
 #[derive(Debug, PartialEq, Eq, Clone, Copy, Serialize, Deserialize)]
 #[cfg_attr(feature = "schema", derive(JsonSchema))]
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
@@ -494,7 +496,9 @@ pub struct NsisConfig {
   #[serde(default, alias = "display-language-selector")]
   pub display_language_selector: bool,
   /// Set the compression algorithm used to compress files in the installer.
-  pub set_compressor: Option<NsisCompressor>,
+  ///
+  /// See <https://nsis.sourceforge.io/Reference/SetCompressor>
+  pub compression: Option<NsisCompressor>,
 }
 
 /// Install Modes for the NSIS installer.
diff --git a/tooling/bundler/src/bundle/settings.rs b/tooling/bundler/src/bundle/settings.rs
index 6f2200b50d1f..bf85fa62378d 100644
--- a/tooling/bundler/src/bundle/settings.rs
+++ b/tooling/bundler/src/bundle/settings.rs
@@ -289,8 +289,8 @@ pub struct NsisSettings {
   /// Whether to display a language selector dialog before the installer and uninstaller windows are rendered or not.
   /// By default the OS language is selected, with a fallback to the first language in the `languages` array.
   pub display_language_selector: bool,
-  /// Set compression algorithm used in the installer.
-  pub set_compressor: Option<NsisCompressor>,
+  /// Set compression algorithm used to compress files in the installer.
+  pub compression: Option<NsisCompressor>,
 }
 
 /// The Windows bundle settings.
diff --git a/tooling/bundler/src/bundle/windows/nsis.rs b/tooling/bundler/src/bundle/windows/nsis.rs
index e45b5b01c43a..bce01b9621b7 100644
--- a/tooling/bundler/src/bundle/windows/nsis.rs
+++ b/tooling/bundler/src/bundle/windows/nsis.rs
@@ -243,16 +243,15 @@ fn build_nsis_app_installer(
         to_json(dunce::canonicalize(sidebar_image)?),
       );
     }
-    if let Some(compressor) = &nsis.set_compressor {
-      data.insert(
-        "set_compressor",
-        to_json(match compressor {
-          NsisCompressor::Zlib => "zlib",
-          NsisCompressor::Bzip2 => "bzip2",
-          NsisCompressor::Lzma => "lzma",
-        }),
-      );
-    }
+    data.insert(
+      "set_compressor",
+      to_json(match compressor.unwrap_or(NsisCompressor::Lzma) {
+        NsisCompressor::Zlib => "zlib",
+        NsisCompressor::Bzip2 => "bzip2",
+        NsisCompressor::Lzma => "lzma",
+      }),
+    );
+    
 
     data.insert(
       "display_language_selector",
diff --git a/tooling/bundler/src/bundle/windows/templates/installer.nsi b/tooling/bundler/src/bundle/windows/templates/installer.nsi
index 7eb7958442c5..9194455cac3d 100644
--- a/tooling/bundler/src/bundle/windows/templates/installer.nsi
+++ b/tooling/bundler/src/bundle/windows/templates/installer.nsi
@@ -28,7 +28,7 @@
 !define WEBVIEW2INSTALLERPATH "{{webview2_installer_path}}"
 !define UNINSTKEY "Software\Microsoft\Windows\CurrentVersion\Uninstall\${PRODUCTNAME}"
 !define MANUPRODUCTKEY "Software\${MANUFACTURER}\${PRODUCTNAME}"
-!define SETCOMPRESSOR "{{set_compressor}}"
+!define COMPRESSION "{{compression}}"
 
 Name "${PRODUCTNAME}"
 BrandingText "{{copyright}}"
@@ -36,11 +36,7 @@ OutFile "${OUTFILE}"
 Unicode true
 
 ; Set the compression algorithm. Default is LZMA.
-!if "${SETCOMPRESSOR}" == ""
-  SetCompressor /SOLID lzma
-!else
-  SetCompressor /SOLID "${SETCOMPRESSOR}"
-!endif
+SetCompressor /SOLID "${SETCOMPRESSOR}"
 
 VIProductVersion "${VERSIONWITHBUILD}"
 VIAddVersionKey "ProductName" "${PRODUCTNAME}"