theme/candy: SC2154

gaelicWizard · gaelicWizard · commit c7c447a54b75 · 2022-01-08T08:45:52.000-08:00
Handle all unbound parameters, even colors!
diff --git a/themes/candy/candy.theme.bash b/themes/candy/candy.theme.bash
@@ -1,12 +1,14 @@
 # shellcheck shell=bash
 # shellcheck disable=SC2034 # Expected behavior for themes.
-# shellcheck disable=SC2154 #TODO: fix these all.
 
 function prompt_command() {
-	PS1="${green}\u@\h $(clock_prompt) ${reset_color}${white}\w${reset_color}$(scm_prompt_info)${blue} →${bold_blue} ${reset_color} ${normal}"
+	local clock_prompt scm_prompt_info
+	clock_prompt="$(clock_prompt)"
+	scm_prompt_info="$(scm_prompt_info)"
+	PS1="${green?}\u@\h ${clock_prompt} ${reset_color?}${white?}\w${reset_color?}${scm_prompt_info}${blue?} →${bold_blue?} ${reset_color?} ${normal?}"
 }
 
-THEME_CLOCK_COLOR=${THEME_CLOCK_COLOR:-"$blue"}
-THEME_CLOCK_FORMAT=${THEME_CLOCK_FORMAT:-"%I:%M:%S"}
+: "${THEME_CLOCK_COLOR:=${blue?}}"
+: "${THEME_CLOCK_FORMAT:="%I:%M:%S"}"
 
 safe_append_prompt_command prompt_command
