Skip to content

🐛 Fix xml UTF8 encoding #496

Open
HamzaAlayed wants to merge 5 commits intointuit:masterfrom
HamzaAlayed:master
Open

🐛 Fix xml UTF8 encoding #496
HamzaAlayed wants to merge 5 commits intointuit:masterfrom
HamzaAlayed:master

Conversation

@HamzaAlayed
Copy link
Copy Markdown

@HamzaAlayed HamzaAlayed commented Aug 8, 2023

🐛 Use UTF8 encoding to fix some special characters in xml response and fix this error DOMDocument::loadXML(): PCDATA invalid Char .....

@HamzaAlayed HamzaAlayed changed the title Fix xml utf8 encoding 🐛 Fix xml UTF8 encoding Aug 8, 2023
…rface or enum named QuickBooksOnline\API\Core\HttpClients\Integer does not exist
@kkmuffme
Copy link
Copy Markdown

While this is possible, I don't think that's a good idea.

Why not encode it with UTF-8?

if (mb_detect_encoding($xml_string, 'UTF-8', true) === false) {
        $xml_string = utf8_encode($xml_string);
}

@HamzaAlayed
Copy link
Copy Markdown
Author

@hlu2 could you take a look?

@alirezaGAlizade alirezaGAlizade mentioned this pull request Nov 14, 2025
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