Skip to content

PHP 8.1 - DOMDocument::createTextNode - #19

Closed
Juwebweb wants to merge 1 commit into
nullivex:masterfrom
Juwebweb:patch-1
Closed

Juwebweb wants to merge 1 commit into
nullivex:masterfrom
Juwebweb:patch-1

Conversation

@Juwebweb

@Juwebweb Juwebweb commented May 12, 2022 •

Copy link
Copy Markdown

Dear Array2XML people,

I found an issue as I upgraded to PHP 8.1, the following exception came up from line 174:
DOMDocument::createTextNode(): Passing null to parameter 1 ($data) of type string is deprecated

It might have to do with the following, from https://www.php.net/manual/en/domdocument.createtextnode
"Version 8.1.0 - In case of an error, a DomException is thrown now. Previously, false was returned."

I found a quick fix that works just fine in my situation...
Please have a look, it might be the correct way!

Thank you,

Dear Array2XML people,

I found an issue as I upgraded to PHP 8.1, the following exception came up from line 174:
DOMDocument::createTextNode(): Passing null to parameter 1 ($data) of type string is deprecated

It may has to do with the following from https://www.php.net/manual/en/domdocument.createtextnode
"Version 8.1.0 - In case of an error, a DomException is thrown now. Previously, false was returned."

I found a quick fix that works just fine in my situation...
Please have a look, it might be the correct way! 

Thank you,
@Juwebweb Juwebweb closed this by deleting the head repository Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant