From 5f91c84f47f0e0749501544127c94a8b4af0027c Mon Sep 17 00:00:00 2001
From: Mrin <mrin9@yahoo.com>
Date: Thu, 12 Sep 2019 16:30:04 -0700
Subject: [PATCH] fixed page breaks and extra spaces between various
 status-codes with missing body

---
 index.html                 |  6 ++--
 src/utils/common-utils.js  | 65 ++++++++++++--------------------------
 src/utils/pdf-gen-utils.js | 54 +++++++++++++++++--------------
 src/utils/pdf-gen.js       |  7 ++--
 4 files changed, 57 insertions(+), 75 deletions(-)

diff --git a/index.html b/index.html
index 639ffb5..b3c193f 100644
--- a/index.html
+++ b/index.html
@@ -17,11 +17,11 @@
     https://assets.zuora.com/zuora-documentation/swagger.yaml <<< Large spec with lot of markdown
   -->
 
-  
-  <rapi-pdf id='rapipdf' spec-url="./examples/specs/test.json" include-info="false" include-security="false" include-toc="false"> </rapi-pdf>
   <!--
-  <rapi-pdf id='rapipdf' spec-url="https://api.apis.guru/v2/specs/bitbucket.org/2.0/swagger.json" include-info="false" include-security="false" include-toc="false"> </rapi-pdf>
+  <rapi-pdf id='rapipdf' spec-url="./examples/specs/test.json" include-info="false" include-security="false" include-toc="false"> </rapi-pdf>
   -->
+  <rapi-pdf id='rapipdf' spec-url="https://petstore.swagger.io/v2/swagger.json"> </rapi-pdf>
+  
 
   <button onclick="changeSpec()">Change Spec</button>
   <script type="text/javascript">
diff --git a/src/utils/common-utils.js b/src/utils/common-utils.js
index 6c7feda..98d438e 100644
--- a/src/utils/common-utils.js
+++ b/src/utils/common-utils.js
@@ -194,13 +194,7 @@ export function schemaToPdf (schema, obj=[], name, level=0) {
         { 
           colSpan: 3,
           stack:[
-            (name ? {
-              text:[ 
-                {text:name, style:['small', 'mono', 'blue']}, 
-                {text:` {`, style:['small', 'mono', 'blue']},
-              ],
-            }
-            :{text:`root { `, style:['small', 'mono', 'blue']}),
+            {text:`${name?name:'object'} { `, style:['small', 'mono', 'blue']}, 
             {text:(schema.description ? schema.description : ''), style:['sub', 'blue'],margin:[0,2,0,0]},
             {
               margin: [10, 0, 0, 0],
@@ -257,13 +251,7 @@ export function schemaToPdf (schema, obj=[], name, level=0) {
         { 
           colSpan: 3,
           stack:[
-            {
-              text:[ 
-                {text:name, style:['small', 'mono', 'blue']}, 
-                {text:`${typeOfArr==='object'?'[{':'['}`, style:['small', 'mono', (typeOfArr === 'object'?'blue':'lightGray') ]},
-              ],
-              margin:0
-            },
+            {text:`${name?name:'array '} ${typeOfArr==='object'?'[{':'['}` , style:['small', 'mono', (typeOfArr === 'object'?'blue':'lightGray')], margin:0}, 
             {text:(schema.description ? schema.description : ''), style:['small','lightGray'],margin:[0,2,0,0]},
             {
               margin: [10, 0, 0, 0],
@@ -281,7 +269,7 @@ export function schemaToPdf (schema, obj=[], name, level=0) {
     }
     else{
       obj = [ 
-        {text:name, style:['small', 'mono', 'blue'],margin:0},
+        {text:name, style:['small', 'mono'],margin:0},
         {text:`[${typeOfArr}]`, style:['small','mono', 'lightGray'], margin:0},
         {text:(schema.description?schema.description:''), style:['small', 'lightGray'],margin:[0,2,0,0]}
       ];
@@ -303,37 +291,24 @@ export function schemaToPdf (schema, obj=[], name, level=0) {
       }
     });
     
-    // This is problematic
-    if (allOfRows.length >= 1){
-      obj = [{ 
-        colSpan: 3,
-        stack:[
-          {text:`${name?name:'root'} {`, style:['small', 'mono', 'blue']},
-          {text:(schema.description ? schema.description : ''), style:['sub', 'blue'],margin:[0,2,0,0]},
-          {
-            margin: [10, 0, 0, 0],
-            widths: [ 'auto', 'auto', '*' ],
-            layout: rowLinesOnlyTableLayout,
-            table: {
-              dontBreakRows: true,
-              body: allOfRows
-            }
-          },
-          {text:`}`, style:['small', 'mono', 'blue']}
-        ]
-      }]
-    }
-    
-    /*
-    obj = [ 
-      {text:name,style:['small', 'mono'],margin:0},
-      {text:(schema.type ? schema.type:''), style:['small', 'mono', 'lightGray'],margin:0},
-      {text:(schema.description?schema.description:''), style:['small', 'lightGray'], margin:[0,2,0,0]}
-    ];
-    */
-    
+    obj = [{ 
+      colSpan: 3,
+      stack:[
+        {text:`${name?name:'object'} {`, style:['small', 'mono', 'blue']},
+        {text:(schema.description ? schema.description : ''), style:['sub', 'blue'],margin:[0,2,0,0]},
+        {
+          margin: [10, 0, 0, 0],
+          widths: [ 'auto', 'auto', '*' ],
+          layout: rowLinesOnlyTableLayout,
+          table: {
+            dontBreakRows: true,
+            body: allOfRows
+          }
+        },
+        {text:`}`, style:['small', 'mono', 'blue']}
+      ]
+    }]
     
-
   }
 
   // Primitive Type (String, Integer, Boolean etc)
diff --git a/src/utils/pdf-gen-utils.js b/src/utils/pdf-gen-utils.js
index b667802..47015c4 100644
--- a/src/utils/pdf-gen-utils.js
+++ b/src/utils/pdf-gen-utils.js
@@ -26,7 +26,7 @@ export function getInfoDef(spec, bookTitle, localize){
           contactEmail,
           contactUrl,
           termsOfService
-        ]}
+        ]},
       ]
     }
 
@@ -47,13 +47,14 @@ export function getInfoDef(spec, bookTitle, localize){
       (spec.info.title ? {text:spec.info.title, style:['title', 'right']} : ''),
       (spec.info.version ? {text:`${localize.apiVersion}: ${spec.info.version}`, style:['p','b', 'right', 'alternate']} : ''),
       specInfDescrMarkDef,
-      ...contactDef
+      ...contactDef,
+      {text:'', pageBreak:'after'},
     ];
-
   }
   else{
     content = [
-      {text:bookTitle?bookTitle:apiVersion.apiReference, style:['h1', 'bold', 'primary','right', 'topMargin1']}
+      {text:bookTitle?bookTitle:apiVersion.apiReference, style:['h1', 'bold', 'primary','right', 'topMargin1']},
+      {text:'', pageBreak:'after'},
     ];
   }
   return content;
