Fix bug on TOC to readingOrder that caused problems with repeated loa…

…ding of same HTML.
vivliostyle · Feb 27, 2019 · bd00aa9 · bd00aa9
1 parent c77d407
commit bd00aa9
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/adapt/epub.js b/src/adapt/epub.js
@@ -1049,8 +1049,9 @@ adapt.epub.OPFDoc.prototype.initWithWebPubManifest = function(manifestObj, doc)
                          ".toc a[href]," +
                          "#toc a[href]";
         Array.from(doc.querySelectorAll(selector)).forEach(anchorElem => {
-            const path = this.getPathFromURL(adapt.base.stripFragment(anchorElem.href));
-            const url = path ? encodeURI(path) : anchorElem.href;
+            const hrefNoFragment = adapt.base.stripFragment(anchorElem.href);
+            const path = this.getPathFromURL(hrefNoFragment);
+            const url = path !== null ? encodeURI(path) : hrefNoFragment;
             if (manifestObj["readingOrder"].indexOf(url) == -1) {
                 manifestObj["readingOrder"].push(url);
             }