Remove "primary" option from schema builder as it is not used anymore

See doctrine/dbal@138eb85#diff-300f55366adb50a32a40882ebdc95c163b141f64cba5f45f20bda04a907b3eb3L82

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Author: MorrisJobke

lib/private/DB/MDB2SchemaReader.php

@@ -208,7 +208,8 @@ private function loadField($table, $xml) {
 					break;
 				case 'primary':
 					$primary = $this->asBool($child);
-					$options['primary'] = $primary;
+					# $options['primary'] = true;
+					# not used anymore https://github.com/doctrine/dbal/commit/138eb85234a1faeaa2e6a32cd7bcc66bb51c64e8#diff-300f55366adb50a32a40882ebdc95c163b141f64cba5f45f20bda04a907b3eb3L82
 					break;
 				case 'precision':
 					$precision = (string)$child;
@@ -257,7 +258,8 @@ private function loadField($table, $xml) {
 			if (!empty($options['autoincrement'])
 				&& !empty($options['notnull'])
 			) {
-				$options['primary'] = true;
+				# $options['primary'] = true;
+				# not used anymore https://github.com/doctrine/dbal/commit/138eb85234a1faeaa2e6a32cd7bcc66bb51c64e8#diff-300f55366adb50a32a40882ebdc95c163b141f64cba5f45f20bda04a907b3eb3L82
 			}
 
 			$table->addColumn($name, $type, $options);

tests/lib/DB/ConnectionTest.php

@@ -44,7 +44,7 @@ protected static function dropTestTable() {
 
 	protected function setUp(): void {
 		parent::setUp();
-		$this->connection = \OC::$server->getDatabaseConnection();
+		$this->connection = \OC::$server->get(\OC\DB\Connection::class);
 	}
 
 	protected function tearDown(): void {