Include the properties required for the new sidebar

This also includes removes the filter by description and whether a doc object has signatures so that everything is included. Any doc objects with @hide are still filtered out.

Author: chasenlehara

generators/search-map.js

@@ -24,33 +24,43 @@ module.exports = function(docMap, siteConfig) {
 		var searchMap = {};
 		var name;
 		for (name in docMap) {
-			if (docMap.hasOwnProperty(name)) {
-				var docObj = docMap[name];
-				var description = docObj.description && docObj.description.trim();
-				var signaturesHaveContent = docObj.signatures && docObj.signatures.some(function(signature){
-					return signature.params || signature.return || signature.options;
-				});
-				// If there is no body, it's likely we don't want to index it
-				if ((description && description !== 'undefined') || signaturesHaveContent) {
-					var helpers = bitDocsHelpers(docMap, siteConfig, function(){
-						return docObj;
-					}, {});
+			var docObj = (docMap.hasOwnProperty(name)) ? docMap[name] : null;
+			if (docObj && !docObj.hide) {
+
+				var helpers = bitDocsHelpers(docMap, siteConfig, function(){
+					return docObj;
+				}, {});
 
-					// Convert bit-docs markdown to HTML
-					var descriptionAsHTML = helpers.makeLinks(helpers.makeHtml(description));
+				// Convert bit-docs markdown to HTML
+				var description = docObj.description && docObj.description.trim();
+				var descriptionAsHTML = helpers.makeLinks(helpers.makeHtml(description));
 
-					// Only allow certain HTML elements
-					var descriptionAsStrippedHTML = striptags(descriptionAsHTML, ['a', 'code', 'em', 'strong']);
+				// Only allow certain HTML elements
+				var descriptionAsStrippedHTML = striptags(descriptionAsHTML, ['a', 'code', 'em', 'strong']);
 
-					var searchObj = {
-						name: docObj.name,
-						title: docObj.title,
-						description: descriptionAsStrippedHTML,
-						url: filename(docObj, siteConfig),
-						dest: 'doc/'
-					};
-					searchMap[name] = searchObj;
+				var searchObj = {
+					name: docObj.name,
+					title: docObj.title,
+					description: descriptionAsStrippedHTML,
+					url: filename(docObj, siteConfig),
+					dest: 'doc/'
+				};
+				if (docObj.collection) {
+					searchObj.collection = docObj.collection;
+				}
+				if (docObj.order) {
+					searchObj.order = docObj.order;
+				}
+				if (docObj.parent) {
+					searchObj.parent = docObj.parent;
+				}
+				if (docObj.subchildren) {
+					searchObj.subchildren = docObj.subchildren;
+				}
+				if (docObj.type) {
+					searchObj.type = docObj.type;
 				}
+				searchMap[name] = searchObj;
 			}
 		}
 

test.js

@@ -42,6 +42,14 @@ describe("bitDocs.generators.searchMap",function(){
 		});
 	});
 
+	it("(bitDocs.generators.searchMap.searchMap) Excludes docObjects with @hide", function(){
+		return docMapPromise.then(function(docMap){
+			return searchMap(docMap, siteConfig).then(function(searchMapResult){
+				assert.ok(!searchMapResult['guides/what-is-canjs']);
+			});
+		});
+	});
+
 	it("(bitDocs.generators.searchMap.searchMap) Strips HTML from the description", function(){
 		return docMapPromise.then(function(docMap){
 			return searchMap(docMap, siteConfig);
@@ -60,18 +68,18 @@ describe("bitDocs.generators.searchMap",function(){
 		});
 	});
 
-	it("(bitDocs.generators.searchMap.searchMap) Filters hidden and empty docObjects", function(){
+	it("(bitDocs.generators.searchMap.searchMap) Does not filter hidden and empty docObjects", function(){
 		return docMapPromise.then(function(docMap){
 			return searchMap(docMap, siteConfig).then(function(searchMapResult){
-				assert.ok(!searchMapResult.Component);
+				assert.ok(searchMapResult.Component);
 			});
 		});
 	});
 
-	it("(bitDocs.generators.searchMap.searchMap) Filters docObjects with only “undefined\n” as their description", function(){
+	it("(bitDocs.generators.searchMap.searchMap) Does not filter docObjects with only “undefined\n” as their description", function(){
 		return docMapPromise.then(function(docMap){
 			return searchMap(docMap, siteConfig).then(function(searchMapResult){
-				assert.ok(!searchMapResult['can.Control']);
+				assert.ok(searchMapResult['can.Control']);
 			});
 		});
 	});

testDocMap.json

@@ -178,5 +178,11 @@
     "description": "undefined\n",
     "url": "can.Control.html",
     "dest": "doc/"
+  },
+  "guides/what-is-canjs": {
+    "description": "\n",
+    "name": "guides/what-is-canjs",
+    "title": "What is CanJS?",
+    "hide": true
   }
 }