Merge pull request #654 from illicitonion/spuriousdiv

Remove spurious </div> from empty object response schema
swagger-api · Dec 8, 2015 · 1379944 · 1379944
2 parents c1c60d4 + ab9f2fa
commit 1379944
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/browser/swagger-client.js b/browser/swagger-client.js
diff --git a/browser/swagger-client.min.js b/browser/swagger-client.min.js
diff --git a/lib/schema-markup.js b/lib/schema-markup.js
@@ -469,7 +469,7 @@ function schemaToHTML(name, schema, models, modelPropertyMacro) {
         html += '<div>' + addReference(schema, name) + '</div>';
       } else if (type === 'object') {
         if (_.isPlainObject(schema.properties)) {
-          html += _.map(schema.properties, function (property, name) {
+          var contents = _.map(schema.properties, function (property, name) {
             var propertyIsRequired = (_.indexOf(schema.required, name) >= 0);
             var cProperty = _.cloneDeep(property);
 
@@ -520,7 +520,9 @@ function schemaToHTML(name, schema, models, modelPropertyMacro) {
           }).join(',</div>');
         }
 
-        html += '</div>';
+        if (contents) {
+          html += contents + '</div>';
+        }
       } else {
         html += '<div>' + primitiveToOptionsHTML(schema, type) + '</div>';
       }