make connction string more generic
eman1986 opened this issue · 8 comments
The most glaring issue I see with this PDO class is that its hard coded to use MySQL.
The connection string portion should get rewritten so that any connection string supported by PDO will work instead of just one.
Ah interesting. Out of curiosity, which of these drivers would you consider using thin-pdo for?
I don't see why it couldn't be built to handle all of them. at the very least Postgresql, sqlite, and ms sql server.
I agree, it would be a great feature to support all the PHP PDO drivers. I think postgres is especially useful. That said, unless you have a specific use case you are requesting support for, I sadly don't have the time to give this the attention it deserves. In the meantime though, I highly encourage you to contribute to this project! :)
I might just do that :)
👍
I'm 90% done, the connection part has been written to accept any PDO driver, but SQL Server & Oracle I know from my experience with them handle limiting of records differently, so little stuff like that is my next task. :)
I'll keep you updated on my progress on this.
Sounds great! I look forward to the pull request :)
On Mar 23, 2013, at 4:45 PM, eman1986 notifications@github.com wrote:
I'm 90% done, the connection part has been written to accept any PDO driver, but SQL Server & Oracle I know from my experience with them handle limiting of records differently, so little stuff like that is my next task. :)
I'll keep you updated on my progress on this.
—
Reply to this email directly or view it on GitHub.
sent a pull request :)
there is technically a part two to this but this will at least provide multiple database engine support.