Quote:
Originally Posted by JimmXinu
That's an issue with the new Marking features in Calibre. From #833:
|
Sorry -- I forgot about this.
I *think* it can happen only if
- There are no marked books.
- A set of book IDs are marked, where at least one of the IDs is for a deleted or non-existent book.
This sequence avoids the check for deleted books (book_id is None).
Note that if there are other marks then the check is done. The list of marked books must be empty before the problematic book is added.
There are some other errors in my changed code that could cause problems. The biggest is one place where I confused the book_id with the booklist row number. Sigh.
EDIT for explanation: many years ago, V0 .. V2 I think, calibre used the row number in the GUI for the book id, changing the 'real' id into the row number when needed. This changed with Kovid's new database code in V3 (I think), where the real book id became the one-true-ID. However, there is still a pile of legacy code in the GUI that uses the 'old' system. I confused myself when I looked at the API and chose the wrong method. As I said, sigh...