🐛 Added support for attributes that are not assigned

deamme · Oct 13, 2017 · 7323a76 · 7323a76
1 parent 038cf81
commit 7323a76
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 8 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ts-transform-inferno",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "A typescript transformer for InfernoJS",
   "bugs": {
     "url": "https://github.com/deamme/ts-transform-inferno/issues"

diff --git a/src/index.ts b/src/index.ts
@@ -220,6 +220,7 @@ export default function (CONFIG?: Config) {
       let assignArgs = [];
       for (let i = 0; i < astProps.length; i++) {
         let astProp = astProps[i];
+        const initializer = astProp.initializer
 
         if (astProp.kind === ts.SyntaxKind.JsxSpreadAttribute) {
           assignArgs = [
@@ -239,12 +240,12 @@ export default function (CONFIG?: Config) {
             !isComponent &&
             (propName === "className" || propName === "class")
           ) {
-            className = getValue(astProp.initializer);
+            className = getValue(initializer);
           } else if (!isComponent && propName === "htmlFor") {
             props.push(
               ts.createPropertyAssignment(
                 getName("for"),
-                getValue(astProp.initializer)
+                getValue(initializer)
               )
             );
           } else if (propName.substr(0, 11) === "onComponent" && isComponent) {
@@ -256,15 +257,15 @@ export default function (CONFIG?: Config) {
             ref.properties.push(
               ts.createObjectLiteral(
                 getName(propName),
-                getValue(astProp.initializer)
+                getValue(initializer)
               )
             );
           } else if (!isComponent && propName in svgAttributes) {
             // React compatibility for SVG Attributes
             props.push(
               ts.createPropertyAssignment(
                 getName(svgAttributes[propName]),
-                getValue(astProp.initializer)
+                getValue(initializer)
               )
             );
           } else {
@@ -279,16 +280,16 @@ export default function (CONFIG?: Config) {
                 hasKeyedChildren = true;
                 break;
               case "ref":
-                ref = getValue(astProp.initializer);
+                ref = getValue(initializer);
                 break;
               case "key":
-                key = getValue(astProp.initializer);
+                key = getValue(initializer);
                 break;
               default:
                 props.push(
                   ts.createPropertyAssignment(
                     getName(propName),
-                    getValue(astProp.initializer)
+                    initializer ? getValue(initializer) : ts.createTrue()
                   )
                 );
             }

diff --git a/tests/cases/divWithNoAssignedAttributes.tsx b/tests/cases/divWithNoAssignedAttributes.tsx
@@ -0,0 +1 @@
+<div foo Bar STAR />
diff --git a/tests/references/divWithNoAssignedAttributes.jsx b/tests/references/divWithNoAssignedAttributes.jsx
@@ -0,0 +1,3 @@
+var Inferno = require("inferno");
+var createVNode = Inferno.createVNode;
+createVNode(2, "div", null, null, { "foo": true, "Bar": true, "STAR": true });