Merge pull request #224 from smacker/fix_values_as_search_result

Fix search results that has only values
bblfsh · Aug 8, 2019 · 73f2105 · 73f2105
2 parents 4ef0ebc + 61d9e8f
commit 73f2105
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/components/UASTViewer.js b/src/components/UASTViewer.js
@@ -13,7 +13,13 @@ export const getSearchResults = uast => {
     return null;
   }
 
-  if (Array.isArray(rootNode.n)) {
+  // for search server returns [result, result, result]
+  // in case when all the results are nodes we can ignore wrapper node
+  // and display only results for better UI/UX.
+  //
+  // for values (strings, ints, bool) we have to keep the wrapping node
+  // otherwise it's not a tree and uast-viewer wouldn't be able to display it
+  if (Array.isArray(rootNode.n) && rootNode.n.every(c => c.id !== undefined)) {
     return rootNode.n.map(c => c.id);
   }