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

add $isXsdValidationEnabled to SimplifiedXmlDriver constructor #10579

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

dmaicher
Copy link
Contributor

This adds the missing $isXsdValidationEnabled argument to the SimplifiedXmlDriver constructor.

Will enable us to fix this deprecation on DoctrineBundle.

@dmaicher dmaicher marked this pull request as ready for review March 13, 2023 19:20
@ostrolucky ostrolucky added this to the 2.14.2 milestone Mar 14, 2023
@ostrolucky ostrolucky added the Bug label Mar 14, 2023
@ostrolucky ostrolucky merged commit abb3009 into doctrine:2.14.x Mar 14, 2023
@dmaicher dmaicher deleted the simplified_xml_driver branch March 14, 2023 08:59
@eisberg
Copy link

eisberg commented Apr 12, 2023

After this change we get an error:

ibxml error: Element '{http://gediminasm.org/schemas/orm/doctrine-extensions-mapping}timestampable': No matching global element declaration available, but demanded by the strict wildcard.
in src/Resources/config/doctrine/UserGroupRelation.orm.xml at line 21

@ro0NL
Copy link

ro0NL commented Apr 13, 2023

ref #10626

AFAIK it's a doctrine/orm issue, not a bundle one

either way ... please fix :) this was quite a patch release: 2.14.1...2.14.2

@greg0ire
Copy link
Member

Yeah quite big lol… we should really release more often. Anyway, as mentioned elsewhere, this is not an ORM issue, it's a bundle one. 2.14.2 merely reveals the latent issue. If you downgrade the bundle to 2.8.x, it will disappear.

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.

6 participants