Consistency of undefined fallback

Author: JamesHenry

lib/convert.js

@@ -808,7 +808,7 @@ module.exports = function convert(config) {
                 value: convertChild(node.initializer),
                 computed: nodeUtils.isComputedProperty(node.name),
                 static: nodeUtils.hasStaticModifierFlag(node),
-                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) ? true : undefined
+                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) || undefined
             });
 
             if (node.type) {
@@ -1291,9 +1291,9 @@ module.exports = function convert(config) {
                     range: [node.getStart(), node.end],
                     loc: nodeUtils.getLoc(node, ast),
                     accessibility: nodeUtils.getTSNodeAccessibility(node),
-                    readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) ? true : undefined,
+                    readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) || undefined,
                     static: nodeUtils.hasModifier(SyntaxKind.StaticKeyword, node),
-                    export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) ? true : undefined,
+                    export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) || undefined,
                     parameter: result
                 };
             }
@@ -1946,9 +1946,9 @@ module.exports = function convert(config) {
                 key: convertChild(node.name),
                 params: convertParameters(node.parameters),
                 typeAnnotation: (node.type) ? convertTypeAnnotation(node.type) : null,
-                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) ? true : undefined,
+                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) || undefined,
                 static: nodeUtils.hasModifier(SyntaxKind.StaticKeyword, node),
-                export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) ? true : undefined
+                export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) || undefined
             });
 
             const accessibility = nodeUtils.getTSNodeAccessibility(node);
@@ -1971,9 +1971,9 @@ module.exports = function convert(config) {
                 key: convertChild(node.name),
                 typeAnnotation: (node.type) ? convertTypeAnnotation(node.type) : undefined,
                 initializer: convertChild(node.initializer) || undefined,
-                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) ? true : undefined,
-                static: nodeUtils.hasModifier(SyntaxKind.StaticKeyword, node) ? true : undefined,
-                export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) ? true : undefined
+                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) || undefined,
+                static: nodeUtils.hasModifier(SyntaxKind.StaticKeyword, node) || undefined,
+                export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) || undefined
             });
 
             const accessibility = nodeUtils.getTSNodeAccessibility(node);
@@ -1989,9 +1989,9 @@ module.exports = function convert(config) {
                 type: AST_NODE_TYPES.TSIndexSignature,
                 index: convertChild(node.parameters[0]),
                 typeAnnotation: (node.type) ? convertTypeAnnotation(node.type) : null,
-                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) ? true : undefined,
+                readonly: nodeUtils.hasModifier(SyntaxKind.ReadonlyKeyword, node) || undefined,
                 static: nodeUtils.hasModifier(SyntaxKind.StaticKeyword, node),
-                export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) ? true : undefined
+                export: nodeUtils.hasModifier(SyntaxKind.ExportKeyword, node) || undefined
             });
 
             const accessibility = nodeUtils.getTSNodeAccessibility(node);