Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SQL Server Unique indexes constraint #2393

Closed
j-d opened this issue May 13, 2016 · 3 comments
Closed

SQL Server Unique indexes constraint #2393

j-d opened this issue May 13, 2016 · 3 comments

Comments

@j-d
Copy link

j-d commented May 13, 2016

Hello,
This line is incorrect

$constraint = $this->_appendUniqueConstraintDefinition($constraint, $index);

This issue was introduced in SQL Server 2008 according to https://msdn.microsoft.com/en-us/library/ms188783.aspx.

(And I have SQL Server 2005 and it doesn't work)

@Ocramius
Copy link
Member

What exactly is incorrect about it? Can you point to a failure or reproducible test case about it?

@j-d
Copy link
Author

j-d commented May 14, 2016

Thanks for your prompt response @Ocramius.

I have given it a go here at writting the test and the fix here: #2394

I am not sure how to make a functional test on this project that runs it on SQL Server 2005 (where it breaks). I also don't have an enviroment here to test this test right now; but hopefully you will see what I was trying to do.

Also, I believe this is incorrect:

Because that method is only overwritten by PostgreSql, but according to the MS documentation I referenced above, this appears to be supported on MS SQL Server 2008+; so I have changed it too.

@morozov morozov closed this as not planned Won't fix, can't repro, duplicate, stale Sep 7, 2022
@github-actions
Copy link

github-actions bot commented Oct 8, 2022

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants