Disable status buttons for deleted files in history

reviewable/pr10074/r12^2
Melik 1 year ago committed by Frans de Jonge
parent 77b110a987
commit 628cacf1e6

@ -373,7 +373,7 @@ function FileManager:setupLayout()
table.insert(buttons, { table.insert(buttons, {
{ {
text = _("Reset settings"), text = _("Reset settings"),
enabled = is_file and DocSettings:hasSidecarFile(BaseUtil.realpath(file)), enabled = DocSettings:hasSidecarFile(BaseUtil.realpath(file)),
callback = function() callback = function()
UIManager:show(ConfirmBox:new{ UIManager:show(ConfirmBox:new{
text = T(_("Reset settings for this document?\n\n%1\n\nAny highlights or bookmarks will be permanently lost."), BD.filepath(file)), text = T(_("Reset settings for this document?\n\n%1\n\nAny highlights or bookmarks will be permanently lost."), BD.filepath(file)),

@ -89,12 +89,13 @@ function FileManagerHistory:onMenuHold(item)
local readerui_instance = require("apps/reader/readerui"):_getRunningInstance() local readerui_instance = require("apps/reader/readerui"):_getRunningInstance()
local currently_opened_file = readerui_instance and readerui_instance.document and readerui_instance.document.file local currently_opened_file = readerui_instance and readerui_instance.document and readerui_instance.document.file
self.histfile_dialog = nil self.histfile_dialog = nil
local is_file = lfs.attributes(item.file, "mode") == "file"
local status = filemanagerutil.getStatus(item.file) local status = filemanagerutil.getStatus(item.file)
local buttons = { local buttons = {
{ {
{ {
text = _("Mark as reading"), text = _("Mark as reading"),
enabled = status ~= "reading", enabled = is_file and status ~= "reading",
callback = function() callback = function()
filemanagerutil.setStatus(item.file, "reading") filemanagerutil.setStatus(item.file, "reading")
if self._manager.filter ~= "all" then if self._manager.filter ~= "all" then
@ -108,7 +109,7 @@ function FileManagerHistory:onMenuHold(item)
}, },
{ {
text = _("Put on hold"), text = _("Put on hold"),
enabled = status ~= "abandoned", enabled = is_file and status ~= "abandoned",
callback = function() callback = function()
filemanagerutil.setStatus(item.file, "abandoned") filemanagerutil.setStatus(item.file, "abandoned")
if self._manager.filter ~= "all" then if self._manager.filter ~= "all" then
@ -122,7 +123,7 @@ function FileManagerHistory:onMenuHold(item)
}, },
{ {
text = _("Mark as read"), text = _("Mark as read"),
enabled = status ~= "complete", enabled = is_file and status ~= "complete",
callback = function() callback = function()
filemanagerutil.setStatus(item.file, "complete") filemanagerutil.setStatus(item.file, "complete")
if self._manager.filter ~= "all" then if self._manager.filter ~= "all" then

Loading…
Cancel
Save