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

Doctrine database configuration error #6691

Closed
AscanderS opened this issue Jun 28, 2016 · 3 comments
Closed

Doctrine database configuration error #6691

AscanderS opened this issue Jun 28, 2016 · 3 comments
Labels
Doctrine hasPR A Pull Request has already been submitted for this issue.

Comments

@AscanderS
Copy link

AscanderS commented Jun 28, 2016

This is a discrepancy between the Symfony book and Symfony version 2.8 (at least) which is very disturbing for someone trying Symfony with PostgreSQL for the first time.

On page http://symfony.com/doc/2.8/book/doctrine.html It is explained that setting database_driver on app/config/parameters.yml allows to choose a driver.

# app/config/parameters.yml
parameters:
    database_driver:    pdo_mysql

However, parameter database_driver is not used (at least in version 2.8) since in app/config/config.yml you find that pdo_mysql is wired:

# app/config/config.yml
doctrine:
    dbal:
        driver:   pdo_mysql

There is no intention to add a database_driver parameter (see symfony/symfony#19196), so please correct the corresponding page on the symfony book.

@linaori
Copy link
Contributor

linaori commented Jun 28, 2016

Fixed by #6689 (as mentioned in your previous issue)

@AscanderS
Copy link
Author

Thanks. I hope I'll eventually appear fixed on the Symfony Book from version 2.8 (at least) on.
Not the case for now.

@xabbuh xabbuh added Doctrine hasPR A Pull Request has already been submitted for this issue. labels Jun 30, 2016
@xabbuh xabbuh closed this as completed Jun 30, 2016
@xabbuh
Copy link
Member

xabbuh commented Jun 30, 2016

The fix has been merged and will soon be visible on symfony.com. Thanks for reporting it @AscanderS and for fixing it @iltar!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Doctrine hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Development

No branches or pull requests

3 participants