fix: special property names in flowtypes

documentationjs · Sep 14, 2017 · 7fc6ca8 · 7fc6ca8
1 parent a820174
commit 7fc6ca8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/flow_doctrine.js b/src/flow_doctrine.js
@@ -27,7 +27,7 @@ function propertyToField(property) {
   }
   return {
     type: 'FieldType',
-    key: property.key.name,
+    key: property.key.name || property.key.value,
     value: type
   };
 }

diff --git a/src/infer/params.js b/src/infer/params.js
@@ -197,8 +197,8 @@ function paramToDoc(
       });
     }
     case 'ObjectProperty': {
-      return _.assign(paramToDoc(param.value, prefix + '.' + param.key.name), {
-        name: prefix + '.' + param.key.name
+      return _.assign(paramToDoc(param.value, prefix + '.' + param.key.name || param.key.value), {
+        name: prefix + '.' + param.key.name || param.key.value
       });
     }
     case 'RestProperty': // (a, ...b)

diff --git a/src/infer/properties.js b/src/infer/properties.js
@@ -12,6 +12,7 @@ function prefixedName(name, prefix) {
 
 function propertyToDoc(property, prefix): CommentTag {
   var type = flowDoctrine(property.value);
+  var name = property.key.name || property.key.value
   if (property.optional) {
     type = {
       type: 'OptionalType',
@@ -20,7 +21,7 @@ function propertyToDoc(property, prefix): CommentTag {
   }
   return {
     title: 'property',
-    name: prefixedName(property.key.name, prefix),
+    name: prefixedName(name, prefix),
     lineNumber: property.loc.start.line,
     type
   };