diff --git a/apps/cloud_federation_api/lib/Migration/Version1016Date202502262004.php b/apps/cloud_federation_api/lib/Migration/Version1016Date202502262004.php index a3523d45e38..b582c827f91 100644 --- a/apps/cloud_federation_api/lib/Migration/Version1016Date202502262004.php +++ b/apps/cloud_federation_api/lib/Migration/Version1016Date202502262004.php @@ -12,9 +12,11 @@ namespace OCA\CloudFederationAPI\Migration; use Closure; use OCP\DB\ISchemaWrapper; use OCP\DB\Types; +use OCP\Migration\Attributes\CreateTable; use OCP\Migration\IOutput; use OCP\Migration\SimpleMigrationStep; +#[CreateTable(table: 'federated_invites', columns: ['id', 'user_id', 'recipient_provider', 'recipient_user_id', 'recipient_name', 'recipient_email', 'token', 'accepted', 'created_at', 'expired_at', 'accepted_at'], description: 'Supporting the OCM Invitation Flow feature')] class Version1016Date202502262004 extends SimpleMigrationStep { /** * @param IOutput $output diff --git a/apps/dav/lib/Migration/Version1034Date20250605132605.php b/apps/dav/lib/Migration/Version1034Date20250605132605.php index d46892f6e5a..2eff120c393 100644 --- a/apps/dav/lib/Migration/Version1034Date20250605132605.php +++ b/apps/dav/lib/Migration/Version1034Date20250605132605.php @@ -12,9 +12,14 @@ namespace OCA\DAV\Migration; use Closure; use OCP\DB\ISchemaWrapper; use OCP\DB\Types; +use OCP\Migration\Attributes\AddColumn; +use OCP\Migration\Attributes\ColumnType; +use OCP\Migration\Attributes\CreateTable; use OCP\Migration\IOutput; use OCP\Migration\SimpleMigrationStep; +#[AddColumn(table: 'dav_shares', name: 'token', type: ColumnType::STRING)] +#[CreateTable(table: 'calendars_federated', columns: ['id', 'display_name', 'color', 'uri', 'principaluri', 'remote_Url', 'token', 'sync_token', 'last_sync', 'shared_by', 'shared_by_display_name', 'components', 'permissions'], description: 'Supporting Federated Calender')] class Version1034Date20250605132605 extends SimpleMigrationStep { /** * @param IOutput $output diff --git a/apps/dav/lib/Migration/Version1034Date20250813093701.php b/apps/dav/lib/Migration/Version1034Date20250813093701.php index 10be71f067b..e3571781042 100644 --- a/apps/dav/lib/Migration/Version1034Date20250813093701.php +++ b/apps/dav/lib/Migration/Version1034Date20250813093701.php @@ -13,10 +13,12 @@ use Closure; use OCP\DB\ISchemaWrapper; use OCP\DB\QueryBuilder\IQueryBuilder; use OCP\IDBConnection; +use OCP\Migration\Attributes\DataCleansing; use OCP\Migration\IOutput; use OCP\Migration\SimpleMigrationStep; use Override; +#[DataCleansing(table: 'properties', description: 'remove commonly used custom properties set as default')] class Version1034Date20250813093701 extends SimpleMigrationStep { public function __construct( private IDBConnection $db, diff --git a/build/psalm/OcpSinceChecker.php b/build/psalm/OcpSinceChecker.php index 38af92cf4ef..c4e8f717828 100644 --- a/build/psalm/OcpSinceChecker.php +++ b/build/psalm/OcpSinceChecker.php @@ -26,6 +26,7 @@ class OcpSinceChecker implements Psalm\Plugin\EventHandler\AfterClassLikeVisitIn $isTesting = str_contains($statementsSource->getFilePath(), '/lib/public/Notification/') || str_contains($statementsSource->getFilePath(), '/lib/public/Config/') + || str_contains($statementsSource->getFilePath(), '/lib/public/Migration/Attributes/') || str_contains($statementsSource->getFilePath(), 'CalendarEventStatus'); if ($isTesting) { diff --git a/core/Migrations/Version32000Date20250620081925.php b/core/Migrations/Version32000Date20250620081925.php index 13e1ac0f87d..a161964da67 100644 --- a/core/Migrations/Version32000Date20250620081925.php +++ b/core/Migrations/Version32000Date20250620081925.php @@ -9,8 +9,11 @@ declare(strict_types=1); namespace OC\Core\Migrations; +use OCP\Migration\Attributes\DataCleansing; + /** * Run the old migration Version24000Date20211210141942 again. */ +#[DataCleansing(table: 'preferences', description: 'lowercase accounts email address')] class Version32000Date20250620081925 extends Version24000Date20211210141942 { } diff --git a/core/Migrations/Version32000Date20250731062008.php b/core/Migrations/Version32000Date20250731062008.php index 26f0d20d19c..5976be55a54 100644 --- a/core/Migrations/Version32000Date20250731062008.php +++ b/core/Migrations/Version32000Date20250731062008.php @@ -12,6 +12,7 @@ namespace OC\Core\Migrations; use Closure; use OCP\DB\ISchemaWrapper; use OCP\IDBConnection; +use OCP\Migration\Attributes\DataCleansing; use OCP\Migration\IOutput; use OCP\Migration\SimpleMigrationStep; use Override; @@ -21,6 +22,8 @@ use Override; * This migration will clean up existing duplicates. * The new unique constraint is added in @see \OC\Core\Listener\AddMissingIndicesListener */ +#[DataCleansing(table: 'vcategory', description: 'Cleanup of duplicate vcategory records')] +#[DataCleansing(table: 'vcategory_to_object', description: 'Update object references')] class Version32000Date20250731062008 extends SimpleMigrationStep { public function __construct( private IDBConnection $connection, diff --git a/lib/composer/composer/autoload_classmap.php b/lib/composer/composer/autoload_classmap.php index 3af23258c72..1dfbd6cc06d 100644 --- a/lib/composer/composer/autoload_classmap.php +++ b/lib/composer/composer/autoload_classmap.php @@ -678,6 +678,8 @@ return array( 'OCP\\Migration\\Attributes\\ColumnMigrationAttribute' => $baseDir . '/lib/public/Migration/Attributes/ColumnMigrationAttribute.php', 'OCP\\Migration\\Attributes\\ColumnType' => $baseDir . '/lib/public/Migration/Attributes/ColumnType.php', 'OCP\\Migration\\Attributes\\CreateTable' => $baseDir . '/lib/public/Migration/Attributes/CreateTable.php', + 'OCP\\Migration\\Attributes\\DataCleansing' => $baseDir . '/lib/public/Migration/Attributes/DataCleansing.php', + 'OCP\\Migration\\Attributes\\DataMigrationAttribute' => $baseDir . '/lib/public/Migration/Attributes/DataMigrationAttribute.php', 'OCP\\Migration\\Attributes\\DropColumn' => $baseDir . '/lib/public/Migration/Attributes/DropColumn.php', 'OCP\\Migration\\Attributes\\DropIndex' => $baseDir . '/lib/public/Migration/Attributes/DropIndex.php', 'OCP\\Migration\\Attributes\\DropTable' => $baseDir . '/lib/public/Migration/Attributes/DropTable.php', diff --git a/lib/composer/composer/autoload_static.php b/lib/composer/composer/autoload_static.php index 4fe519f6be3..4666d787063 100644 --- a/lib/composer/composer/autoload_static.php +++ b/lib/composer/composer/autoload_static.php @@ -719,6 +719,8 @@ class ComposerStaticInit749170dad3f5e7f9ca158f5a9f04f6a2 'OCP\\Migration\\Attributes\\ColumnMigrationAttribute' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/ColumnMigrationAttribute.php', 'OCP\\Migration\\Attributes\\ColumnType' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/ColumnType.php', 'OCP\\Migration\\Attributes\\CreateTable' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/CreateTable.php', + 'OCP\\Migration\\Attributes\\DataCleansing' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/DataCleansing.php', + 'OCP\\Migration\\Attributes\\DataMigrationAttribute' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/DataMigrationAttribute.php', 'OCP\\Migration\\Attributes\\DropColumn' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/DropColumn.php', 'OCP\\Migration\\Attributes\\DropIndex' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/DropIndex.php', 'OCP\\Migration\\Attributes\\DropTable' => __DIR__ . '/../../..' . '/lib/public/Migration/Attributes/DropTable.php', diff --git a/lib/public/Migration/Attributes/AddColumn.php b/lib/public/Migration/Attributes/AddColumn.php index d84a0c1f60c..e371ec81d19 100644 --- a/lib/public/Migration/Attributes/AddColumn.php +++ b/lib/public/Migration/Attributes/AddColumn.php @@ -9,13 +9,13 @@ declare(strict_types=1); namespace OCP\Migration\Attributes; use Attribute; +use OCP\AppFramework\Attribute\Consumable; /** * attribute on new column creation - * - * @since 30.0.0 */ #[Attribute(Attribute::IS_REPEATABLE | Attribute::TARGET_CLASS)] +#[Consumable(since: '30.0.0')] class AddColumn extends ColumnMigrationAttribute { /** * @return string diff --git a/lib/public/Migration/Attributes/AddIndex.php b/lib/public/Migration/Attributes/AddIndex.php index ee22fe7f128..86b5a89c389 100644 --- a/lib/public/Migration/Attributes/AddIndex.php +++ b/lib/public/Migration/Attributes/AddIndex.php @@ -9,13 +9,13 @@ declare(strict_types=1); namespace OCP\Migration\Attributes; use Attribute; +use OCP\AppFramework\Attribute\Consumable; /** * attribute on index creation - * - * @since 30.0.0 */ #[Attribute(Attribute::IS_REPEATABLE | Attribute::TARGET_CLASS)] +#[Consumable(since: '30.0.0')] class AddIndex extends IndexMigrationAttribute { /** * @return string diff --git a/lib/public/Migration/Attributes/ColumnMigrationAttribute.php b/lib/public/Migration/Attributes/ColumnMigrationAttribute.php index 30b6fe008e6..d3f4aa05ade 100644 --- a/lib/public/Migration/Attributes/ColumnMigrationAttribute.php +++ b/lib/public/Migration/Attributes/ColumnMigrationAttribute.php @@ -8,14 +8,13 @@ declare(strict_types=1); */ namespace OCP\Migration\Attributes; -use JsonSerializable; +use OCP\AppFramework\Attribute\Consumable; /** * generic class related to migration attribute about column changes - * - * @since 30.0.0 */ -class ColumnMigrationAttribute extends MigrationAttribute implements JsonSerializable { +#[Consumable(since: '30.0.0')] +class ColumnMigrationAttribute extends MigrationAttribute { /** * @param string $table name of the database table * @param string $name name of the column diff --git a/lib/public/Migration/Attributes/ColumnType.php b/lib/public/Migration/Attributes/ColumnType.php index ab6044e3ae2..54c0365bc9e 100644 --- a/lib/public/Migration/Attributes/ColumnType.php +++ b/lib/public/Migration/Attributes/ColumnType.php @@ -8,12 +8,14 @@ declare(strict_types=1); */ namespace OCP\Migration\Attributes; +use OCP\AppFramework\Attribute\Consumable; + /** * enum ColumnType based on OCP\DB\Types * * @see \OCP\DB\Types - * @since 30.0.0 */ +#[Consumable(since: '30.0.0')] enum ColumnType : string { /** @since 30.0.0 */ case BIGINT = 'bigint'; diff --git a/lib/public/Migration/Attributes/CreateTable.php b/lib/public/Migration/Attributes/CreateTable.php index df0418fa4bc..c774b0aff56 100644 --- a/lib/public/Migration/Attributes/CreateTable.php +++ b/lib/public/Migration/Attributes/CreateTable.php @@ -9,13 +9,13 @@ declare(strict_types=1); namespace OCP\Migration\Attributes; use Attribute; +use OCP\AppFramework\Attribute\Consumable; /** * attribute on table creation - * - * @since 30.0.0 */ #[Attribute(Attribute::IS_REPEATABLE | Attribute::TARGET_CLASS)] +#[Consumable(since: '30.0.0')] class CreateTable extends TableMigrationAttribute { /** * @return string diff --git a/lib/public/Migration/Attributes/DataCleansing.php b/lib/public/Migration/Attributes/DataCleansing.php new file mode 100644 index 00000000000..6215c8b0268 --- /dev/null +++ b/lib/public/Migration/Attributes/DataCleansing.php @@ -0,0 +1,27 @@ +getTable() . '\''; + } +} diff --git a/lib/public/Migration/Attributes/DataMigrationAttribute.php b/lib/public/Migration/Attributes/DataMigrationAttribute.php new file mode 100644 index 00000000000..08d12a9f37b --- /dev/null +++ b/lib/public/Migration/Attributes/DataMigrationAttribute.php @@ -0,0 +1,18 @@ +