fix(tables): remove double-calling, consolidate db writes and use current year for SPDX header

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

Author: AndyScherzinger

lib/Service/RowService.php

@@ -98,7 +98,7 @@ public function findAllByTable(int $tableId, string $userId, ?int $limit = null,
 				$showColumnIds = array_map(fn (Column $column) => $column->getId(), $tableColumns);
 
 				$table = $this->tableMapper->find($tableId);
-				$sort = !empty($table->getSortArray()) ? $table->getSortArray() : null;
+				$sort = $table->getSortArray() ?: null;
 
 				return $this->row2Mapper->findAll($showColumnIds, $tableId, $limit, $offset, null, $sort, $userId);
 			} else {

lib/UserMigration/TablesMigrator.php

@@ -200,6 +200,7 @@ public function import(
 
 				$columnIdMap = $this->importColumns($importSource, $newTable, $table, $columnIdMap);
 
+				$needsUpdate = false;
 				if (!empty($table['columnOrder'])) {
 					$remapped = array_map(function (array $entry) use ($columnIdMap): array {
 						if (isset($entry['columnId'])) {
@@ -208,7 +209,7 @@ public function import(
 						return $entry;
 					}, $table['columnOrder']);
 					$newTable->setColumnOrder(json_encode($remapped));
-					$this->tableMapper->update($newTable);
+					$needsUpdate = true;
 				}
 				if (!empty($table['sort'])) {
 					$remapped = array_map(function (array $entry) use ($columnIdMap): array {
@@ -218,6 +219,9 @@ public function import(
 						return $entry;
 					}, $table['sort']);
 					$newTable->setSort(json_encode($remapped));
+					$needsUpdate = true;
+				}
+				if ($needsUpdate) {
 					$this->tableMapper->update($newTable);
 				}
 

src/modules/main/partials/editTablePartials/DefaultSortRules.vue

@@ -1,5 +1,5 @@
 <!--
-  - SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
+  - SPDX-FileCopyrightText: 2026 Nextcloud GmbH and Nextcloud contributors
   - SPDX-License-Identifier: AGPL-3.0-or-later
 -->
 <template>