feat: page-list nav does not have to match reading order

rdeltour · rdeltour · commit d0f12a983c28 · 2022-01-24T04:03:40.000+01:00
This commit: - removes the call to check the page list reading order - keeps the checking code, as well as NAV-011 - update tests Fix #1237
diff --git a/src/main/java/com/adobe/epubcheck/opf/XRefChecker.java b/src/main/java/com/adobe/epubcheck/opf/XRefChecker.java
@@ -310,7 +310,6 @@ public void checkReferences()
       }
     }
     checkReadingOrder(tocLinks, -1, -1);
-    checkReadingOrder(pageListLinks, -1, -1);
     checkReadingOrder(overlayLinks, -1, -1);
   }
 
diff --git a/src/test/resources/epub3/navigation-publication.feature b/src/test/resources/epub3/navigation-publication.feature
@@ -71,14 +71,12 @@ Feature: EPUB 3 ▸ Navigation Document ▸ Full Publication Checks
     When checking EPUB 'nav-page-list-reading-order-valid'
     Then no errors or warnings are reported
 
-  Scenario: Report a `page-list nav` whose links do not match the spine order 
+  Scenario: Verify a `page-list nav` whose links do not match the spine order 
     When checking EPUB 'nav-page-list-unordered-spine-warning'
-    Then warning NAV-011 is reported
     And no other errors or warnings are reported
 
-  Scenario: Report a `page-list nav` whose links do match the document order
+  Scenario: Verify a `page-list nav` whose links do match the document order
     When checking EPUB 'nav-page-list-unordered-fragments-warning'
-    Then warning NAV-011 is reported
     And no other errors or warnings are reported
 
 

Original file line number	Diff line number	Diff line change
`@@ -310,7 +310,6 @@ public void checkReferences()`
`310`	`310`	`}`
`311`	`311`	`}`
`312`	`312`	`checkReadingOrder(tocLinks, -1, -1);`
`313`		`- checkReadingOrder(pageListLinks, -1, -1);`
`314`	`313`	`checkReadingOrder(overlayLinks, -1, -1);`
`315`	`314`	`}`
`316`	`315`