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

[DBAL-1121] Allow optional host and server connection parameters for sqlanywhere driver #777

Merged

Conversation

deeky666
Copy link
Member

This patch makes it possible to connect to a SQL Anywhere database without having to specify host and/or server connection parameter.
The original assumption that the server parameter is necessary to connect to the database was wrong. It is only required if the host is running multiple named server instances and a specific instance should be connected to.
Also if the host parameter is not specified, it will default to localhost now.
The LINKS DSN parameter was replaced by the simpler and encouraged HOST parameter as the LINKS parameter is only required if you want to pass specific TCP/IP options.

@doctrinebot
Copy link

Hello,

thank you for creating this pull request. I have automatically opened an issue
on our Jira Bug Tracker for you. See the issue link:

http://www.doctrine-project.org/jira/browse/DBAL-1121

We use Jira to track the state of pull requests and the versions they got
included in.

@deeky666 deeky666 changed the title Make host and server connection parameters optional for sqlanywhere driver [DBAL-1121] Make host and server connection parameters optional for sqlanywhere driver Jan 18, 2015
@Ocramius Ocramius changed the title [DBAL-1121] Make host and server connection parameters optional for sqlanywhere driver [DBAL-1121] Allow optional host and server connection parameters for sqlanywhere driver Jan 18, 2015
Ocramius added a commit that referenced this pull request Jan 18, 2015
@Ocramius Ocramius merged commit 470cde9 into doctrine:master Jan 18, 2015
Ocramius added a commit that referenced this pull request Jan 18, 2015
@Ocramius
Copy link
Member

Merged, thanks @deeky666!

master: 9e7a971
2.5: cf3afae

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants