[Fix] jsx-curly-brace-presence: avoid autofixing attributes with do…

…uble quotes to a double quoted attribute

Fixes #3814

(backport from cf868f2)

CHANGELOG.md

@@ -5,6 +5,11 @@ This change log adheres to standards from [Keep a CHANGELOG](https://keepachange
 
 ## Unreleased
 
+### Fixed
+* [`jsx-curly-brace-presence`]: avoid autofixing attributes with double quotes to a double quoted attribute ([#3814][] @ljharb)
+
+[#3814]: https://github.com/jsx-eslint/eslint-plugin-react/issues/3814
+
 ## [7.35.1] - 2024.09.02
 
 ### Fixed

lib/rules/jsx-curly-brace-presence.js

@@ -199,10 +199,14 @@ module.exports = {
             const parentType = JSXExpressionNode.parent.type;
 
             if (parentType === 'JSXAttribute') {
-              textToReplace = `"${expressionType === 'TemplateLiteral'
-                ? expression.quasis[0].value.raw
-                : expression.raw.slice(1, -1)
-              }"`;
+              if (expressionType !== 'TemplateLiteral' && /["]/.test(expression.raw.slice(1, -1))) {
+                textToReplace = expression.raw;
+              } else {
+                textToReplace = `"${expressionType === 'TemplateLiteral'
+                  ? expression.quasis[0].value.raw
+                  : expression.raw.slice(1, -1)
+                }"`;
+              }
             } else if (jsxUtil.isJSX(expression)) {
               textToReplace = getText(context, expression);
             } else {

tests/lib/rules/jsx-curly-brace-presence.js

@@ -945,5 +945,15 @@ ruleTester.run('jsx-curly-brace-presence', rule, {
       output: `<Foo bar="'" />`,
       errors: [{ messageId: 'unnecessaryCurly' }],
     },
+    {
+      code: `
+        <Foo help={'The maximum time range for searches. (i.e. "P30D" for 30 days, "PT24H" for 24 hours)'} />
+      `,
+      options: ['never'],
+      output: `
+        <Foo help='The maximum time range for searches. (i.e. "P30D" for 30 days, "PT24H" for 24 hours)' />
+      `,
+      errors: [{ messageId: 'unnecessaryCurly' }],
+    }
   )),
 });