@@ -63,7 +64,7 @@ export function getInfoDef(spec, bookTitle, localize){
 export function getSecurityDef(spec, tableLayout, localize){
   let content =[];
   if (spec.securitySchemes){
-    content.push( {text:localize.securityAndAuthentication, style:['h3', 'b', 'primary','right', 'topMargin3'], pageBreak:'before'} );
+    content.push( {text:localize.securityAndAuthentication, style:['h3', 'b', 'primary','right', 'topMargin3']} );
     content.push({text:localize.securitySchemes, style:['b','tableMargin']});
     let tableContent = [
       [ {text: localize.type, style: ['small','b']}, {text: localize.description, style: ['small','b']} ]
@@ -81,15 +82,17 @@ export function getSecurityDef(spec, tableLayout, localize){
         body: tableContent,
       },
       layout: tableLayout,
-      style: 'tableMargin'
+      style: 'tableMargin',
+      pageBreak:'after'
     });
+    
   }
   return content;
 }
 
 // API details def
 export function getApiDef(spec, filterPath, sectionHeading, tableLayout, localize){
-  let content =[{text: sectionHeading, style:['h2','b'],pageBreak:'before'}];
+  let content =[{text: sectionHeading, style:['h2','b']}];
   let tagSeq=0;
 
   // Sort by Tag name (allready sorted)
@@ -212,14 +215,14 @@ export function getApiDef(spec, filterPath, sectionHeading, tableLayout, localiz
           tocMargin: [0, 10, 0, 0],
         },
         tagDescrMarkDef,
-        operationContent
+        operationContent,
+        {text:'', pageBreak:'after'}
       );
     }
     
   });
-
+  //content.push({text:'', pageBreak:'after'});
   return content;
-
 }
 
 
@@ -276,9 +279,9 @@ function getParameterTableDef(parameters, paramType, tableLayout, localize){
   }
   let tableContent = [
     [ 
-      {text: localize.name, style: ['sub','b','alternate']}, 
-      {text: localize.type, style: ['sub','b','alternate']},
-      {text: localize.description, style: ['sub','b','alternate']}
+      {text: localize.name, style: ['sub', 'b', 'alternate']}, 
+      {text: localize.type, style: ['sub', 'b', 'alternate']},
+      {text: localize.description, style: ['sub', 'b', 'alternate']}
     ]
   ];
   
