gtx-cli 2.4.9 → 2.4.10
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +6 -0
- package/dist/react/jsx/utils/parseJsx.js +22 -7
- package/package.json +1 -1
package/CHANGELOG.md
CHANGED
|
@@ -1,5 +1,11 @@
|
|
|
1
1
|
# gtx-cli
|
|
2
2
|
|
|
3
|
+
## 2.4.10
|
|
4
|
+
|
|
5
|
+
### Patch Changes
|
|
6
|
+
|
|
7
|
+
- [#778](https://github.com/generaltranslation/gt/pull/778) [`0d8f414`](https://github.com/generaltranslation/gt/commit/0d8f4144696873b15f0aaa6744bdb7390d472279) Thanks [@SamEggert](https://github.com/SamEggert)! - skip dynamic expressions for HTML content props in parseJSX
|
|
8
|
+
|
|
3
9
|
## 2.4.9
|
|
4
10
|
|
|
5
11
|
### Patch Changes
|
|
@@ -8,6 +8,7 @@ import { isAcceptedPluralForm } from 'generaltranslation/internal';
|
|
|
8
8
|
import { handleChildrenWhitespace } from '../trimJsxStringChildren.js';
|
|
9
9
|
import { isStaticExpression } from '../evaluateJsx.js';
|
|
10
10
|
import { GT_ATTRIBUTES, mapAttributeName, VARIABLE_COMPONENTS, } from './constants.js';
|
|
11
|
+
import { HTML_CONTENT_PROPS } from 'generaltranslation/types';
|
|
11
12
|
/**
|
|
12
13
|
* Builds a JSX tree from a given node, recursively handling children.
|
|
13
14
|
* @param node - The node to build the tree from
|
|
@@ -80,15 +81,29 @@ export function buildJSXTree(importAliases, node, unwrappedExpressions, updates,
|
|
|
80
81
|
attrValue = attr.value.value;
|
|
81
82
|
}
|
|
82
83
|
else if (t.isJSXExpressionContainer(attr.value)) {
|
|
83
|
-
if (
|
|
84
|
-
|
|
85
|
-
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
84
|
+
// Check if this is an HTML content prop (title, placeholder, alt, etc.)
|
|
85
|
+
const isHtmlContentProp = Object.values(HTML_CONTENT_PROPS).includes(attrName);
|
|
86
|
+
if (isHtmlContentProp) {
|
|
87
|
+
// For HTML content props, only accept static string expressions
|
|
88
|
+
const staticAnalysis = isStaticExpression(attr.value.expression);
|
|
89
|
+
if (staticAnalysis.isStatic &&
|
|
90
|
+
staticAnalysis.value !== undefined) {
|
|
91
|
+
attrValue = staticAnalysis.value;
|
|
89
92
|
}
|
|
93
|
+
// Otherwise attrValue stays null and won't be included
|
|
94
|
+
}
|
|
95
|
+
else {
|
|
96
|
+
// For non-HTML-content props, validate plural/branch then build tree
|
|
97
|
+
if ((elementIsPlural && isAcceptedPluralForm(attrName)) ||
|
|
98
|
+
(elementIsBranch && attrName !== 'branch')) {
|
|
99
|
+
// Make sure that variable strings like {`I have ${count} book`} are invalid!
|
|
100
|
+
if (t.isTemplateLiteral(attr.value.expression) &&
|
|
101
|
+
!isStaticExpression(attr.value.expression).isStatic) {
|
|
102
|
+
unwrappedExpressions.push(generate(attr.value).code);
|
|
103
|
+
}
|
|
104
|
+
}
|
|
105
|
+
attrValue = buildJSXTree(importAliases, attr.value.expression, unwrappedExpressions, updates, errors, warnings, file, true);
|
|
90
106
|
}
|
|
91
|
-
attrValue = buildJSXTree(importAliases, attr.value.expression, unwrappedExpressions, updates, errors, warnings, file, true);
|
|
92
107
|
}
|
|
93
108
|
}
|
|
94
109
|
props[attrName] = attrValue;
|