From 905bcbe372c44905d4ef302a18c98b7f305915ca Mon Sep 17 00:00:00 2001
From: Stephen Cameron <stephen@cubedesigners.com>
Date: Wed, 19 Dec 2018 10:59:19 +0100
Subject: [PATCH] Suppress bookmarks option in print/download modal when
 bookmarks are disabled. WIP #1806 @0.25

---
 js/libs/fluidbook/fluidbook.print.js | 26 ++++++++++++++------------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/js/libs/fluidbook/fluidbook.print.js b/js/libs/fluidbook/fluidbook.print.js
index e81d3f0b..2bbed8ee 100644
--- a/js/libs/fluidbook/fluidbook.print.js
+++ b/js/libs/fluidbook/fluidbook.print.js
@@ -144,18 +144,20 @@ FluidbookPrint.prototype = {
         view += '</div>'; // .print-option
 
         // Bookmarks
-        var hasBookmarks = this.fluidbook.bookmarks.hasBookmarkedPages(),
-        bookmarksDisabled = hasBookmarks ? '' : 'disabled';
-
-        view += '<div class="print-option bookmarks-option '+ bookmarksDisabled +'">';
-        view += '<label for="bookmarkedPages">';
-        view += '<div class="doubleThumb bookmarks">';
-        view += this.fluidbook.bookmarks.getPrintPreview();
-        view += '</div>'; // .doubleThumb
-        view += '</label>';
-        view += '<input type="radio" name="pageChoice" value="bookmarks" id="bookmarkedPages" '+ bookmarksDisabled +'>'
-        view += '<label for="bookmarkedPages" class="print-label-text">' + __('bookmarks') + '</label>';
-        view += '</div>'; // .print-option
+        if (this.fluidbook.datas.bookmark) {
+          var hasBookmarks = this.fluidbook.bookmarks.hasBookmarkedPages(),
+          bookmarksDisabled = hasBookmarks ? '' : 'disabled';
+
+          view += '<div class="print-option bookmarks-option '+ bookmarksDisabled +'">';
+          view += '<label for="bookmarkedPages">';
+          view += '<div class="doubleThumb bookmarks">';
+          view += this.fluidbook.bookmarks.getPrintPreview();
+          view += '</div>'; // .doubleThumb
+          view += '</label>';
+          view += '<input type="radio" name="pageChoice" value="bookmarks" id="bookmarkedPages" '+ bookmarksDisabled +'>'
+          view += '<label for="bookmarkedPages" class="print-label-text">' + __('bookmarks') + '</label>';
+          view += '</div>'; // .print-option
+        }
 
         view += '<div class="print-option"></div>'; // Extra empty div to ensure flexbox space-between works as expected
 
-- 
2.39.5