googleads / googleads/googleads-php-lib

xml reader error for php 8.2

Open
#786 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
PHP
Stars
676
Forks
754
PR merge metrics
No merged PRs in 30d

Description

Hi All,

We are using the google ad manager PHP library to push order and line items to GAM from our platform.
We had PHP version 8.1 and GAM API version v202302. We were able to push the order and line item successfully to GAM and get the response back properly.

However we moved from PHP version 8.1 to 8.2 and then when we tried to push to GAM we received error as -
Warning: XMLReader::XML(): Unable to load source data in .../googleads/googleads-php-lib/src/Google/AdsApi/Common/Util/SoapHeaders.php on line 75

We had not made any code changes in our platform apart from the PHP version upgrade to 8.2 from 8.1. Can you please help us out and let us know how we can move forward so that we can use PHP 8.2.

Please let me know if more information is needed.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at src/Google/AdsApi/Common/Util/SoapHeaders.php line 75 and reproduce the XMLReader warning with PHP 8.2 using the described order or line-item request. Compare the same request under PHP 8.1 and inspect the XML source available at the failure point. Done means the request completes under PHP 8.2 without the XML reader error.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
api
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.