Fix ID assertion in section base CFI

johnfactotum · johnfactotum · commit 32afb79fd1fd · 2021-12-27T14:52:07.000+08:00
XML ID assertions should contain the `id` attribute of the element itself. Fixes #1236
diff --git a/src/packaging.js b/src/packaging.js
@@ -167,6 +167,7 @@ class Packaging {
 			// var manifestPropArray = manifestProps.length ? manifestProps.split(" ") : [];
 
 			var itemref = {
+				"id" : item.getAttribute("id"),
 				"idref" : idref,
 				"linear" : item.getAttribute("linear") || "yes",
 				"properties" : propArray,
diff --git a/src/spine.js b/src/spine.js
@@ -51,7 +51,7 @@ class Spine {
 			var spineItem;
 
 			item.index = index;
-			item.cfiBase = this.epubcfi.generateChapterComponent(this.spineNodeIndex, item.index, item.idref);
+			item.cfiBase = this.epubcfi.generateChapterComponent(this.spineNodeIndex, item.index, item.id);
 
 			if (item.href) {
 				item.url = resolver(item.href, true);
