DDC-1290: Version fields do not accept smallint and bigint #1622

Closed
opened 2026-01-22 13:20:15 +01:00 by admin · 2 comments
Owner

Originally created by @doctrinebot on GitHub (Jul 24, 2011).

Originally assigned to: @beberlei on GitHub.

Jira issue originally created by user guite:

In ClassMetadataInfo there is a method called setVersionMapping.

It sets default values for the version fields, but only for integer and datetime.
As I expected that smallint and bigint would be handled the same way, I would consider the fact they are missing as a bug.

So instead of

if ($mapping['type'] == 'integer') {

it should probably be

if (in_array($mapping['type'], array('integer', 'smallint', 'bigint'))) {

Originally created by @doctrinebot on GitHub (Jul 24, 2011). Originally assigned to: @beberlei on GitHub. Jira issue originally created by user guite: In ClassMetadataInfo there is a method called setVersionMapping. It sets default values for the version fields, but only for integer and datetime. As I expected that smallint and bigint would be handled the same way, I would consider the fact they are missing as a bug. So instead of if ($mapping['type'] == 'integer') { it should probably be if (in_array($mapping['type'], array('integer', 'smallint', 'bigint'))) {
admin added the Bug label 2026-01-22 13:20:15 +01:00
admin closed this issue 2026-01-22 13:20:17 +01:00
Author
Owner

@doctrinebot commented on GitHub (Jul 26, 2011):

Comment created by @beberlei:

I call it improvement and its fixed in master, will be in 2.2

@doctrinebot commented on GitHub (Jul 26, 2011): Comment created by @beberlei: I call it improvement and its fixed in master, will be in 2.2
Author
Owner

@doctrinebot commented on GitHub (Jul 26, 2011):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Jul 26, 2011): Issue was closed with resolution "Fixed"
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#1622