Skip to content

Deprecate "version" column platform option#6940

Merged
morozov merged 1 commit intodoctrine:4.3.xfrom
morozov:deprecate-version
May 2, 2025
Merged

Deprecate "version" column platform option#6940
morozov merged 1 commit intodoctrine:4.3.xfrom
morozov:deprecate-version

Conversation

@morozov
Copy link
Member

@morozov morozov commented May 2, 2025

See #6939 for more context.

The version option is undocumented, untested, and it's hard to tell what it's supposed to do given the name. My best guess is that it makes the platform use a type analogous to TIMESTAMP but with the default value equal to "now". From the old JIRA on the internet archive (I cannot find the link), it looks like a leftover from the times when the ORM and the DBAL were a single project, so this option was meant to support some versioning.

We deprecated and removed the "now" functionality in #4753.

@morozov morozov added this to the 4.3.0 milestone May 2, 2025
@morozov morozov marked this pull request as ready for review May 2, 2025 05:46
@morozov morozov requested a review from greg0ire May 2, 2025 05:46
'doctrine/dbal',
'https://github.com/doctrine/dbal/pull/6940',
'The "version" column platform option is deprecated.',
__METHOD__,
Copy link
Member

@greg0ire greg0ire May 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no sprintf placeholder for this parameter.

@morozov morozov force-pushed the deprecate-version branch from 9cb51ba to 6cea9db Compare May 2, 2025 14:47
@morozov morozov merged commit 5e58518 into doctrine:4.3.x May 2, 2025
95 checks passed
@morozov morozov deleted the deprecate-version branch May 2, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants