no more errors on missing name, but a nice warning

Author: justinbmeyer

.gitignore

@@ -15,3 +15,5 @@ lib/configured/test/tmp/
 lib/configured/test/api/
 lib/configured/test/docs/
 lib/generators/html/test/tmp/
+.DS_Store
+.DS_Store?

documentjs.json

@@ -3,7 +3,7 @@
 		"docs": {
 			"glob": {
 				"pattern": "{docs,tags,lib}/**/*.{js,md}",
-				"ignore": "lib/{configured,generate,find}/test/**/*"
+				"ignore": "lib/{configured,generate,find,generators/html}/test/**/*"
 			}
 		},
 		"examples/multi": {

lib/configured/generate_project.js

@@ -73,6 +73,8 @@ function document(project, parent, options ){
 module.exports = document;
 
 /**
+ * @function documentjs.configured.generateProject generateProject
+ * @hide
  * 
  * @param {Object} docConfig A docConfig loaded from `{project.path}/documentjs.json`.
  * 

lib/process/add_doc_object_to_doc_map.js

@@ -1,3 +1,5 @@
+var _ = require("lodash");
+
 module.exports = function(docObject, docMap, filename, line){
 	if (docObject.name) {
 
@@ -21,14 +23,8 @@ module.exports = function(docObject, docMap, filename, line){
 		if (line) {
 			docObject.line = line;
 		}
-	} else {
-		throw new Error({
-			message: "Created a docObject without a name. "+(filename || "" + (line ? ":"+line : "")), 
-			docObject: docObject,
-			toString: function(){
-				return this.message;
-			}
-		});
+	} else if(!_.isEmpty( _.omit(docObject,['body','description']) )  && docObject.type !== "hide"){
+		console.log("WARNING!!\nNo name for:\n",docObject);
 	}
 };
 

lib/process/clean_doc_map.js

@@ -6,7 +6,6 @@ module.exports = function(docMap, options){
 
 	if(!options.parent) {
 		var info = docMapInfo(docMap);
-		
 		if(_.size(info.nameHeightMap) === 1) {
 			// everything is under one object, so use that
 			options.parent = info.maxName;
@@ -19,6 +18,9 @@ module.exports = function(docMap, options){
 				type: "page"
 			};
 			options.parent = info.maxParent;
+		} else if(info.sortedNames.length > 1 && info.sortedNames[0].childCount > info.sortedNames[1].childCount * 5) {
+			options.parent = info.maxName;
+			console.log("Parent-less comments: "+_.map(info.sortedNames.slice(1),"name")+".");
 		} else {
 			// we need to balance an empty project, which should probably have everything
 			// just added to it
@@ -35,7 +37,7 @@ module.exports = function(docMap, options){
 			});
 
 			options.parent = maxParent;
-			
+			console.log("Parent-less comments:"+_.keys(info.nameHeightMap).join(", ")+".");
 		}
 
 		console.warn("Guessed parent '"+options.parent+"'. Set parent in your siteConfig.");

lib/process/doc_map_info.js

@@ -1,5 +1,5 @@
 // Goes through the graph and figures out the highest parent
-
+var _ = require("lodash");
 
 var getTop = function(docMap, child){
 	var parent;
@@ -53,9 +53,13 @@ module.exports = function(docMap){
 		}
 
 	}
+	var sortedNameHeightMap= _.sortBy(_.map(nameHeightMap, function(val, name){
+		return {name: name, childCount: val};
+	}), "childCount").reverse();
 	return {
 		maxName: maxName,
 		nameHeightMap: nameHeightMap,
+		sortedNames: sortedNameHeightMap,
 		maxParent: maxParent,
 		parentHeightMap: parentHeightMap
 	};