wallabag plugin shouldn't delete articles by default

[anarcat]

• KOReader version: 2022.03
• Device: Kobo Aura

Issue

By default, a well-configured Wallabag plugin will delete remote article when the reader has marked it as "finished". I think that's counter intuitive and dangerous: the file and the remote article will potentially then be deleted and lost forever.

Instead, the default should be to mark the remote article as read and, maybe, delete locally, but do not remotely delete by default unless explicitly opted in b y the user.

[Frenzie]

At a quick glance, presumably it would suffice to change this line.

koreader/plugins/wallabag.koplugin/main.lua

Line 50 in eeefc65

self.is_delete_finished = true

[hrdl-github]

Or this one, to mark it as read on the server:

koreader/plugins/wallabag.koplugin/main.lua

Line 54 in 4fe39d6

self.is_archiving_deleted = false