Skip to content

Disable automapping and adding explicit entity config#59

Merged
wachterjohannes merged 3 commits into
php-task:masterfrom
mamazu:explicit_doctrine_mapping
May 19, 2026
Merged

Disable automapping and adding explicit entity config#59
wachterjohannes merged 3 commits into
php-task:masterfrom
mamazu:explicit_doctrine_mapping

Conversation

@mamazu
Copy link
Copy Markdown
Contributor

@mamazu mamazu commented Apr 29, 2026

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets -
Related issues/PRs sulu/sulu#8530
License MIT

What's in this PR?

Adding explicit mapping for doctrine entities.

@alexander-schranz
Copy link
Copy Markdown
Member

@wachterjohannes is there a target branch for the version supported by automation bundle 2.x? https://github.com/sulu/SuluAutomationBundle/blob/13071beb9974958c557109d4a748ffe2273ef726/composer.json#L18-L19

@mamazu think we can then set auto mapping to false in the tests:

?

@alexander-schranz
Copy link
Copy Markdown
Member

alexander-schranz commented May 19, 2026

the used github actions image is not longer available and so the CI does not run: https://github.blog/changelog/2025-01-15-github-actions-ubuntu-20-runner-image-brownout-dates-and-other-breaking-changes/#ubuntu-20-image-is-closing-down

we should update to 22.04 or 24.04 ubuntu the CI

Reference: sulu/sulu#7615

@wachterjohannes wachterjohannes force-pushed the explicit_doctrine_mapping branch from a02c09a to 51db440 Compare May 19, 2026 13:00
Comment thread src/DependencyInjection/TaskExtension.php Outdated
Co-authored-by: Alexander Schranz <alexander@sulu.io>
@alexander-schranz alexander-schranz changed the title Disabling automapping and adding explicit entity config Disable automapping and adding explicit entity config May 19, 2026
@wachterjohannes wachterjohannes merged commit 6dc8f1e into php-task:master May 19, 2026
11 checks passed
@mamazu mamazu deleted the explicit_doctrine_mapping branch May 19, 2026 14:20
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.

3 participants