PHP 8.1 DomDocument ceateElements throwing error

Managed to reproduce this, and it's probably due to the code creating empty packages for example, among other things.

So yeah... we can track it here, but this has to be fixed in the library:

See https://github.com/gabrielbull/php-ups-api/issues/364.

Another option would be for us to fork the library, and use ours instead.

+1 for forking and moving forward without errors

Maintainer of gabrielbull/php-ups-api seems to be no longer working on the project. There hasn't been any activity for more than a year.
We need to fork it. Drupal 10 requires at least PHP 8.1

There's also https://github.com/wpdesk/php-ups-api which is being actively maintained but their branch diverged from the main repo a while ago and there's a lot of differences between them. I don't know enough about the codebase to tell if those differences are important or not. The code does appear to be used in production though. Might be worthwhile swapping them and see what breaks?

In the meantime anyone here needing a stopgap fix, you can apply this patch to gabrielbull/php-ups-api using composer to get PHP 8.1 compatibility.

Here is how you can apply this patch since it's a vendor project:

"gabrielbull/ups-api": {
"php 8.1 compatibility /project/commerce_ups/issues/3326304": "patches/gabrielbull-3326304-fix-php81-depreciations.patch"
}

i was able to add the patch like this :

"gabrielbull/ups-api": {
                "php 8.1 compatibility /project/commerce_ups/issues/3326304": "https://www.drupal.org/files/issues/2023-11-06/fix-php81-depreciations.patch"
            }