From bf8b8b71008b49d4ad41c37ec168f5b4656ca42a Mon Sep 17 00:00:00 2001
From: Andreas Kristiansen <6977712+AndreasK79@users.noreply.github.com>
Date: Sun, 21 Dec 2025 08:41:48 +0100
Subject: [PATCH] Limit callbook grid fixer to 150 per run, and add batch
 update

---
 application/models/Logbookadvanced_model.php  | 30 +++++++++++++------
 .../views/logbookadvanced/dbtoolsdialog.php   |  2 +-
 2 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/application/models/Logbookadvanced_model.php b/application/models/Logbookadvanced_model.php
index ed4d2011d..c7f0bce01 100644
--- a/application/models/Logbookadvanced_model.php
+++ b/application/models/Logbookadvanced_model.php
@@ -1776,29 +1776,41 @@ class Logbookadvanced_model extends CI_Model {
 		$result = $this->getMissingGridQsos();
 
 		$count = 0;
+		$batch_updates = [];
+
 		$this->db->trans_start();
+
 		if (count($result) > 0) {
+			if (!$this->load->is_loaded('callbook')) {
+				$this->load->library('callbook');
+			}
+
 			foreach ($result as $row) {
 				$callsign = $row->col_call;
-				if (!$this->load->is_loaded('callbook')) {
-					$this->load->library('callbook');
-				}
-
 				$callbook = $this->callbook->getCallbookData($callsign);
 
 				if (isset($callbook)) {
 					if (isset($callbook['error'])) {
 						log_message('error', "Error: " . $callbook['error']);
 					} else {
-						if ($callbook['gridsquare'] != '') {
-							$sql = "update " . $this->config->item('table_name') . " set COL_GRIDSQUARE = ? where COL_PRIMARY_KEY = ?";
-							$this->db->query($sql, array($callbook['gridsquare'], $row->col_primary_key));
-							$count++;
+						if (isset($callbook['gridsquare']) && $callbook['gridsquare'] != '') {
+							// Prepare data for batch update
+							$batch_updates[] = [
+								'COL_PRIMARY_KEY' => $row->col_primary_key,
+								'COL_GRIDSQUARE' => $callbook['gridsquare']
+							];
 						}
 					}
 				}
 			}
+
+			// Perform batch update if there are any updates
+			if (!empty($batch_updates)) {
+				$this->db->update_batch($this->config->item('table_name'), $batch_updates, 'COL_PRIMARY_KEY');
+				$count = count($batch_updates);
+			}
 		}
+
 		$this->db->trans_complete();
 
 		return $count;
@@ -1810,7 +1822,7 @@ class Logbookadvanced_model extends CI_Model {
 				WHERE station_profile.user_id = ?
 				AND (qsos.COL_GRIDSQUARE IS NULL OR qsos.COL_GRIDSQUARE = '')
 				AND (qsos.COL_VUCC_GRIDS IS NULL OR qsos.COL_VUCC_GRIDS = '')
-				ORDER BY COL_TIME_ON DESC limit 250";
+				ORDER BY COL_TIME_ON DESC limit 150";
 
 		$query = $this->db->query($sql, [$this->session->userdata('user_id')]);
 
diff --git a/application/views/logbookadvanced/dbtoolsdialog.php b/application/views/logbookadvanced/dbtoolsdialog.php
index fedec1139..c9a14318c 100644
--- a/application/views/logbookadvanced/dbtoolsdialog.php
+++ b/application/views/logbookadvanced/dbtoolsdialog.php
@@ -95,7 +95,7 @@
                     <div>
                         <h6 class="mb-1"><?= __("Lookup QSOs with missing grid in callbook") ?></h6>
                         <p class="mb-1 small text-muted"><?= __("Use callbook lookup to set gridsquare") ?></p>
-						<p class="mb-1 small alert-danger"><?= __("This is limited to 250 callsigns for each run!") ?></p>
+						<p class="mb-1 small alert-danger"><?= __("This is limited to 150 callsigns for each run!") ?></p>
                     </div>
                     <div class="d-flex nowrap">
 						<button type="button" class="btn btn-sm btn-success me-1 ld-ext-right" id="checkGridsBtn" onclick="checkGrids()">