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

Fix: Additional HrefTransform validation for $type and $id #434

Conversation

patkul0
Copy link
Contributor

@patkul0 patkul0 commented Feb 8, 2024

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no

Add validation for $type and $id parameters because they are required by the Pimcore element service which has strict_types declaration.

By default, FormBuilder sets these two parameters as null for empty languages. For example, because of that, the user was not able to load configuration that contains the Snippet type field.

@solverat solverat added the bug label Feb 9, 2024
@solverat solverat added this to the 5.0.4 milestone Feb 9, 2024
@solverat solverat merged commit 2e032ca into dachcom-digital:master Feb 9, 2024
2 of 3 checks passed
@solverat
Copy link
Member

solverat commented Feb 9, 2024

Thanks, @patkul0!

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

Successfully merging this pull request may close these issues.

2 participants