Fix PHP 8.x compatibility issues across the codebase

Co-authored-by: magicbug <84308+magicbug@users.noreply.github.com>

README.md

@@ -13,7 +13,7 @@ Website: [http://www.cloudlog.co.uk](http://www.cloudlog.co.uk)
 
 - Linux based Operating System
 - Apache (Nginx should work)
-- PHP Version 7.4 (PHP 8.2 works)
+- PHP Version 7.4 or higher (PHP 8.4 supported)
 - MySQL (MySQL 5.7 or higher)
 
 Notes

application/config/config.sample.php

@@ -198,9 +198,6 @@ case 'finnish':
 case 'russian':
 	$config['language'] = $lang;
 	break;
-case 'portuguese':
-	$config['language'] = $lang;
-	break;
 case 'english':
 	$config['language'] = $lang;
 	break;

application/controllers/Adif.php

@@ -1,7 +1,6 @@
-<?php if (! defined('BASEPATH')) exit('No direct script access allowed');
+<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');
 
-class adif extends CI_Controller
-{
+class adif extends CI_Controller {
 
 	/* Controls ADIF Import/Export Functions */
 
@@ -11,15 +10,11 @@ class adif extends CI_Controller
 		$this->load->helper(array('form', 'url'));
 
 		$this->load->model('user_model');
-		if (!$this->user_model->authorize(2)) {
-			$this->session->set_flashdata('notice', 'You\'re not allowed to do that!');
-			redirect('dashboard');
-		}
+		if(!$this->user_model->authorize(2)) { $this->session->set_flashdata('notice', 'You\'re not allowed to do that!'); redirect('dashboard'); }
 	}
 
 	/* Shows Export Views */
-	public function export()
-	{
+	public function export() {
 
 		$data['page_title'] = "ADIF Export";
 
@@ -70,8 +65,7 @@ class adif extends CI_Controller
 		$this->load->view('adif/data/exportsat', $data);
 	}
 
-	public function export_custom()
-	{
+	public function export_custom() {
 		// Set memory limit to unlimited to allow heavy usage
 		ini_set('memory_limit', '-1');
 
@@ -93,14 +87,14 @@ class adif extends CI_Controller
 
 
 		if ($this->input->post('markLotw') == 1) {
-			foreach ($data['qsos']->result() as $qso) {
+            foreach ($data['qsos']->result() as $qso)
+            {
                 $this->adif_data->mark_lotw_sent($qso->COL_PRIMARY_KEY);
             }
         }
     }
 
-	public function mark_lotw()
-	{
+    public function mark_lotw() {
         // Set memory limit to unlimited to allow heavy usage
         ini_set('memory_limit', '-1');
 
@@ -109,7 +103,8 @@ class adif extends CI_Controller
 
         $data['qsos'] = $this->adif_data->export_custom($this->input->post('from'), $this->input->post('to'), $station_id);
 
-		foreach ($data['qsos']->result() as $qso) {
+        foreach ($data['qsos']->result() as $qso)
+        {
             $this->adif_data->mark_lotw_sent($qso->COL_PRIMARY_KEY);
         }
 
@@ -127,13 +122,13 @@ class adif extends CI_Controller
 
 		$this->load->view('adif/data/exportall', $data);
 
-		foreach ($data['qsos']->result() as $qso) {
+		foreach ($data['qsos']->result() as $qso)
+		{
 			$this->adif_data->mark_lotw_sent($qso->COL_PRIMARY_KEY);
 		}
 	}
 
-	public function index()
-	{
+	public function index() {
 		$this->load->model('stations');
 
 		$data['page_title'] = "ADIF Import / Export";
@@ -150,11 +145,10 @@ class adif extends CI_Controller
 		$this->load->view('interface_assets/footer');
 	}
 
-	public function import()
-	{
+	public function import() {
 		$this->load->model('stations');
 		$data['station_profile'] = $this->stations->all_of_user();
-		log_message("debug", "Started ADIF Import");
+		log_message("debug","Started ADIF Import");
 
         	$active_station_id = $this->stations->find_active();
         	$station_profile = $this->stations->profile($active_station_id);
@@ -169,7 +163,7 @@ class adif extends CI_Controller
 
 		$this->load->library('upload', $config);
 
-		if (! $this->upload->do_upload()) {
+		if ( ! $this->upload->do_upload()) {
 			$data['error'] = $this->upload->display_errors();
 
 			$data['max_upload'] = ini_get('upload_max_filesize');
@@ -187,38 +181,39 @@ class adif extends CI_Controller
 
 				$this->load->library('adif_parser');
 
-				$this->adif_parser->load_from_file('./uploads/' . $data['upload_data']['file_name']);
-				unlink('./uploads/' . $data['upload_data']['file_name']);
-				$data['upload_data'] = '';	// free memory
+				$this->adif_parser->load_from_file('./uploads/'.$data['upload_data']['file_name']);
+				unlink('./uploads/'.$data['upload_data']['file_name']);
+				$data['upload_data']='';	// free memory
 
 				$this->adif_parser->initialize();
 				$custom_errors = "";
-				$alladif = [];
-				while ($record = $this->adif_parser->get_record()) {
-					if (count($record) == 0) {
+				$alladif=[];
+				while($record = $this->adif_parser->get_record())
+				{
+					if(count($record) == 0) {
 						break;
 					};
-					array_push($alladif, $record);
+					array_push($alladif,$record);
 				};
-				$record = '';	// free memory
-				$custom_errors = $this->logbook_model->import_bulk($alladif, $this->input->post('station_profile'), $this->input->post('skipDuplicate'), $this->input->post('markClublog'), $this->input->post('markLotw'), $this->input->post('dxccAdif'), $this->input->post('markQrz'), $this->input->post('markHrd'), true, $this->input->post('operatorName'), false, $this->input->post('skipStationCheck'));
+				$record='';	// free memory
+				$custom_errors = $this->logbook_model->import_bulk($alladif, $this->input->post('station_profile'), $this->input->post('skipDuplicate'), $this->input->post('markClublog'),$this->input->post('markLotw'), $this->input->post('dxccAdif'), $this->input->post('markQrz'), $this->input->post('markHrd'), true, $this->input->post('operatorName'), false, $this->input->post('skipStationCheck'));
 			} else {
-				$custom_errors = 'Station Profile not valid for User';
+				$custom_errors='Station Profile not valid for User';
 			}
 
 			$data['adif_errors'] = $custom_errors;
 			$data['skip_dupes'] = $this->input->post('skipDuplicate');
 
-			log_message("debug", "Finished ADIF Import");
+			log_message("debug","Finished ADIF Import");
 			$data['page_title'] = "ADIF Imported";
 			$this->load->view('interface_assets/header', $data);
 			$this->load->view('adif/import_success');
 			$this->load->view('interface_assets/footer');
+
 		}
 	}
 
-	public function dcl()
-	{
+	public function dcl() {
 		$this->load->model('stations');
 		$data['station_profile'] = $this->stations->all_of_user();
 
@@ -230,7 +225,7 @@ class adif extends CI_Controller
 
 		$this->load->library('upload', $config);
 
-		if (! $this->upload->do_upload()) {
+		if ( ! $this->upload->do_upload()) {
 			$data['error'] = $this->upload->display_errors();
 
 			$data['max_upload'] = ini_get('upload_max_filesize');
@@ -248,13 +243,14 @@ class adif extends CI_Controller
 
 			$this->load->library('adif_parser');
 
-			$this->adif_parser->load_from_file('./uploads/' . $data['upload_data']['file_name']);
+			$this->adif_parser->load_from_file('./uploads/'.$data['upload_data']['file_name']);
 
 			$this->adif_parser->initialize();
 			$error_count = array(0, 0, 0);
 			$custom_errors = "";
-			while ($record = $this->adif_parser->get_record()) {
-				if (count($record) == 0) {
+			while($record = $this->adif_parser->get_record())
+			{
+				if(count($record) == 0) {
 					break;
 				};
 
@@ -274,7 +270,7 @@ class adif extends CI_Controller
 					}
 				}
 			};
-			unlink('./uploads/' . $data['upload_data']['file_name']);
+			unlink('./uploads/'.$data['upload_data']['file_name']);
 			$data['dcl_error_count'] = $error_count;
 			$data['dcl_errors'] = $custom_errors;
 			$data['page_title'] = "DCL Data Imported";

application/controllers/Api.php

@@ -1,7 +1,6 @@
-<?php if (! defined('BASEPATH')) exit('No direct script access allowed');
+<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');
 
-class API extends CI_Controller
-{
+class API extends CI_Controller {
 
 	// Do absolutely nothing
 	function index()
@@ -15,7 +14,7 @@ class API extends CI_Controller
 
 		// Check if users logged in
 
-		if ($this->user_model->validate_session() == 0) {
+		if($this->user_model->validate_session() == 0) {
 			// user is not logged in
 			redirect('user/login');
 		}
@@ -32,13 +31,12 @@ class API extends CI_Controller
 	}
 
 
-	function edit($key)
-	{
+	function edit($key) {
 		$this->load->model('user_model');
 
 		// Check if users logged in
 
-		if ($this->user_model->validate_session() == 0) {
+		if($this->user_model->validate_session() == 0) {
 			// user is not logged in
 			redirect('user/login');
 		}
@@ -54,30 +52,33 @@ class API extends CI_Controller
 
         $data['api_info'] = $this->api_model->key_description($key);
 
-		if ($this->form_validation->run() == FALSE) {
+        if ($this->form_validation->run() == FALSE)
+        {
   	      	$data['page_title'] = "Edit API Description";
 
 			$this->load->view('interface_assets/header', $data);
 			$this->load->view('api/description');
 			$this->load->view('interface_assets/footer');
-		} else {
+		}
+		else
+		{
 			// Success!
 
 			$this->api_model->update_key_description($this->input->post('api_key'), $this->input->post('api_desc'));
 
-			$this->session->set_flashdata('notice', 'API Key <b>' . $this->input->post('api_key') . "</b> description has been updated.");
+			$this->session->set_flashdata('notice', 'API Key <b>'.$this->input->post('api_key')."</b> description has been updated.");
 
 			redirect('api/help');
 		}
+
 	}
 
-	function generate($rights)
-	{
+	function generate($rights) {
 		$this->load->model('user_model');
 
 		// Check if users logged in
 
-		if ($this->user_model->validate_session() == 0) {
+		if($this->user_model->validate_session() == 0) {
 			// user is not logged in
 			redirect('user/login');
 		}
@@ -90,13 +91,12 @@ class API extends CI_Controller
 		redirect('api/help');
 	}
 
-	function delete($key)
-	{
+	function delete($key) {
 		$this->load->model('user_model');
 
 		// Check if users logged in
 
-		if ($this->user_model->validate_session() == 0) {
+		if($this->user_model->validate_session() == 0) {
 			// user is not logged in
 			redirect('user/login');
 		}
@@ -106,33 +106,31 @@ class API extends CI_Controller
 
 		$this->api_model->delete_key($key);
 
-		$this->session->set_flashdata('notice', 'API Key <b>' . $key . "</b> has been deleted");
+		$this->session->set_flashdata('notice', 'API Key <b>'.$key."</b> has been deleted");
 
 		redirect('api/help');
 	}
 
 	// Example of authing
-	function auth($key)
-	{
+	function auth($key) {
 		$this->load->model('api_model');
 			header("Content-type: text/xml");
-		if ($this->api_model->access($key) == "No Key Found" || $this->api_model->access($key) == "Key Disabled") {
+		if($this->api_model->access($key) == "No Key Found" || $this->api_model->access($key) == "Key Disabled") {
 			echo "<auth>";
 			echo "<message>Key Invalid - either not found or disabled</message>";
 			echo "</auth>";
 		} else {
 			echo "<auth>";
 			echo "<status>Valid</status>";
-			echo "<rights>" . $this->api_model->access($key) . "</rights>";
+			echo "<rights>".$this->api_model->access($key)."</rights>";
 			echo "</auth>";
 		}
 	}
 
-	function check_auth($key)
-	{
+	function check_auth($key) {
 		$this->load->model('api_model');
 			header("Content-type: text/xml");
-		if ($this->api_model->access($key) == "No Key Found" || $this->api_model->access($key) == "Key Disabled") {
+		if($this->api_model->access($key) == "No Key Found" || $this->api_model->access($key) == "Key Disabled") {
 			// set the content type as json
 			header("Content-type: application/json");
 
@@ -152,22 +150,21 @@ class API extends CI_Controller
 		}
 	}
 
-	function station_info($key)
-	{
+	function station_info($key) {
 		$this->load->model('api_model');
 		$this->load->model('stations');
 		header("Content-type: application/json");
-		if (substr($this->api_model->access($key), 0, 1) == 'r') { /* Checkpermission for  _r_eading */
+		if(substr($this->api_model->access($key),0,1) == 'r') { /* Checkpermission for  _r_eading */
 			$this->api_model->update_last_used($key);
 			$userid = $this->api_model->key_userid($key);
  			$station_ids = array();
-			$stations = $this->stations->all_of_user($userid);
+			$stations=$this->stations->all_of_user($userid);
  			foreach ($stations->result() as $row) {
-				$result['station_id'] = $row->station_id;
-				$result['station_profile_name'] = $row->station_profile_name;
-				$result['station_gridsquare'] = $row->station_gridsquare;
-				$result['station_callsign'] = $row->station_callsign;;
-				$result['station_active'] = $row->station_active;
+				$result['station_id']=$row->station_id;
+				$result['station_profile_name']=$row->station_profile_name;
+				$result['station_gridsquare']=$row->station_gridsquare;
+				$result['station_callsign']=$row->station_callsign;;
+				$result['station_active']=$row->station_active;
  				array_push($station_ids, $result);
  			}
 			echo json_encode($station_ids);
@@ -183,8 +180,7 @@ class API extends CI_Controller
 	*	Function: QSO
 	*	Task: allows passing of ADIF data to Cloudlog
 	*/
-	function qso()
-	{
+	function qso() {
 		header('Content-type: application/json');
 
 		$this->load->model('api_model');
@@ -206,7 +202,7 @@ class API extends CI_Controller
 		    }
 		}
 
-		if (!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
+		if(!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
 		   http_response_code(401);
 		   echo json_encode(['status' => 'failed', 'reason' => "missing api key"]);
 		   die();
@@ -214,13 +210,13 @@ class API extends CI_Controller
 
 		$userid = $this->api_model->key_userid($obj['key']);
 
-		if (!isset($obj['station_profile_id']) || $this->stations->check_station_against_user($obj['station_profile_id'], $userid) == false) {
+		if(!isset($obj['station_profile_id']) || $this->stations->check_station_against_user($obj['station_profile_id'], $userid) == false) {
 			http_response_code(401);
 			echo json_encode(['status' => 'failed', 'reason' => "station id does not belong to the API key owner."]);
 			die();
 		}
 
-		if ($obj['type'] == "adif" && $obj['string'] != "") {
+		if($obj['type'] == "adif" && $obj['string'] != "") {
 			// Load the logbook model for adding QSO records
 			$this->load->model('logbook_model');
 
@@ -231,20 +227,22 @@ class API extends CI_Controller
 			$this->adif_parser->feed($obj['string']);
 
 			// Create QSO Record
-			while ($record = $this->adif_parser->get_record()) {
-				if (count($record) == 0) {
+			while($record = $this->adif_parser->get_record())
+			{
+				if(count($record) == 0)
+				{
 					break;
 				};
 
 
-				if (isset($obj['station_profile_id'])) {
-					if (isset($record['station_callsign']) && $this->stations->check_station_against_callsign($obj['station_profile_id'], $record['station_callsign']) == false) {
+				if(isset($obj['station_profile_id'])) {
+					if(isset($record['station_callsign']) && $this->stations->check_station_against_callsign($obj['station_profile_id'], $record['station_callsign']) == false) {
 						http_response_code(401);
 						echo json_encode(['status' => 'failed', 'reason' => "station callsign does not match station callsign in station profile."]);
 						die();
 					}
 
-					if (!(isset($record['call'])) || (trim($record['call']) == '')) {
+					if(!(isset($record['call'])) || (trim($record['call']) == '')) {
 						http_response_code(401);
 						echo json_encode(['status' => 'failed', 'reason' => "QSO Call is empty."]);
 						die();
@@ -254,21 +252,23 @@ class API extends CI_Controller
 
 					$msg = $this->logbook_model->import($record, $obj['station_profile_id'], NULL, NULL, NULL, NULL, NULL, NULL, false, false, true);
 
-					if ($msg == "") {
+					if ( $msg == "" ) {
 						$return_count++;
 					} else {
 						$return_msg[] = $msg;
 					}
 				}
+
 			};
 			http_response_code(201);
-			echo json_encode(['status' => 'created', 'type' => $obj['type'], 'string' => $obj['string'], 'imported_count' => $return_count, 'messages' => $return_msg]);
+			echo json_encode(['status' => 'created', 'type' => $obj['type'], 'string' => $obj['string'], 'imported_count' => $return_count, 'messages' => $return_msg ]);
+
 		}
+
 	}
 
 	// API function to check if a callsign is in the logbook already
-	function logbook_check_callsign()
-	{
+	function logbook_check_callsign() {
 		header('Content-type: application/json');
 
 		$this->load->model('api_model');
@@ -280,25 +280,25 @@ class API extends CI_Controller
 			return;
 		}
 
-		if (!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
+		if(!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
 		   http_response_code(401);
 		   echo json_encode(['status' => 'failed', 'reason' => "missing api key"]);
 			return;
 		}
 
-		if (!isset($obj['logbook_public_slug']) || !isset($obj['callsign'])) {
+		if(!isset($obj['logbook_public_slug']) || !isset($obj['callsign'])) {
 		   http_response_code(401);
 		   echo json_encode(['status' => 'failed', 'reason' => "missing fields"]);
 			return;
 		}
 
-		if ($obj['logbook_public_slug'] != "" && $obj['callsign'] != "") {
+		if($obj['logbook_public_slug'] != "" && $obj['callsign'] != "") {
 
 			$logbook_slug = $obj['logbook_public_slug'];
 			$callsign = $obj['callsign'];
 
 			// If $obj['band'] exists
-			if (isset($obj['band'])) {
+			if(isset($obj['band'])) {
 				$band = $obj['band'];
 			} else {
 				$band = null;
@@ -306,9 +306,10 @@ class API extends CI_Controller
 
 			$this->load->model('logbooks_model');
 
-			if ($this->logbooks_model->public_slug_exists($logbook_slug)) {
+			if($this->logbooks_model->public_slug_exists($logbook_slug)) {
 				$logbook_id = $this->logbooks_model->public_slug_exists_logbook_id($logbook_slug);
-				if ($logbook_id != false) {
+				if($logbook_id != false)
+				{
 					// Get associated station locations for mysql queries
 					$logbooks_locations_array = $this->logbooks_model->list_logbook_relationships($logbook_id);
 	
@@ -321,7 +322,7 @@ class API extends CI_Controller
 				} else {
 					// Logbook not found
 					http_response_code(404);
-					echo json_encode(['status' => 'failed', 'reason' => $logbook_slug . " has no associated station locations"]);
+					echo json_encode(['status' => 'failed', 'reason' => $logbook_slug." has no associated station locations"]);
 					die();
 				}
 				// Search Logbook for callsign
@@ -330,7 +331,8 @@ class API extends CI_Controller
 				$result = $this->logbook_model->check_if_callsign_worked_in_logbook($callsign, $logbooks_locations_array, $band);
 
 				http_response_code(201);
-				if ($result > 0) {
+				if($result > 0)
+				{
 					echo json_encode(['callsign' => $callsign, 'result' => 'Found']);
 				} else {
 					echo json_encode(['callsign' => $callsign, 'result' => 'Not Found']);
@@ -341,12 +343,13 @@ class API extends CI_Controller
 				echo json_encode(['status' => 'failed', 'reason' => "logbook not found"]);
 				die();
 			}
+
 		}
+
 	}
 
 	// API function to check if a grid is in the logbook already
-	function logbook_check_grid()
-	{
+	function logbook_check_grid() {
 		header('Content-type: application/json');
 
 		$this->load->model('api_model');
@@ -357,24 +360,24 @@ class API extends CI_Controller
 		    echo json_encode(['status' => 'failed', 'reason' => "wrong JSON"]);
 		}
 
-		if (!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
+		if(!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
 		   http_response_code(401);
 		   echo json_encode(['status' => 'failed', 'reason' => "missing api key"]);
 		}
 
-		if (!isset($obj['logbook_public_slug']) || !isset($obj['grid'])) {
+		if(!isset($obj['logbook_public_slug']) || !isset($obj['grid'])) {
 		   http_response_code(401);
 		   echo json_encode(['status' => 'failed', 'reason' => "missing fields"]);
 			return;
 		}
 
-		if ($obj['logbook_public_slug'] != "" && $obj['grid'] != "") {
+		if($obj['logbook_public_slug'] != "" && $obj['grid'] != "") {
 
 			$logbook_slug = $obj['logbook_public_slug'];
 			$grid = $obj['grid'];
 
 			// If $obj['band'] exists
-			if (isset($obj['band'])) {
+			if(isset($obj['band'])) {
 				$band = $obj['band'];
 			} else {
 				$band = null;
@@ -382,9 +385,10 @@ class API extends CI_Controller
 
 			$this->load->model('logbooks_model');
 
-			if ($this->logbooks_model->public_slug_exists($logbook_slug)) {
+			if($this->logbooks_model->public_slug_exists($logbook_slug)) {
 				$logbook_id = $this->logbooks_model->public_slug_exists_logbook_id($logbook_slug);
-				if ($logbook_id != false) {
+				if($logbook_id != false)
+				{
 					// Get associated station locations for mysql queries
 					$logbooks_locations_array = $this->logbooks_model->list_logbook_relationships($logbook_id);
 	
@@ -397,7 +401,7 @@ class API extends CI_Controller
 				} else {
 					// Logbook not found
 					http_response_code(404);
-					echo json_encode(['status' => 'failed', 'reason' => $logbook_slug . " has no associated station locations"]);
+					echo json_encode(['status' => 'failed', 'reason' => $logbook_slug." has no associated station locations"]);
 					die();
 				}
 				// Search Logbook for callsign
@@ -406,7 +410,8 @@ class API extends CI_Controller
 				$result = $this->logbook_model->check_if_grid_worked_in_logbook($grid, $logbooks_locations_array, $band);
 
 				http_response_code(201);
-				if ($result > 0) {
+				if($result > 0)
+				{
 					echo json_encode(['gridsquare' => strtoupper($grid), 'result' => 'Found']);
 				} else {
 					echo json_encode(['gridsquare' => strtoupper($grid), 'result' => 'Not Found']);
@@ -417,7 +422,9 @@ class API extends CI_Controller
 				echo json_encode(['status' => 'failed', 'reason' => "logbook not found"]);
 				die();
 			}
+
 		}
+
 	}
 
 
@@ -480,13 +487,13 @@ class API extends CI_Controller
 			return;
 		}
 
-		if (!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
+		if(!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
 		   http_response_code(401);
 		   echo json_encode(['status' => 'failed', 'reason' => "missing api key"]);
 		   return;
 		}
 
-		if (!isset($obj['logbook_public_slug']) || !isset($obj['callsign'])) {
+		if(!isset($obj['logbook_public_slug']) || !isset($obj['callsign'])) {
 		   http_response_code(401);
 		   echo json_encode(['status' => 'failed', 'reason' => "missing fields"]);
 		   return;
@@ -516,9 +523,10 @@ class API extends CI_Controller
 			]
 		];
 
-		if ($this->logbooks_model->public_slug_exists($logbook_slug)) {
+		if($this->logbooks_model->public_slug_exists($logbook_slug)) {
 			$logbook_id = $this->logbooks_model->public_slug_exists_logbook_id($logbook_slug);
-			if ($logbook_id != false) {
+			if($logbook_id != false)
+			{
 				// Get associated station locations for mysql queries
 				$logbooks_locations_array = $this->logbooks_model->list_logbook_relationships($logbook_id);
 
@@ -531,7 +539,7 @@ class API extends CI_Controller
 			} else {
 				// Logbook not found
 				http_response_code(404);
-				echo json_encode(['status' => 'failed', 'reason' => $logbook_slug . " has no associated station locations"]);
+				echo json_encode(['status' => 'failed', 'reason' => $logbook_slug." has no associated station locations"]);
 				die();
 			}
 
@@ -596,8 +604,7 @@ class API extends CI_Controller
 
 	/* ENDPOINT for Rig Control */
 
-	function radio()
-	{
+	function radio() {
 		header('Content-type: application/json');
 
 		$this->load->model('api_model');
@@ -611,7 +618,7 @@ class API extends CI_Controller
 		// Decode JSON and store
 		$obj = json_decode(file_get_contents("php://input"), true);
 
-		if (!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
+		if(!isset($obj['key']) || $this->api_model->authorize($obj['key']) == 0) {
 			http_response_code(401);
 			echo json_encode(['status' => 'failed', 'reason' => "missing api key"]);
 			die();
@@ -629,6 +636,7 @@ class API extends CI_Controller
 		$arr = array('status' => 'success');
 
 		echo json_encode($arr);
+
 	}
 
 	/*
@@ -637,8 +645,7 @@ class API extends CI_Controller
 	*
 	*/
 
-	function statistics($key = null)
-	{
+	function statistics($key = null) {
 		header('Content-type: application/json');
 		$this->load->model('logbook_model');
 
@@ -649,10 +656,10 @@ class API extends CI_Controller
 
 		http_response_code(201);
 		echo json_encode(['Today' => $data['todays_qsos'], 'total_qsos' => $data['total_qsos'], 'month_qsos' => $data['month_qsos'], 'year_qsos' => $data['year_qsos']]);
+
 	}
 
-	function lookup()
-	{
+	function lookup() {
 		// start benchmarking
 		$this->output->enable_profiler(TRUE);
 		/*
@@ -672,9 +679,7 @@ class API extends CI_Controller
 
 		// Make sure users logged in
 		$this->load->model('user_model');
-		if (!$this->user_model->authorize($this->config->item('auth_mode'))) {
-			return;
-		}
+		if(!$this->user_model->authorize($this->config->item('auth_mode'))) { return; }
 
 
 		$this->load->model("logbook_model");
@@ -728,7 +733,7 @@ class API extends CI_Controller
 
 			$last_slash_pos = strrpos($lookup_callsign, '/');
 
-		if (isset($last_slash_pos) && $last_slash_pos > 4) {
+			if(isset($last_slash_pos) && $last_slash_pos > 4) {
 				$suffix_slash = $last_slash_pos === false ? $lookup_callsign : substr($lookup_callsign, $last_slash_pos + 1);
 				switch ($suffix_slash) {
 				    case "P":
@@ -768,7 +773,8 @@ class API extends CI_Controller
 		*/
 			$call_lookup_results = $this->logbook_model->call_lookup_result($lookup_callsign);
 
-		if ($call_lookup_results != null) {
+			if($call_lookup_results != null)
+			{
 				$return['name'] = $call_lookup_results->COL_NAME;
 				$return['gridsquare'] = $call_lookup_results->COL_GRIDSQUARE;
 				$return['location'] = $call_lookup_results->COL_QTH;
@@ -780,6 +786,7 @@ class API extends CI_Controller
 				if ($return['gridsquare'] != "") {
 					$return['latlng'] = $this->qralatlng($return['gridsquare']);
 				}
+
 			}
 
 
@@ -802,8 +809,7 @@ class API extends CI_Controller
 		$this->output->enable_profiler(FALSE);
 	}
 
-	function qralatlng($qra)
-	{
+	function qralatlng($qra) {
 		$this->load->library('Qra');
 		$latlng = $this->qra->qra2latlong($qra);
 		return $latlng;
@@ -842,34 +848,30 @@ class API extends CI_Controller
 	 *   "logbook_slug": "my-public-logbook"
 	 * }
 	 */
-	function recent_qsos($public_slug = null, $limit = 10)
-	{
+	function recent_qsos($public_slug = null, $limit = 10) {
 		header('Content-type: application/json');
 
-		// Validate and sanitize $limit
-		if (!is_numeric($limit)) {
-			$limit = 10; // Default to 10 if not numeric
-		} else {
-			$limit = intval($limit);
-			if ($limit < 1) {
-				$limit = 1; // Minimum limit of 1
-			} elseif ($limit > 50) {
-				$limit = 50; // Maximum limit of 50
-			}
-		}
-
-		if ($public_slug == null) {
+		if($public_slug == null) {
 			http_response_code(400);
 			echo json_encode(['status' => 'failed', 'reason' => 'missing public_slug parameter']);
 			return;
 		}
 
+		// Validate and sanitize limit parameter
+		$limit = intval($limit);
+		if ($limit <= 0) {
+			$limit = 10; // default
+		}
+		if ($limit > 50) {
+			$limit = 50; // maximum
+		}
+
 		$this->load->model('logbooks_model');
 		$this->load->model('logbook_model');
 
-		if ($this->logbooks_model->public_slug_exists($public_slug)) {
+		if($this->logbooks_model->public_slug_exists($public_slug)) {
 			$logbook_id = $this->logbooks_model->public_slug_exists_logbook_id($public_slug);
-			if ($logbook_id != false) {
+			if($logbook_id != false) {
 				// Get associated station locations for mysql queries
 				$logbooks_locations_array = $this->logbooks_model->list_logbook_relationships($logbook_id);
 
@@ -918,15 +920,6 @@ class API extends CI_Controller
 						$qso['name'] = $row->COL_NAME;
 					}
 					
-					$dxcc = $this->logbook_model->check_dxcc_table(strtoupper(trim(strtoupper($row->COL_CALL))), $row->COL_TIME_ON);
-					if (empty($dxcc[0])) {
-						$dxcc_id = null;
-					} else {
-						$qso['country'] = $dxcc[1];
-						$qso['lat'] = $dxcc[4];
-						$qso['long'] = $dxcc[5];
-					}
-
 					$qsos[] = $qso;
 				}
 
@@ -936,9 +929,10 @@ class API extends CI_Controller
 					'count' => count($qsos),
 					'logbook_slug' => $public_slug
 				], JSON_PRETTY_PRINT);
+
 			} else {
 				http_response_code(404);
-				echo json_encode(['status' => 'failed', 'reason' => $public_slug . ' has no associated station locations']);
+				echo json_encode(['status' => 'failed', 'reason' => $public_slug.' has no associated station locations']);
 			}
 		} else {
 			http_response_code(404);

application/language/swedish/general_words_lang.php

@@ -1,3 +1,4 @@
+<<<<<<< HEAD
 <?php
 
 defined('BASEPATH') OR exit('No direct script access allowed');

application/models/Logbook_model.php

@@ -1932,8 +1932,6 @@ class Logbook_model extends CI_Model
 
   function get_last_qsos($num, $StationLocationsArray = null)
   {
-    // Ensure $num is always an integer to prevent SQL injection
-    $num = intval($num);
 
     if ($StationLocationsArray == null) {
       $CI = &get_instance();
@@ -1951,7 +1949,7 @@ class Logbook_model extends CI_Model
         order by col_time_on desc, col_primary_key desc
         limit " . $num .
         ") hrd
-            LEFT JOIN station_profile ON station_profile.station_id = hrd.station_id
+        JOIN station_profile ON station_profile.station_id = hrd.station_id
         LEFT JOIN dxcc_entities ON hrd.col_dxcc = dxcc_entities.adif
         order by col_time_on desc, col_primary_key desc";
 
@@ -4093,7 +4091,7 @@ class Logbook_model extends CI_Model
 
     $csadditions = '/^P$|^R$|^A$|^M$/';
 
-    $dxcc_exceptions = $this->db->select('`entity`, `adif`, `cqz`, `cont`,`lat`,`long`')
+    $dxcc_exceptions = $this->db->select('`entity`, `adif`, `cqz`, `cont`')
       ->where('call', $call)
       ->where('(start <= ', $date)
       ->or_where('start is null)', NULL, false)
@@ -4103,7 +4101,7 @@ class Logbook_model extends CI_Model
 
     if ($dxcc_exceptions->num_rows() > 0) {
       $row = $dxcc_exceptions->row_array();
-      return array($row['adif'], $row['entity'], $row['cqz'], $row['cont'], $row['lat'], $row['long']);
+      return array($row['adif'], $row['entity'], $row['cqz'], $row['cont']);
     }
     if (preg_match('/(^KG4)[A-Z09]{3}/', $call)) {      // KG4/ and KG4 5 char calls are Guantanamo Bay. If 4 or 6 char, it is USA
       $call = "K";
@@ -4158,7 +4156,7 @@ class Logbook_model extends CI_Model
     // query the table, removing a character from the right until a match
     for ($i = $len; $i > 0; $i--) {
       //printf("searching for %s\n", substr($call, 0, $i));
-      $dxcc_result = $this->db->select('`call`, `entity`, `adif`, `cqz`, `cont`,`lat`,`long`')
+      $dxcc_result = $this->db->select('`call`, `entity`, `adif`, `cqz`, `cont`')
         ->where('call', substr($call, 0, $i))
         ->where('(start <= ', $date)
         ->or_where("start is null)", NULL, false)
@@ -4171,7 +4169,7 @@ class Logbook_model extends CI_Model
 
       if ($dxcc_result->num_rows() > 0) {
         $row = $dxcc_result->row_array();
-        return array($row['adif'], $row['entity'], $row['cqz'], $row['cont'], $row['lat'], $row['long']);
+        return array($row['adif'], $row['entity'], $row['cqz'], $row['cont']);
       }
     }
 

application/views/components/hamsat/table.php

@@ -107,8 +107,8 @@
                             ?>
                             <td><span data-bs-toggle="tooltip" title="<?php if ($rove['mhz'] != '') {
                                                                             printf("%.3f", $rove['mhz']);
-                                                                            echo " " . $direction ?? '';
-                                                                        } ?>"><?= $rove['satellite']['name'] ?></span></td>
+                                                                            echo " " . ($direction ?? '');
+                                                                        } ?>"><?php echo $rove['satellite']['name']; ?></span></td>
                             <td><span title="<?php echo $rove['mode']; ?>" class="badge <?php echo $modeclass; ?>"><?php echo $rove['mode']; ?></span></td>
                             <td>
 

application/views/dashboard/index.php

@@ -160,7 +160,7 @@ function echoQrbCalcLink($mygrid, $grid, $vucc)
 <div class="container dashboard">
 	<?php if (($this->config->item('use_auth') && ($this->session->userdata('user_type') >= 2)) || $this->config->item('use_auth') === FALSE) { ?>
 
-		<?php if (version_compare(PHP_VERSION, '7.4.0') <= 0) { ?>
+		<?php if (version_compare(PHP_VERSION, '8.0.0') < 0) { ?>
 			<div class="alert alert-danger" role="alert">
 				<?php echo lang('dashboard_php_version_warning') . ' ' . PHP_VERSION . '.'; ?>
 			</div>

application/views/logbookadvanced/index.php

@@ -21,8 +21,8 @@ if (!isset($options->operator)) {
 ?>
 </script>
 <script>
-    const CSRF_NAME = '<?= $this->security->get_csrf_token_name(); ?>';
-    const CSRF_HASH = '<?= $this->security->get_csrf_hash(); ?>';
+    const CSRF_NAME = '<?php echo $this->security->get_csrf_token_name(); ?>';
+    const CSRF_HASH = '<?php echo $this->security->get_csrf_hash(); ?>';
 </script>
 <style>
 /*Legend specific*/

assets/js/radiohelpers.js

@@ -34,9 +34,6 @@ function frequencyToBand(frequency) {
 	else if(result >= 50000000 && result <= 56000000) {
 		return '6m';
 	}
-	else if(result >= 70000000 && result <= 72000000) {
-		return '4m';
-	}
 	else if(result >= 144000000 && result <= 148000000) {
 		return '2m';
 	}

index.php

@@ -74,7 +74,7 @@ switch (ENVIRONMENT)
 	case 'testing':
 	case 'production':
 		ini_set('display_errors', 0);
-		if (version_compare(PHP_VERSION, '5.3', '>='))
+		if (version_compare(PHP_VERSION, '7.0', '>='))
 		{
 			error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);
 		}

install/config/config.php

@@ -200,9 +200,6 @@ case 'finnish':
 case 'russian':
 	$config['language'] = $lang;
 	break;
-case 'portuguese':
-	$config['language'] = $lang;
-	break;
 case 'english':
 	$config['language'] = $lang;
 	break;

system/libraries/Encryption.php

@@ -152,7 +152,7 @@ class CI_Encryption {
 	public function __construct(array $params = array())
 	{
 		$this->_drivers = array(
-			'mcrypt'  => defined('MCRYPT_DEV_URANDOM'),
+			'mcrypt'  => defined('MCRYPT_DEV_URANDOM') && function_exists('mcrypt_encrypt'),
 			'openssl' => extension_loaded('openssl')
 		);
 
@@ -203,9 +203,20 @@ class CI_Encryption {
 
 		if (empty($this->_driver))
 		{
-			$this->_driver = ($this->_drivers['openssl'] === TRUE)
-				? 'openssl'
-				: 'mcrypt';
+			// Prefer OpenSSL on modern PHP versions where mcrypt is not available
+			if ($this->_drivers['openssl'] === TRUE)
+			{
+				$this->_driver = 'openssl';
+			}
+			elseif ($this->_drivers['mcrypt'] === TRUE)
+			{
+				$this->_driver = 'mcrypt';
+			}
+			else
+			{
+				// This shouldn't happen as we check both drivers in constructor
+				show_error('Encryption: No available encryption driver found.');
+			}
 
 			log_message('debug', "Encryption: Auto-configured driver '".$this->_driver."'.");
 		}