Better detect if fallback editor needs to be used

src/cli/init.go

@@ -17,7 +17,7 @@ func TempInitCli() {
 	}
 
 	// textEditor
-	textEditor := input("Text editor (leave blank for $EDITOR, falls back to \"vi\"):")
+	textEditor := input("Text editor (leave blank for $EDITOR, falls back to \"" + offline.FallbackEditor + "\"):")
 
 	offline.TempInit(map[string]string{"textEditor": textEditor, "gpgID": gpgID})
 }

src/offline/init.go

@@ -16,9 +16,9 @@ func TempInit(configFileMap map[string]string) {
 
 	// ensure textEditor is set
 	if configFileMap["textEditor"] == "" {
-		textEditor := os.Getenv("EDITOR")
+		textEditor, editorEnvPresent := os.LookupEnv("EDITOR")
-		if textEditor == "" {
+		if !editorEnvPresent {
-			textEditor = fallbackEditor
+			textEditor = FallbackEditor
 		}
 		configFileMap["textEditor"] = textEditor
 	}

src/offline/initUNIX.go

@@ -7,7 +7,7 @@ import (
 	"os"
 )
 
-const fallbackEditor = "vi" // vi is pre-installed on most UNIX systems
+const FallbackEditor = "vi" // vi is pre-installed on most UNIX systems
 
 // dirInit creates the libmutton directories
 func dirInit() {

src/offline/initWIN.go

@@ -7,7 +7,7 @@ import (
 	"os"
 )
 
-const fallbackEditor = "nvim" // since there is no pre-installed CLI editor on Windows, default to the most popular one
+const FallbackEditor = "nvim" // since there is no pre-installed CLI editor on Windows, default to the most popular one
 
 // dirInit creates the libmutton directories
 func dirInit() {