@@ -292,9 +295,9 @@ function getParameterTableDef(parameters, paramType, tableLayout, localize){
         type = "array of " + param.items.type;
       }
       tableContent.push([
-        { text:paramName, style:['small','mono'] },
-        { text:type + format, style:['small','mono'] },
-        { text:param.description, style:['small'],margin:[0,2,0,0]},
+        { text:paramName, style:['small', 'mono'] },
+        { text:type + format, style:['small', 'mono' ] },
+        { text:param.description, style:['small'], margin:[0,2,0,0]},
       ]);  
     }
 
@@ -312,11 +315,11 @@ function getParameterTableDef(parameters, paramType, tableLayout, localize){
         },
         {
           stack:[
-            { text: `${paramSchema.type==='array' ? paramSchema.arrayType:paramSchema.type}${paramSchema.format ? `(${paramSchema.format})`:'' }`, style:['small','mono']},
+            { text: `${paramSchema.type==='array' ? paramSchema.arrayType:paramSchema.type}${paramSchema.format ? `(${paramSchema.format})`:'' }`, style:['small','mono', 'gray']},
             ( paramSchema.constrain ? { text: paramSchema.constrain, style:['small', 'gray']}:''),
             ( paramSchema.allowedValues ? { text:[
                 {text: localize.allowed+': ', style:['b','small']},
-                {text: paramSchema.allowedValues, style:['small', 'gray']}
+                {text: paramSchema.allowedValues, style:['small']}
               ]} : ''
             ),
             ( paramSchema.pattern ? { text: `${localize.pattern}: ${paramSchema.pattern}`, style:['small','gray']}:''),
@@ -355,7 +358,7 @@ function getResponseDef(responses, tableLayout, localize){
         origSchema = JSON.parse(JSON.stringify(origSchema, removeCircularReferences()));
         reponseModelTableDef = schemaToPdf(origSchema);
         if (reponseModelTableDef && reponseModelTableDef[0] && reponseModelTableDef[0].stack){
-          reponseModelTableDef[0].colSpan=undefined;
+          reponseModelTableDef[0].colSpan = undefined;
           reponseModelTableDef = {
             margin:[0,5,0,0],
             //layout:tableLayout,
@@ -381,16 +384,19 @@ function getResponseDef(responses, tableLayout, localize){
       margin:[0,10,0,0]
     });
 
-    allResponseModelTabelDefs.map(function(respModelTableDef){
-      respDef.push(respModelTableDef);
-    })
+    if (responses[statusCode].content){
+      allResponseModelTabelDefs.map(function(respModelTableDef){
+        respDef.push(respModelTableDef);
+      })
+    }
   }
+  
   return respDef;
 }
 
 //API List Def
 export function getApiListDef(spec, sectionHeading, tableLayout, localize) {
-  let content =[{text: sectionHeading, style:['h3','b'],pageBreak:'before'}];
+  let content =[{text: sectionHeading, style:['h3','b'],pageBreak:'none'}];
   spec.tags.map(function(tag, i){
     let tableContent = [
       [ {text: localize.method, style: ['small','b']}, {text: localize.api, style: ['small','b']}]
@@ -411,7 +417,7 @@ export function getApiListDef(spec, sectionHeading, tableLayout, localize) {
     });
 
     content.push(
-      {text: tag.name, style:['h6','b','primary','tableMargin'], pageBreak: i === 0 ? 'none' : 'before'},
+      {text: tag.name, style:['h6','b','primary','tableMargin'], pageBreak: i === 0 ? 'none' : 'after'},
       {text: tag.description, style:['p']},
       {
         table: {
diff --git a/src/utils/pdf-gen.js b/src/utils/pdf-gen.js
index 8d9b9d0..21df767 100644
--- a/src/utils/pdf-gen.js
+++ b/src/utils/pdf-gen.js
@@ -73,10 +73,11 @@ export default async function createPdf(specUrl, options){
       toc: {
         title: {text: options.localize.index, style:['b', 'h2']},
         numberStyle: {bold: true},
-        style:['small'],
-      }
+        style:['small']
+      },
+      pageBreak:'after'
     };
-    allContent.push({text:'', pageBreak:'after'});
+    // allContent.push({text:'', pageBreak:'after'});
     allContent.push(tocDef);
   }
   if (options.includeSecurity){