Fix condition for ISO-8859-1 encoding selection in GetTargetEncoding method (#681)

Shane32 · web-flow · commit 80448a12990f · 2025-10-14T15:34:47.000-04:00
diff --git a/QRCoder/QRCodeGenerator/ByteDataSegment.cs b/QRCoder/QRCodeGenerator/ByteDataSegment.cs
@@ -122,7 +122,7 @@ private static Encoding GetTargetEncoding(string plainText, EciMode eciMode, boo
         Encoding targetEncoding;
 
         // Check if the text is valid ISO-8859-1 and UTF-8 is not forced, then encode using ISO-8859-1.
-        if (IsValidISO(plainText) && !forceUtf8)
+        if (eciMode == EciMode.Default && !forceUtf8 && IsValidISO(plainText))
         {
             targetEncoding = _iso8859_1;
             includeUtf8BOM = false;

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ private static Encoding GetTargetEncoding(string plainText, EciMode eciMode, boo`
`122`	`122`	`Encoding targetEncoding;`
`123`	`123`
`124`	`124`	`// Check if the text is valid ISO-8859-1 and UTF-8 is not forced, then encode using ISO-8859-1.`
`125`		`- if (IsValidISO(plainText) && !forceUtf8)`
	`125`	`+ if (eciMode == EciMode.Default && !forceUtf8 && IsValidISO(plainText))`
`126`	`126`	`{`
`127`	`127`	`targetEncoding = _iso8859_1;`
`128`	`128`	`includeUtf8BOM = false;`