onlyhavecans/ReadItLater-Calibre-Plugin

Mark as read functionality broken

Closed this issue · 4 comments

With the newest version setting mark_as_read_after_dl = True causes a crash.

The following fixes the crash but doesn't achieve the desired result:

Change

def cleanup(self):
    if self.mark_as_read_after_dl:
        self.mark_as_read([x[1]['item_id'] for x in self.articles])
    else:
        pass

To

def cleanup(self):
    if self.mark_as_read_after_dl:
        self.mark_as_read([x['item_id'] for x in self.articles])
    else:
        pass

Yipes! Sorry about this. Can you please post the details of the crash?

The crash is just on the lookup x[1]. If that's fixed then it does not crash but fails to mark articles as read.

I'll see if I can't figure out what's going wrong... Manually marking the articles as read isn't much trouble.

I found the fix, will push to github when I get home

I found the fix, will push to github soon