need pagination-context for sortfields; only disable pagination-code on view

Signed-off-by: Michael Kaufmann <d00p@froxlor.org>

install/froxlor.sql.php

@@ -696,7 +696,7 @@ opcache.validate_timestamps'),
 	('system', 'distribution', ''),
 	('system', 'update_channel', 'stable'),
 	('system', 'updatecheck_data', ''),
-	('system', 'update_notify_last', '2.0.1'),
+	('system', 'update_notify_last', '2.0.2'),
 	('system', 'traffictool', 'goaccess'),
 	('api', 'enabled', '0'),
 	('2fa', 'enabled', '1'),
@@ -740,7 +740,7 @@ opcache.validate_timestamps'),
 	('panel', 'logo_overridetheme', '0'),
 	('panel', 'logo_overridecustom', '0'),
 	('panel', 'settings_mode', '0'),
-	('panel', 'version', '2.0.1'),
+	('panel', 'version', '2.0.2'),
 	('panel', 'db_version', '202212060');
 
 

install/updates/froxlor/update_2.x.inc.php

@@ -67,8 +67,8 @@ if (Froxlor::isFroxlorVersion('0.10.38.3')) {
 	) ENGINE=InnoDB CHARSET=utf8 COLLATE=utf8_general_ci;";
 	Database::query($sql);
 	// new customer allowed_mysqlserver field
-	Database::query("ALTER TABLE `" . TABLE_PANEL_CUSTOMERS . "` ADD `allowed_mysqlserver` text NOT NULL default '[0]';");
+	Database::query("ALTER TABLE `" . TABLE_PANEL_CUSTOMERS . "` ADD `allowed_mysqlserver` text NOT NULL;");
-	Database::query("ALTER TABLE `" . TABLE_PANEL_CUSTOMERS . "` CHANGE COLUMN `allowed_phpconfigs` `allowed_phpconfigs` text NOT NULL default '';");
+	Database::query("ALTER TABLE `" . TABLE_PANEL_CUSTOMERS . "` CHANGE COLUMN `allowed_phpconfigs` `allowed_phpconfigs` text NOT NULL;");
 	Database::query("ALTER TABLE `" . TABLE_PANEL_CUSTOMERS . "` CHANGE COLUMN `customernumber` `customernumber` varchar(100) NOT NULL default '';");
 	$has_customer_table_update_200 = true;
 	// ftp_users adjustments
@@ -240,3 +240,8 @@ if (Froxlor::isFroxlorVersion('2.0.0')) {
 
 	Froxlor::updateToVersion('2.0.1');
 }
+
+if (Froxlor::isFroxlorVersion('2.0.1')) {
+	Update::showUpdateStep("Updating from 2.0.1 to 2.0.2", false);
+	Froxlor::updateToVersion('2.0.2');
+}

lib/Froxlor/Froxlor.php

@@ -31,7 +31,7 @@ final class Froxlor
 {
 
 	// Main version variable
-	const VERSION = '2.0.1';
+	const VERSION = '2.0.2';
 
 	// Database version (YYYYMMDDC where C is a daily counter)
 	const DBVERSION = '202212060';

lib/Froxlor/UI/Collection.php

@@ -113,17 +113,15 @@ class Collection
 	public function withPagination(array $columns, array $default_sorting = []): Collection
 	{
 		// Get only searchable columns
-		/*
 		$sortableColumns = [];
 		foreach ($columns as $key => $column) {
-			if (isset($column['sortable']) && $column['sortable']) {
+			if (!isset($column['sortable']) || (isset($column['sortable']) && $column['sortable'])) {
 				$sortableColumns[$key] = $column;
 			}
 		}
-		*/
 
 		// Prepare pagination
-		$this->pagination = new Pagination($columns, $this->count(), (int)Settings::Get('panel.paging'), $default_sorting);
+		$this->pagination = new Pagination($sortableColumns, $this->count(), (int)Settings::Get('panel.paging'), $default_sorting);
 		$this->params = array_merge($this->params, $this->pagination->getApiCommandParams());
 
 		return $this;

lib/Froxlor/UI/Pagination.php

@@ -63,7 +63,9 @@ class Pagination
 		$this->perPage = $perPage;
 		$this->pageno = 1;
 		// add default limitation by settings
-		$this->addLimit(Settings::Get('panel.paging'));
+		if (Settings::Get('panel.paging') > 0) {
+			$this->addLimit(Settings::Get('panel.paging'));
+		}
 		// check search request
 		$this->searchtext = '';
 		if (count($fields) > 0) {
@@ -205,7 +207,7 @@ class Pagination
 				"total" => $this->entries,
 				"per_page" => $this->perPage,
 				"current_page" => $this->pageno,
-				"last_page" => ceil($this->entries / $this->perPage),
+				"last_page" => (Settings::Get('panel.paging') > 0) ? ceil($this->entries / $this->perPage) : -1,
 				"from" => $this->data['sql_offset'] ?? null,
 				"to" => min($this->data['sql_offset'] + $this->perPage, $this->entries),
 				'sortfields' => array_keys($this->fields),

lib/tablelisting/tablelisting.sslcertificates.php

@@ -54,11 +54,13 @@ return [
 				'label' => lng('ssl_certificates.valid_from'),
 				'field' => 'validfromdate',
 				'searchable' => false,
+				'sortable' => false,
 			],
 			'c.validtodate' => [
 				'label' => lng('ssl_certificates.valid_until'),
 				'field' => 'validtodate',
 				'searchable' => false,
+				'sortable' => false,
 			],
 		],
 		'visible_columns' => Listing::getVisibleColumnsForListing('sslcertificates_list', [