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

set doctrine.orm.controller_resolver.auto_mapping: true #738

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

asdfzdfj
Copy link
Contributor

it looks like doctrine bundle is about to deprecate controller auto mapping stuff, and it's giving out deprecation notice about this

explicitly set doctrine.orm.controller_resolver.auto_mapping: true as suggested by deprecation notice to keep existing behavior and remove the notice

see doctrine/DoctrineBundle#1762 for more info

@melroy89
Copy link
Member

melroy89 commented Apr 30, 2024

Actually if I read the documentation correctly, there is a good reason why Symfony is trying to disabled automapper by default. Maybe we should set it to false (default soon) as well as use the MapEntity configuration options instead.

In the next Symfony major release this automapper option will be set to false by default for a good reason after all.

Meaning I would like to see this option to be set to false and use MapEntity

it looks like doctrine bundle is about to deprecate
doctrine.orm.controller_resolver.auto_mapping and it's giving out
deprecation notice about this

explicitly set doctrine.orm.controller_resolver.auto_mapping: true as
suggested by deprecation log to keep existing behavior and remove the
notice

see doctrine/DoctrineBundle#1762 for more info
@asdfzdfj asdfzdfj force-pushed the fix/doctrine-orm-controller-auto-mapping-deprecation branch from 300c615 to 44520e8 Compare April 30, 2024 11:16
@asdfzdfj asdfzdfj merged commit 484acd6 into main Apr 30, 2024
7 checks passed
@asdfzdfj asdfzdfj deleted the fix/doctrine-orm-controller-auto-mapping-deprecation branch April 30, 2024 11:19
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

Successfully merging this pull request may close these issues.

None yet

3 participants