Pick a shorter name for the transfer ownership table

Signed-off-by: Joas Schilling <coding@schilljs.com>

Author: nickvergessen

apps/files/lib/Db/TransferOwnershipMapper.php

@@ -29,7 +29,7 @@
 
 class TransferOwnershipMapper extends QBMapper {
 	public function __construct(IDBConnection $db) {
-		parent::__construct($db, 'user_transfer_ownership', TransferOwnership::class);
+		parent::__construct($db, 'user_transfer_owner', TransferOwnership::class);
 	}
 
 	public function getById(int $id): TransferOwnership {

apps/files/lib/Migration/Version11301Date20191113195931.php

@@ -42,11 +42,12 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 		/** @var ISchemaWrapper $schema */
 		$schema = $schemaClosure();
 
-		$table = $schema->createTable('user_transfer_ownership');
-		$table->addColumn('id', 'integer', [
+		$table = $schema->createTable('user_transfer_owner');
+		$table->addColumn('id', 'bigint', [
 			'autoincrement' => true,
 			'notnull' => true,
-			'length' => 4,
+			'length' => 20,
+			'unsigned' => true,
 		]);
 		$table->addColumn('source_user', 'string', [
 			'notnull' => true,

lib/private/DB/MigrationService.php

@@ -547,7 +547,7 @@ public function ensureOracleIdentifierLengthLimit(Schema $sourceSchema, Schema $
 				if (!$isUsingDefaultName && \strlen($indexName) > 30) {
 					throw new \InvalidArgumentException('Primary index name  on "'  . $table->getName() . '" is too long.');
 				}
-				if ($isUsingDefaultName && \strlen($table->getName()) - $prefixLength > 23) {
+				if ($isUsingDefaultName && \strlen($table->getName()) - $prefixLength >= 23) {
 					throw new \InvalidArgumentException('Primary index name  on "'  . $table->getName() . '" is too long.');
 				}
 			}