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

Update multiple entity managers #8980

Closed
sergiu-popa opened this issue Jan 4, 2018 · 3 comments
Closed

Update multiple entity managers #8980

sergiu-popa opened this issue Jan 4, 2018 · 3 comments
Labels
actionable Clear and specific issues ready for anyone to take them. hasPR A Pull Request has already been submitted for this issue.
Milestone

Comments

@sergiu-popa
Copy link

Can we update this article for SF4? https://symfony.com/doc/current/doctrine/multiple_entity_managers.html

This is the new default configurations for doctrine:

doctrine:
    dbal:
        # configure these for your database server
        driver: 'pdo_mysql'
        server_version: '5.7'
        charset: utf8mb4

        # With Symfony 3.3, remove the `resolve:` prefix
        url: '%env(resolve:DATABASE_URL)%'

This is an example of multiple connections using the url option:

doctrine:
    dbal:
        # configure these for your database server

        # With Symfony 3.3, remove the `resolve:` prefix
        default_connection: default

        connections:
            default:
                url: '%env(resolve:DATABASE_URL)%'
                driver: 'pdo_mysql'
                server_version: '5.7'
                charset: utf8mb4
            books:
                url: '%env(resolve:DATABASE_URL_BOOKS)%'
@javiereguiluz javiereguiluz added actionable Clear and specific issues ready for anyone to take them. hasPR A Pull Request has already been submitted for this issue. labels Jul 19, 2018
@javiereguiluz javiereguiluz added this to the 4.0 milestone Jul 19, 2018
@javiereguiluz
Copy link
Member

Thanks for reporting this and sorry it took us so long to respond. We're fixing it in #10093.

@sergiu-popa
Copy link
Author

sergiu-popa commented Jul 19, 2018 via email

javiereguiluz added a commit that referenced this issue Jul 20, 2018
This PR was squashed before being merged into the 4.0 branch (closes #10093).

Discussion
----------

Updated some config for Symfony 4 practices

Fixes #8980.

Commits
-------

a22d14d Updated some config for Symfony 4 practices
@javiereguiluz
Copy link
Member

Fixed by #10093.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
actionable Clear and specific issues ready for anyone to take them. hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Development

No branches or pull requests

2 participants