theme/essential: SC2154

Handle all unbound parameters, even colors!
pull/2012/head
John D Pell 2021-09-24 13:13:06 -07:00
parent c81c9dcc8c
commit 9f79848eed
1 changed files with 16 additions and 19 deletions

View File

@ -1,42 +1,39 @@
#!/usr/bin/env bash # shellcheck shell=bash
# https://github.com/koalaman/shellcheck/wiki/Sc2154
# shellcheck disable=SC2154
function _user-prompt() { function _user-prompt() {
local -r user='\\u' local -r user='\\u'
if [[ "${EUID}" -eq 0 ]]; then if [[ "${EUID}" -eq 0 ]]; then
# Privileged users: # Privileged users:
local -r user_color="${bold_red}" local -r user_color="${bold_red?}"
else else
# Standard users: # Standard users:
local -r user_color="${bold_green}" local -r user_color="${bold_green?}"
fi fi
# Print the current user's name (colored according to their current EUID): # Print the current user's name (colored according to their current EUID):
echo -e "${user_color}${user}${normal}" echo -e "${user_color}${user}${normal?}"
} }
function _host-prompt() { function _host-prompt() {
local -r host='\\h' local -r host='\\h'
# Check whether or not $SSH_TTY is set: # Check whether or not $SSH_TTY is set:
if [[ -z "${SSH_TTY}" ]]; then if [[ -z "${SSH_TTY:-}" ]]; then
# For local hosts, set the host's prompt color to blue: # For local hosts, set the host's prompt color to blue:
local -r host_color="${bold_blue}" local -r host_color="${bold_blue?}"
else else
# For remote hosts, set the host's prompt color to red: # For remote hosts, set the host's prompt color to red:
local -r host_color="${bold_red}" local -r host_color="${bold_red?}"
fi fi
# Print the current hostname (colored according to $SSH_TTY's status): # Print the current hostname (colored according to $SSH_TTY's status):
echo -e "${host_color}${host}${normal}" echo -e "${host_color}${host}${normal?}"
} }
function _user-at-host-prompt() { function _user-at-host-prompt() {
# Concatenate the user and host prompts into: user@host: # Concatenate the user and host prompts into: user@host:
echo -e "$(_user-prompt)${bold_white}@$(_host-prompt)" echo -e "$(_user-prompt)${bold_white?}@$(_host-prompt)"
} }
function _exit-status-prompt() { function _exit-status-prompt() {
@ -47,32 +44,32 @@ function _exit-status-prompt() {
if [[ "${exit_status}" -eq 0 ]]; then if [[ "${exit_status}" -eq 0 ]]; then
# For commands that return an exit status of zero, set the exit status's # For commands that return an exit status of zero, set the exit status's
# notifier to green: # notifier to green:
local -r exit_status_color="${bold_green}" local -r exit_status_color="${bold_green?}"
else else
# For commands that return a non-zero exit status, set the exit status's # For commands that return a non-zero exit status, set the exit status's
# notifier to red: # notifier to red:
local -r exit_status_color="${bold_red}" local -r exit_status_color="${bold_red?}"
fi fi
echo -ne "${exit_status_color}" echo -ne "${exit_status_color}"
if [[ "${prompt_string}" -eq 1 ]]; then if [[ "${prompt_string}" -eq 1 ]]; then
# $PS1: # $PS1:
echo -e " +${normal} " echo -e " +${normal?} "
elif [[ "${prompt_string}" -eq 2 ]]; then elif [[ "${prompt_string}" -eq 2 ]]; then
# $PS2: # $PS2:
echo -e " |${normal} " echo -e " |${normal?} "
else else
# Default: # Default:
echo -e " ?${normal} " echo -e " ?${normal?} "
fi fi
} }
function _ps1() { function _ps1() {
local -r time='\\t' local -r time='\\t'
echo -ne "${bold_white}${time} " echo -ne "${bold_white?}${time} "
echo -ne "$(_user-at-host-prompt)" echo -ne "$(_user-at-host-prompt)"
echo -e "${bold_white}:${normal}${PWD}" echo -e "${bold_white?}:${normal?}${PWD}"
echo -e "$(_exit-status-prompt 1 "${exit_status}")" echo -e "$(_exit-status-prompt 1 "${exit_status}")"
} }