diff --git a/CHANGELOG.md b/CHANGELOG.md index 6689dac..d7e4181 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,7 @@ ### Added - SC2254: Suggest quoting expansions in case statements - SC2255: Suggest using `$((..))` in `[ 2*3 -eq 6 ]` +- SC2256: Warn about translated strings that are known variables ## v0.7.0 - 2019-07-28 ### Added diff --git a/src/ShellCheck/Analytics.hs b/src/ShellCheck/Analytics.hs index 97054c1..b0928ce 100644 --- a/src/ShellCheck/Analytics.hs +++ b/src/ShellCheck/Analytics.hs @@ -192,6 +192,7 @@ nodeChecks = [ ,checkRedirectionToCommand ,checkDollarQuoteParen ,checkUselessBang + ,checkTranslatedString ] optionalChecks = map fst optionalTreeChecks @@ -3451,6 +3452,24 @@ checkDollarQuoteParen params t = where fix id = fixWith [replaceStart id params 2 "\"$"] +prop_checkTranslatedString1 = verify checkTranslatedString "foo_bar2=val; $\"foo_bar2\"" +prop_checkTranslatedString2 = verifyNot checkTranslatedString "$\"foo_bar2\"" +prop_checkTranslatedString3 = verifyNot checkTranslatedString "$\"..\"" +prop_checkTranslatedString4 = verifyNot checkTranslatedString "var=val; $\"$var\"" +prop_checkTranslatedString5 = verifyNot checkTranslatedString "foo=var; bar=val2; $\"foo bar\"" +checkTranslatedString params (T_DollarDoubleQuoted id ((T_Literal _ s):_)) = + fromMaybe (return ()) $ do + Map.lookup s assignments + return $ + warnWithFix id 2256 "This translated string is the name of a variable. Flip leading $ and \" if this should be a quoted substitution." (fix id) + where + assignments = foldl (flip ($)) Map.empty (map insertAssignment $ variableFlow params) + insertAssignment (Assignment (_, token, name, _)) | isVariableName name = + Map.insert name token + insertAssignment _ = Prelude.id + fix id = fixWith [replaceStart id params 2 "\"$"] +checkTranslatedString _ _ = return () + prop_checkDefaultCase1 = verify checkDefaultCase "case $1 in a) true ;; esac" prop_checkDefaultCase2 = verify checkDefaultCase "case $1 in ?*?) true ;; *? ) true ;; esac" prop_checkDefaultCase3 = verifyNot checkDefaultCase "case $1 in x|*) true ;; esac"