search

szarroug3 / X-Ray_Calibre_Plugin

X-Ray Creator plugin for Calibre
http://www.mobileread.com/forums/showthread.php?t=273189
GNU General Public License v3.0
57 stars 12 forks source link

Fixes #5 - bug that completely broke book parsing. We were ending up… #6

Closed stoduk closed 8 years ago

stoduk commented 8 years ago

… with every character having an alias of "", and so the handling in BookParser.init was setting self._aliases[""] = term.lower() - overwriting it each time. The last character to be processed would remain being the character who had an alias of "", and so we get a huge number of matches for this character and none for any other.

szarroug: this fixes the problem I had been seeing, where I could never end up with a properly parsed book. I suspect this was only broken in github version (perhaps 2.2), hence why most weren't seeing it.

stoduk commented 8 years ago

Muppeted, see #8. If you are going to break things, at least realise and fix it quick :)