close #398 ExtractByOutline can produce output documents without any …

…pages
torakiki · Apr 1, 2021 · f1fdc5a · f1fdc5a
1 parent 16d44d0
commit f1fdc5a
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/sejda-sambox/src/main/java/org/sejda/impl/sambox/component/SamboxOutlineLevelsHandler.java b/sejda-sambox/src/main/java/org/sejda/impl/sambox/component/SamboxOutlineLevelsHandler.java
@@ -87,16 +87,16 @@ public OutlineExtractPageDestinations getExtractPageDestinations(int level, bool
                                 // Eg: a payslip document where each page has a bookmark (xyz destination) that points to the page, bookmark text is employee name
 
                                 endPage = includePageAfter ? after.page : after.page - 1;
+
+                                // no empty documents
+                                if(endPage < startPage) {
+                                    endPage = startPage;
+                                }
                                 // endPage = after.page;
                                 break;
                             }
                         }
 
-                        // handle first page quirk when includePageAfter is false
-                        if (startPage == 1 && endPage == 0) {
-                            endPage = 1;
-                        }
-
                         destinations.add(startPage, title, endPage);
                     }
                 }