Skip to content

bug: SoapCallRegistry::prettyXML avoid E_WARNING with no xml content#38

Merged
ivoba merged 1 commit into
freshcells:mainfrom
gimler:feature/fix_warning_none_xml_content
Jun 1, 2026
Merged

bug: SoapCallRegistry::prettyXML avoid E_WARNING with no xml content#38
ivoba merged 1 commit into
freshcells:mainfrom
gimler:feature/fix_warning_none_xml_content

Conversation

@gimler

@gimler gimler commented May 29, 2026

Copy link
Copy Markdown
Contributor

When the response is not xml content in our case pdf file. We getting an php warning.

Warning: DOMDocument::loadXML(): Start tag expected, '<' not found in Entity, line: 2

@gimler

gimler commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

@ivoba can you take a look please

@ivoba ivoba merged commit 0fc73ab into freshcells:main Jun 1, 2026
5 checks passed
@gimler gimler deleted the feature/fix_warning_none_xml_content branch June 1, 2026 13:09
@gimler

gimler commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

@ivoba thanks. can you also create a new tag please?

@ivoba

ivoba commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

lgtm, tagged as 4.0.1 @gimler 👍

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.

2 participants