From 91a9bcf8b6f355cd85510cfbc6cf7436e0761eba Mon Sep 17 00:00:00 2001 From: Carl Schwan Date: Thu, 13 Aug 2026 14:43:09 +0200 Subject: [PATCH] refactor: allow non-lowercase string in ITable methods Signed-off-by: Carl Schwan --- lib/private/DB/Schema/Table.php | 2 +- lib/public/DB/Schema/ITable.php | 24 ++++++++++++------------ 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/lib/private/DB/Schema/Table.php b/lib/private/DB/Schema/Table.php index dc07a71f0bdf4..34eb24379fcb6 100644 --- a/lib/private/DB/Schema/Table.php +++ b/lib/private/DB/Schema/Table.php @@ -40,7 +40,7 @@ public function getWrappedTable(): DBALTable { #[\Override] public function getName(): string { - /** @var non-empty-lowercase-string $name */ + /** @var non-empty-string $name */ $name = $this->table->getName(); return $name; } diff --git a/lib/public/DB/Schema/ITable.php b/lib/public/DB/Schema/ITable.php index 9e0758dc52bea..e705e27fc9bf5 100644 --- a/lib/public/DB/Schema/ITable.php +++ b/lib/public/DB/Schema/ITable.php @@ -23,7 +23,7 @@ interface ITable { * Returns the name of this table. * * @since 35.0.0 - * @return non-empty-lowercase-string + * @return non-empty-string */ public function getName(): string; @@ -39,10 +39,10 @@ public function getName(): string; public function setPrimaryKey(array $columnNames, string|false $indexName = false): self; /** - * @param list $columnNames + * @param list $columnNames * @param ?non-empty-string $indexName - * @param list $flags - * @param array $options + * @param list $flags + * @param array $options * * @throws SchemaException * @since 35.0.0 @@ -146,7 +146,7 @@ public function addUniqueIndex(array $columnNames, ?string $indexName = null, ar public function renameIndex(string $oldName, ?string $newName = null): self; /** - * @param non-empty-lowercase-string $name + * @param non-empty-string $name * @param Types::*|ColumnType $typeName * @param array{ * notnull?: bool, @@ -167,7 +167,7 @@ public function renameIndex(string $oldName, ?string $newName = null): self; public function addColumn(string $name, string|ColumnType $typeName, array $options = []): IColumn; /** - * @param non-empty-lowercase-string $name + * @param non-empty-string $name * @param array{ * notnull?: bool, * length?: ?int, @@ -189,7 +189,7 @@ public function modifyColumn(string $name, array $options): self; /** * Drops a Column from the Table. * - * @param non-empty-lowercase-string $name + * @param non-empty-string $name * @since 35.0.0 */ public function dropColumn(string $name): self; @@ -197,7 +197,7 @@ public function dropColumn(string $name): self; /** * Returns whether this table has a Column with the given name. * - * @param non-empty-lowercase-string $name The column name. + * @param non-empty-string $name The column name. * @since 35.0.0 */ public function hasColumn(string $name): bool; @@ -205,7 +205,7 @@ public function hasColumn(string $name): bool; /** * Returns the Column with the given name. * - * @param non-empty-lowercase-string $name The column name. + * @param non-empty-string $name The column name. * * @throws SchemaException If the column does not exist. * @since 35.0.0 @@ -242,9 +242,9 @@ public function getIndex(string $name): IIndex; * * Name is inferred from the local columns. * - * @param ITable|non-empty-lowercase-string $foreignTable Table schema instance or table name - * @param list $localColumnNames - * @param list $foreignColumnNames + * @param ITable|non-empty-string $foreignTable Table schema instance or table name + * @param list $localColumnNames + * @param list $foreignColumnNames * @param array $options * * @throws SchemaException