From e1a8bd3bfaa6a78df244017b0362808967cd8400 Mon Sep 17 00:00:00 2001 From: Andreas Kristiansen <6977712+AndreasK79@users.noreply.github.com> Date: Fri, 21 Nov 2025 11:16:32 +0100 Subject: [PATCH] Check for selected lines first --- assets/js/sections/logbookadvanced.js | 27 +++++++++++++++------------ 1 file changed, 15 insertions(+), 12 deletions(-) diff --git a/assets/js/sections/logbookadvanced.js b/assets/js/sections/logbookadvanced.js index fbf258bdb..5ba2b062d 100644 --- a/assets/js/sections/logbookadvanced.js +++ b/assets/js/sections/logbookadvanced.js @@ -1364,6 +1364,21 @@ $(document).ready(function () { // Fix State button handler $('#fixState').click(function (event) { + const id_list = getSelectedIds(); + + if (id_list.length === 0) { + BootstrapDialog.alert({ + title: lang_gen_advanced_logbook_info, + message: lang_gen_advanced_logbook_select_row_state, + type: BootstrapDialog.TYPE_INFO, + closable: false, + draggable: false, + callback: function (result) { + } + }); + return; + } + $.ajax({ url: base_url + 'index.php/logbookadvanced/stateDialog', type: 'post', @@ -1382,18 +1397,6 @@ $(document).ready(function () { action: function (dialogItself) { const id_list = getSelectedIds(); - // Check if any rows are selected - if (id_list.length === 0) { - BootstrapDialog.alert({ - title: lang_gen_advanced_logbook_info, - message: lang_gen_advanced_logbook_select_row_state, - type: BootstrapDialog.TYPE_INFO, - closable: false, - draggable: false - }); - return; - } - if (inStateFixing) { return; }