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

Neos.Media Tag class has incorrect doctrine mappings #3264

Closed
tvt opened this issue Jan 5, 2021 · 1 comment · Fixed by #3524
Closed

Neos.Media Tag class has incorrect doctrine mappings #3264

tvt opened this issue Jan 5, 2021 · 1 comment · Fixed by #3524
Assignees

Comments

@tvt
Copy link

tvt commented Jan 5, 2021

Description

The Tag class has two properties, children and parent. Their relationship is incorrectly mapped and causes doctrine validation to fail.

Steps to Reproduce

  1. Add Neos.Media 5.2 to a project
  2. Run ./flow doctrine:validate

Expected behavior

Validation should pass.

Actual behavior

$ ./flow doctrine:validate
Mapping validation FAILED!
  Neos\Media\Domain\Model\Tag
    The field Neos\Media\Domain\Model\Tag#children is on the inverse side of a bi-directional relationship, but the specified mappedBy association on the target-entity Neos\Media\Domain\Model\Tag#parent does not contain the required 'inversedBy="children"' attribute.

Affected Versions

Neos.Media version 5.2

@albe
Copy link
Member

albe commented Jan 5, 2021

cc @daniellienert

This should be easy to fix. I wonder why the mapping seems to work at runtime though?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants