Resolve PHP 8.1 Deprecations

[natewiebe13]

This resolves php 8.1 deprecations in a backwards compatible way.

More ideally, return types should be added to function definitions, but this would no longer support PHP 5.6 as described in the README. If the team is open to bumping up version requirements to only include supported PHP versions (7.4+), I can revisit this to add return types instead.

[NelinD]

need merge

[stcheng]

Thanks for your contribution. We'll evaluate if we would like to bump the version requirements, but the PR looks good.

[facebook-github-bot]

@stcheng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[stcheng]

@natewiebe13 for this repository, it is only the generated files from the original source stored here. If you want to update the files, you shall submit the PR in the facebook-business-sdk-codegen repository. Thanks.

[NelinD]

@natewiebe13 Will you make it?

[natewiebe13]

@stcheng I only see 2 of the 11 files in the code gen repo:

• https://github.com/facebook/facebook-business-sdk-codegen/blob/main/templates/php/src/FacebookAds/Cursor.php
• https://github.com/facebook/facebook-business-sdk-codegen/blob/main/templates/php/src/FacebookAds/Object/AbstractCrudObject.php.versioned.mustache

Are the other files somewhere else? Or how should I proceed with the changes?

[natewiebe13]

I have also started a PR for those files here: https://github.com/facebook/facebook-business-sdk-codegen/pull/40

[stcheng]

@natewiebe13 thanks i'll check it out.

[natewiebe13]

@stcheng any updates on this?

[ujwaldhakal]

waiting for this PR https://github.com/facebook/facebook-business-sdk-codegen/pull/40 to get merged