View Single Post
Old 07-30-2017, 06:23 AM   #2366
davidfor
Grand Sorcerer
davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.
 
Posts: 24,905
Karma: 47303824
Join Date: Jul 2011
Location: Sydney, Australia
Device: Kobo:Touch,Glo, AuraH2O, GloHD,AuraONE, ClaraHD, Libra H2O; tolinoepos
Quote:
Originally Posted by JimmXinu View Post
replace_br_with_p does add the comment and the div tag as a matched pair. These are added as markers so replace_br_with_p can avoid running on the same text again.

But adapter_literotica has code that removes the ending close div.

I don't really know anything about adapter_literotica's text processing, but it kind of looks like it already attempts to change <br>s to <p>s and therefore should never use replace_br_with_p. Or, conversely, that code should be removed from adapter_literotica. Or even that the adapter_literotica should only be applied when replace_br_with_p:false.

I don't use that site, so I'm fairly open to suggestions.
Sorry, was thinking replace_br_with_p was run on the output from getChapterText. I should have checked.

If I remember rightly, the br removal code in the adapter was my attempt to do what replace_br_with_p does. I forgot to remove it. And I think a div or two has been removed from the page. I'll fix it.
davidfor is offline   Reply With Quote