Fixed bug in page extract / delete common code

Nathan McMinn · Nathan McMinn · commit 6473e2103141 · 2015-11-05T16:06:12.000-06:00
diff --git a/pdf-toolkit-repo/src/main/java/org/alfresco/extension/pdftoolkit/service/PDFToolkitServiceImpl.java b/pdf-toolkit-repo/src/main/java/org/alfresco/extension/pdftoolkit/service/PDFToolkitServiceImpl.java
@@ -1260,9 +1260,14 @@ private NodeRef subsetPDFDocument(NodeRef targetNodeRef, Map<String, Serializabl
 	        
 	        for (int pageNum = 1; pageNum <= pdfReader.getNumberOfPages(); pageNum++) 
 	        {
-	        	if (pagelist.contains(pageNum) && !delete) {
+	        	if (pagelist.contains(pageNum) && !delete) 
+	        	{
 	        		copy.addPage(copy.getImportedPage(pdfReader, pageNum));
 	        	}
+	        	else if (!pagelist.contains(pageNum) && delete)
+	        	{
+	        		copy.addPage(copy.getImportedPage(pdfReader,  pageNum));
+	        	}
 	        }
 	        doc.close();
 	

Original file line number	Diff line number	Diff line change
`@@ -1260,9 +1260,14 @@ private NodeRef subsetPDFDocument(NodeRef targetNodeRef, Map<String, Serializabl`
`1260`	`1260`
`1261`	`1261`	`for (int pageNum = 1; pageNum <= pdfReader.getNumberOfPages(); pageNum++)`
`1262`	`1262`	`{`
`1263`		`- if (pagelist.contains(pageNum) && !delete) {`
	`1263`	`+ if (pagelist.contains(pageNum) && !delete)`
	`1264`	`+ {`
`1264`	`1265`	`copy.addPage(copy.getImportedPage(pdfReader, pageNum));`
`1265`	`1266`	`}`
	`1267`	`+ else if (!pagelist.contains(pageNum) && delete)`
	`1268`	`+ {`
	`1269`	`+ copy.addPage(copy.getImportedPage(pdfReader, pageNum));`
	`1270`	`+ }`
`1266`	`1271`	`}`
`1267`	`1272`	`doc.close();`
`1268`	`1273`