fix: use bigint in initial lock migrations

Assisted-by: Codex:GPT-6
Signed-off-by: Julius Knorr <jus@bitgrid.net>

Author: juliusknorr

lib/Migration/Version0001Date20191105000001.php

@@ -50,12 +50,12 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 		$table = $schema->createTable('files_lock');
 
 		$table->addColumn(
-			'id', 'integer',
+			'id', 'bigint',
 			[
 				'autoincrement' => true,
 				'unsigned' => true,
 				'notnull' => true,
-				'length' => 11
+				'length' => 20
 			]
 		);
 		$table->addColumn(
@@ -66,11 +66,11 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 			]
 		);
 		$table->addColumn(
-			'file_id', 'integer',
+			'file_id', 'bigint',
 			[
 				'notnull' => true,
 				'unsigned' => true,
-				'length' => 11,
+				'length' => 20,
 			]
 		);
 		$table->addColumn(

lib/Migration/Version1000Date20220201111525.php

@@ -48,9 +48,10 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 
 		if (!$table->hasColumn('ttl')) {
 			$table->addColumn(
-				'ttl', Types::INTEGER,
+				'ttl', Types::BIGINT,
 				[
 					'default' => 0,
+					'length' => 20,
 				]
 			);
 			$hasSchemaChanges = true;

tests/Unit/Migration/Version35000Date20260928000000Test.php

@@ -23,7 +23,10 @@
 
 class Version35000Date20260928000000Test extends TestCase {
 
-	public function testMigrationPreservesSchemaAndCanRunAgain(): void {
+	/**
+	 * @dataProvider schemaProvider
+	 */
+	public function testMigrationPreservesSchemaAndCanRunAgain(bool $legacySchema): void {
 		$table = new Table(new DBALTable('files_lock'));
 		$schema = $this->createMock(ISchemaWrapper::class);
 		$schema->method('hasTable')->with('files_lock')->willReturn(false);
@@ -35,6 +38,15 @@ public function testMigrationPreservesSchemaAndCanRunAgain(): void {
 		(new Version0001Date20191105000001($this->createMock(IDBConnection::class)))->changeSchema($output, $schemaClosure, []);
 		(new Version1000Date20220201111525())->changeSchema($output, $schemaClosure, []);
 		(new Version1000Date20220430180808())->changeSchema($output, $schemaClosure, []);
+		foreach (['id', 'file_id', 'ttl'] as $name) {
+			$column = $table->getColumn($name);
+			self::assertSame(Types::BIGINT, $column->getType()->getName());
+			self::assertSame(20, $column->getLength());
+			if ($legacySchema) {
+				$column->setType(Types::INTEGER);
+				$column->setLength($name === 'ttl' ? null : 11);
+			}
+		}
 		$before = clone $table->getWrappedTable();
 
 		$migration = new Version35000Date20260928000000();
@@ -52,4 +64,11 @@ public function testMigrationPreservesSchemaAndCanRunAgain(): void {
 			self::assertEquals($before->getIndexes(), $table->getWrappedTable()->getIndexes());
 		}
 	}
+
+	public static function schemaProvider(): array {
+		return [
+			'fresh install' => [false],
+			'legacy integer columns' => [true],
+		];
+	}
 }