View Single Post
Old 06-10-2014, 06:52 PM   #689
Rev. Bob
Wizard
Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.Rev. Bob ought to be getting tired of karma fortunes by now.
 
Rev. Bob's Avatar
 
Posts: 1,760
Karma: 9918418
Join Date: Feb 2013
Location: Here on the perimeter, there are no stars
Device: Kobo H2O, iPad mini 3, Kindle Touch
Quote:
Originally Posted by JSWolf View Post
But, I don't want the official one fixed without yours also being fixed. And yes, I did PM the author.
Okay, I took a look and I think I see the problem. Please try the attached two-routine version; the only change is a one-line attempted fix to the NCX routine. If that works, I'll recommend including it as an official patch.

The plugin generally uses a variable called "Dirtied" to indicate whether changes need to be written, but in this case it uses a dual test. My fix is to add a line at the end of that dual test to force "Dirtied" to be set to true, which I think will fix everything elegantly.

EDIT: The original fix didn't work, apparently because the error was coded in two places and I only found it in one. I've removed the first attempt from here and will post a new message with the new attempt.

Last edited by Rev. Bob; 06-10-2014 at 08:04 PM.
Rev. Bob is offline   Reply With Quote