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 type errors with XML export on PHP 8.1 #9589

Merged
merged 1 commit into from
Mar 19, 2022

Conversation

vrestihnat
Copy link
Contributor

Please see PHP interface SimpleXMLElement::addAttribute(string $name, string $value = null, string $namespace = null): void ....
The $value must be string or null.

Please see PHP interface SimpleXMLElement::addAttribute(string $name, string $value = null, string $namespace = null): void .... 
The $value must be string or null.
@greg0ire
Copy link
Member

Blocked by #9590

@greg0ire greg0ire closed this Mar 18, 2022
@greg0ire greg0ire reopened this Mar 18, 2022
@derrabus derrabus added this to the 2.11.3 milestone Mar 19, 2022
@derrabus derrabus changed the title Update XmlExporter.php - Type problem in php8.x Fix type errors with XML export on PHP 8.1 Mar 19, 2022
@derrabus derrabus merged commit c6831c6 into doctrine:2.11.x Mar 19, 2022
derrabus added a commit that referenced this pull request Mar 28, 2022
* 2.11.x:
  PHPStan 1.5.0 (#9607)
  Remove Sphinx config
  Use correct syntax for external links
  Update XmlExporter.php - Type problem in php8.x (#9589)
  Ignore deprecation from Persistence
  Stands with Ukraine (#9567)
  Use internal links when self-referencing
  Link to docs for the stable version
derrabus added a commit to derrabus/orm that referenced this pull request Mar 28, 2022
* 2.12.x:
  Update psalm.xml
  PHPStan 1.5.0 (doctrine#9607)
  Remove Sphinx config
  Indicate what feature is deprecated
  Implement int-mask-of where appropriate
  Use correct syntax for external links
  Update XmlExporter.php - Type problem in php8.x (doctrine#9589)
  Ignore deprecation from Persistence
  Stands with Ukraine (doctrine#9567)
  Use internal links when self-referencing
  Link to docs for the stable version
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.

3 participants