View Single Post
Old 12-03-2019, 09:51 AM   #243
KevinH
Sigil Developer
KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.
 
Posts: 8,849
Karma: 6120478
Join Date: Nov 2009
Device: many
Yes - 100% of what DiapDealer said above!

Okay, this was in fact a bug that is hit only when the spine was missing the toc attribute.

I had to literally rewrite the LocateOrCreateNCX routine in ImportEPUB.cpp because it was very broken and had devolved into a mass of conflicting spaghetti "if clauses" with no sanity at all that I just could not follow.

I have now pushed that rewrite to master.

Your test case now works properly on my machine.

Please test this with any other broken ncx related epubs you might have in your collection.

KevinH

Quote:
Originally Posted by snarkophilus View Post
Sorry, I meant TOC entry not bookmark.

Clicking on any entry in the TOC pane gets the error popup about a filename starting with "OEBPS/OEBPS/..." does not exist.
KevinH is offline   Reply With Quote