advance search - schema keys generated using set() to avoid duplicates

Author: mrin9

dist/rapidoc-min.js

@@ -4,6 +4,14 @@ info:
   version: 1.0.0
   title: Testing different data-types
 paths:
+  /nested-object-in-request-body:
+    post:
+      summary: Request body accepting a JSON
+      requestBody:
+        content:
+          application/json:
+            schema:
+              $ref: "#/components/schemas/Person"
   /nested-object:
     get:
       summary: Response schema made of nested objects. 
@@ -173,7 +181,7 @@ components:
     Category:
       type: object
       properties:
-        id:
+        catId:
           description: Category ID
           allOf:
             - $ref: '#/components/schemas/Id'
@@ -199,7 +207,7 @@ components:
           description: Categories this person belongs to
           allOf:
             - $ref: '#/components/schemas/Category'
-        DependentIds:
+        dependentIds:
           type: array
           description: IDs of Dependents .
           items:
@@ -250,4 +258,6 @@ components:
         name:
           description: Tag name
           type: string
-          minLength: 1
+          minLength: 1
+        fff:
+          type: string  

dist/rapidoc-min.js.gz

@@ -53,36 +53,30 @@ export function pathIsInSearch(searchVal, path) {
   return stringToSearch.includes(searchVal);
 }
 
-export function joinObjectKeys(obj, resultStr = '') {
-  if (!obj.properties) {
-    return '';
+export function schemaKeys(schemaProps, result = new Set()) {
+  if (!schemaProps) {
+    return result;
   }
-
-  // We should use a separated string variable in order to exclude next error 'Uncaught RangeError: Invalid string length'
-  let recursiveresultStr = '';
-  resultStr = `${resultStr} ${Object.keys(obj.properties || '').join(' ')}`;
-
-  Object.keys(obj.properties).forEach((propertyName) => {
-    if (obj.properties[propertyName].properties) {
-      recursiveresultStr = `${recursiveresultStr} ${joinObjectKeys(obj.properties[propertyName], resultStr)}`;
-    }
-    if (obj.properties[propertyName].type === 'array') {
-      recursiveresultStr = `${recursiveresultStr} ${joinObjectKeys(obj.properties[propertyName].items, resultStr)}`;
+  Object.keys(schemaProps).forEach((key) => {
+    result.add(key);
+    if (schemaProps[key].properties) {
+      schemaKeys(schemaProps[key].properties, result);
+    } else if (schemaProps[key].items?.properties) {
+      schemaKeys(schemaProps[key].items?.properties, result);
     }
   });
-
-  return `${resultStr} ${recursiveresultStr}`;
+  return result;
 }
 
 export function advanceSearch(searchVal, allSpecTags, searchOptions = []) {
   if (!searchVal.trim() || searchOptions.length === 0) {
     return;
   }
 
-  let stringToSearch = '';
   const pathsMatched = [];
   allSpecTags.forEach((tag) => {
     tag.paths.forEach((path) => {
+      let stringToSearch = '';
       if (searchOptions.includes('search-api-path')) {
         stringToSearch = path.path;
       }
@@ -94,8 +88,12 @@ export function advanceSearch(searchVal, allSpecTags, searchOptions = []) {
       }
 
       if (searchOptions.includes('search-api-request-body') && path.requestBody) {
-        for (const contentType in path.requestBody.content) {
-          stringToSearch = `${stringToSearch} ${joinObjectKeys(path.requestBody.content[contentType].schema, stringToSearch)}`;
+        let schemaKeySet = new Set();
+        for (const contentType in path.requestBody?.content) {
+          if (path.requestBody.content[contentType].schema?.properties) {
+            schemaKeySet = schemaKeys(path.requestBody.content[contentType].schema?.properties);
+          }
+          stringToSearch = `${stringToSearch} ${[...schemaKeySet].join(' ')}`;
         }
       }