fix SQLServerPlatform GUID expression #254

Merged
merged 1 commit into from Jan 25, 2013

Conversation

Projects
None yet
3 participants
@deeky666
Member

deeky666 commented Jan 25, 2013

The GUID expression UUID() does not exist in SQLServer. Instead NEWID() is used. This PR fixes this.

See: http://msdn.microsoft.com/en-us/library/ms190348.aspx

@doctrinebot

This comment has been minimized.

Show comment
Hide comment
@doctrinebot

doctrinebot Jan 25, 2013

Hello,

thank you for positing this Pull Request. I have automatically opened an issue on our Jira Bug Tracker for you with the details of this Pull-Request. See the Link:

http://doctrine-project.org/jira/browse/DBAL-424

Hello,

thank you for positing this Pull Request. I have automatically opened an issue on our Jira Bug Tracker for you with the details of this Pull-Request. See the Link:

http://doctrine-project.org/jira/browse/DBAL-424

@guilhermeblanco

This comment has been minimized.

Show comment
Hide comment
@guilhermeblanco

guilhermeblanco Jan 25, 2013

Member

@deeky666 you're right. SQL Server 2000 and 2005 provides this function.
SQL Server 2005 and later also support NEWSEQUENTIALID(), but it seems there're some bugs related to them that we rely as working around here.
Merging since it completely satisfies the purpose.

Member

guilhermeblanco commented Jan 25, 2013

@deeky666 you're right. SQL Server 2000 and 2005 provides this function.
SQL Server 2005 and later also support NEWSEQUENTIALID(), but it seems there're some bugs related to them that we rely as working around here.
Merging since it completely satisfies the purpose.

guilhermeblanco added a commit that referenced this pull request Jan 25, 2013

@guilhermeblanco guilhermeblanco merged commit 99a1338 into doctrine:master Jan 25, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment