Merge branch '32-filename-sanitization-can-fail-with-ascii-only-target' into 'dev'

Resolve "filename sanitization can fail with ascii-only target"

See merge request atom/AtOM!46

lib/files/getDestFile

@@ -55,11 +55,13 @@ getDestFile() {
 		destfile="${filename##*/}"
 		destfile="${destfile%.*}"
 	fi
-	destfile=$(sanitizeFile "$destfile")
-	destfile=${destfile//$'\n'/::AtOM:NewLine:SQL:Inline::}
 	if (( ${destinationascii["$destination"]} ))
 	then
+		destfile=${destfile//$'\n'/::AtOM:NewLine:SQL:Inline::}
 		echo "$destfile" >&${toascii[1]}
 		read -r -u${toascii[0]} destfile
+		destfile=${destfile//::AtOM:NewLine:SQL:Inline::/$'\n'}
 	fi
+	destfile=$(sanitizeFile "$destfile")
+	destfile=${destfile//$'\n'/::AtOM:NewLine:SQL:Inline::}
 }

lib/files/sanitizeFile

@@ -26,6 +26,9 @@ sanitizeFile() {
 			string=${string/#+(.)/}
 			string=${string/%+(.)/}
 		fi
+
+		# We can't have newlines either
+		string=${string//$'\n'/ }
 	fi
 	echo "$string"
 }