Do not include empty strings in list of bookmarks

pull/292/head
Jonathan Rehm 7 years ago
parent 374b5f4c6e
commit a4f81dd812

@ -8,7 +8,7 @@
var reader = ePubReader(calibre.bookUrl, { var reader = ePubReader(calibre.bookUrl, {
restore: true, restore: true,
bookmarks: [calibre.bookmark] bookmarks: calibre.bookmark ? [calibre.bookmark] : []
}); });
reader.on("reader:bookmarked", updateBookmark.bind(reader, "add")); reader.on("reader:bookmarked", updateBookmark.bind(reader, "add"));
reader.on("reader:unbookmarked", updateBookmark.bind(reader, "remove")); reader.on("reader:unbookmarked", updateBookmark.bind(reader, "remove"));
@ -21,7 +21,7 @@
// Remove other bookmarks (there can only be one) // Remove other bookmarks (there can only be one)
if (action === "add") { if (action === "add") {
this.settings.bookmarks.filter(function (bookmark) { this.settings.bookmarks.filter(function (bookmark) {
return bookmark !== location; return bookmark && bookmark !== location;
}).map(function (bookmark) { }).map(function (bookmark) {
this.removeBookmark(bookmark); this.removeBookmark(bookmark);
}.bind(this)); }.bind(this));

Loading…
Cancel
Save