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

[Bug] JS Errors on Bricks with many-to-many relation and specific configuration #406

Open
aKugler opened this issue Jan 26, 2024 · 2 comments · May be fixed by #462
Open

[Bug] JS Errors on Bricks with many-to-many relation and specific configuration #406

aKugler opened this issue Jan 26, 2024 · 2 comments · May be fixed by #462
Assignees
Milestone

Comments

@aKugler
Copy link

aKugler commented Jan 26, 2024

Hi!

A user creates a new brick in the Admin UI. It contains a FieldContainer and a Fieldset. The fieldset contains an (advanced) many-to-many relation.
objectbrick_Test_export.json
You can then also add corresponding objects in the relation.

Bildschirmfoto 2024-01-26 um 09 34 33

User 2 then has the idea of dragging and dropping the fieldset into the FieldContainer. At that moment, the bricks are no longer displayed, but the browser console is full of JS errors.
Bildschirmfoto 2024-01-26 um 09 34 57
Bildschirmfoto 2024-01-26 um 09 35 19
objectbrick_Test_export-3.json

Pimcore 11.1.4
Admin UI 1.3.1

Best regards
Andreas

@wisconaut wisconaut added this to the 1.3.4 milestone Feb 12, 2024
@wisconaut wisconaut added the Bug label Feb 15, 2024
@exalate-issue-sync exalate-issue-sync bot removed the Bug label Feb 15, 2024
@kingjia90 kingjia90 modified the milestones: 1.3.4, 1.3.5 Feb 19, 2024
@wisconaut wisconaut added the Bug label Feb 26, 2024
@kingjia90 kingjia90 assigned robertSt7 and unassigned kingjia90 Mar 12, 2024
@robertSt7 robertSt7 modified the milestones: 1.3.5, 1.3.6 Mar 12, 2024
@kingjia90 kingjia90 modified the milestones: 1.3.6, 1.4.1 Mar 12, 2024
@robertSt7
Copy link
Contributor

Hi @aKugler thanks for reporting this bug. The problem here is that it shouldn't be possible to put a fieldset into a field container. I will provide a fix for the drag and drop validation

@aKugler
Copy link
Author

aKugler commented Mar 13, 2024

Hi @robertSt7 !
Thank you very much, that sounds good. It's just important to me that no one else stumbles across it. I wouldn't have expected these effects. Probably also because I was working on import and data enrichment and didn't build the data model.

@robertSt7 robertSt7 modified the milestones: 1.4.1, 1.4.2, 1.4.3 Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants