fix(tables): use parent logic/implementation, revert formatting change

AI-assistant: Copilot v1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

Author: AndyScherzinger

lib/Service/ValueObject/ColumnOrderInformation.php

@@ -42,10 +42,14 @@ public function getOrder(): int {
 	}
 
 	public static function fromArray(array $data): self {
+		static::assertRequiredFields($data);
+		return new self((int)$data[self::KEY_ID], (int)$data[self::KEY_ORDER]);
+	}
+
+	protected static function assertRequiredFields(array $data): void {
 		if (!isset($data[self::KEY_ID], $data[self::KEY_ORDER])) {
 			throw new \InvalidArgumentException('Column settings entry is missing required fields: columnId and order are required');
 		}
-		return new self((int)$data[self::KEY_ID], (int)$data[self::KEY_ORDER]);
 	}
 
 	public function offsetExists(mixed $offset): bool {

lib/Service/ValueObject/ViewColumnInformation.php

@@ -42,9 +42,7 @@ public function isMandatory(): bool {
 	}
 
 	public static function fromArray(array $data): static {
-		if (!isset($data[self::KEY_ID], $data[self::KEY_ORDER])) {
-			throw new \InvalidArgumentException('Column settings entry is missing required fields: columnId and order are required');
-		}
+		static::assertRequiredFields($data);
 		return new static(
 			(int)$data[self::KEY_ID],
 			(int)$data[self::KEY_ORDER],
@@ -57,21 +55,17 @@ public static function fromArray(array $data): static {
 	 * @return array{columnId: int, order: int, readonly: bool, mandatory: bool}
 	 */
 	public function jsonSerialize(): array {
-		return [
-			self::KEY_ID => $this->getId(),
-			self::KEY_ORDER => $this->getOrder(),
+		return array_merge(parent::jsonSerialize(), [
 			self::KEY_READONLY => $this->isReadonly(),
 			self::KEY_MANDATORY => $this->isMandatory(),
-		];
+		]);
 	}
 
 	protected function ensureType(string $offset, mixed $value): int|bool {
 		return match ($offset) {
-			self::KEY_ID,
-			self::KEY_ORDER => (int)$value,
-			self::KEY_READONLY,
+			self::KEY_READONLY => (bool)$value,
 			self::KEY_MANDATORY => (bool)$value,
-			default => throw new \InvalidArgumentException("Invalid offset: $offset"),
+			default => parent::ensureType($offset, $value),
 		};
 	}
 }