View Single Post
Old 01-02-2012, 12:52 PM   #323
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,025
Karma: 4604635
Join Date: Dec 2011
Location: Midwest USA
Device: Kobo Clara Colour running KOReader
Quote:
Originally Posted by kiwidude View Post
@JimmXinu - I've uploaded a new version of the zip with the self parameter on the previous post.
@kiwidude - I appreciate your help and guidance. I'm redoing the internals of my plugin to (hopefully) be more robust and 'correct'. Which does delay me from implementing the features I wanted a Reading List API for.

But I took a break to test all the RL methods you've exposed for me. So far I've found everything works as I expected except:
  • remove_books_from_all_lists(book_ids) doesn't remove all books -- doesn't when called from Reading List either. Looks like a real bug.
  • move_books_to_list(...) - unless refresh_screen=False, it sets the library display to marked:reading_list_<fromlist> afterwards. That may be the intended behavior, but it surprised me.
  • edit_list(list_name) it's a corner case, but if called with a non-existent list name, it displays an empty list and errors on 'OK'. I won't object if you declare it undefined behavior.

Thanks again,
Jim
JimmXinu is offline   Reply With Quote