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

Setting DB for DbUse? #54

Open
andyedison opened this issue Feb 11, 2017 · 3 comments
Open

Setting DB for DbUse? #54

andyedison opened this issue Feb 11, 2017 · 3 comments

Comments

@andyedison
Copy link

Hello,

I'm a little confused about the usage of DbUse(). I was imagining it accepting a string that then attempts to change the connection to the database passed in. But it seems to be checking an un-exported variable. Am I missing something?

Thanks.

@MarkSonghurst
Copy link
Contributor

MarkSonghurst commented Feb 11, 2017 via email

@ianic
Copy link
Contributor

ianic commented Feb 12, 2017

Good point andyedison.

Before integrating this pull request DbUse was used during connecting to switch into the database specified in the connection string.
It always uses database from connection string.

After that it is not used any more. It is some kind of rudiment.
I should probably remove it altogether.
Or make it accept a database name and than switch connection to that database.

@andyedison
Copy link
Author

I have a use case that would be great to pass a database name in as a string and switch to that database. I'd volunteer to make the change.

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

No branches or pull requests

3 participants