View Single Post
Old 11-15-2014, 02:18 PM   #3497
JimmXinu
Plugin Developer
JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.
 
JimmXinu's Avatar
 
Posts: 7,043
Karma: 4604637
Join Date: Dec 2011
Location: Midwest USA
Device: Kobo Clara Colour running KOReader
Test Version

Quote:
Originally Posted by JimmXinu View Post
I'm seeing that problem, too. But it's only with Chapter 27 of that story.

So I suspect there's something wrong in the HTML of that chapter. I will investigate further, but there are a couple other issues to look at this morning, too.
The problem is being caused by some incorrect tag nesting in the chapter 27 text:
Code:
<em><br><center>You know this has to be
<br>we always were so free
<br>we promised we'd be
<p>
<strong>...perfect...</em></strong></center>
(This author (or their HTML editor) apparently also likes using <dd> tags. A lot. That's the left ident in most of this story.)

The HTML parser we use is somewhat forgiving of incorrect nesting, but that example takes it too far for the current parser configuration.

I can get it to parse that chapter by telling it to accept <p> tags as self-closing, but that change will effect all future chapters downloaded from fanfiction.net.

I don't think that will cause problems, but I can't promise it won't either. So this is only going into test versions for a while.

I'm sure cryzed is willing to remind me that there are newer parsers available, but they are not easy to include in all the different forms FFDL takes. I may take another look at them, but I'm more concerned right now about calibre 2.10 breaking keyboard shortcuts in FFDL.

Last edited by JimmXinu; 11-15-2014 at 03:12 PM. Reason: Remove obsolete test versions - replaced by newer test or released version.
JimmXinu is offline