Please inform the customer of conversation # 1148586 when this conversation has been closed.

Please include this customer of conversation # 1174638 when this conversation has been closed.

Please inform the customer of conversation # 1185511 when this conversation has been closed.

Is this going to be fixed at some point given PHP 8.1 released in 2021 and the current version is 8.3.
It looks like a small adjustment is needed to the function call to resolve this.

+1 https://wordpress.org/support/topic/deprecation-warning-in-php-8-2/

Literally all that needs to be done to fix this is change line 49 to this:

return \urldecode( \trim( (string) \apply_filters( 'wpseo_canonical', $this->presentation->canonical, $this->presentation ) ) );

Note that the (string) cast that is currently on the trim() call is redundant because trim() can only return a string.

Hi @monkeyArms

Thank you for taking the time to check this issue and for sharing the fix! We truly appreciate your contribution and enthusiasm.

We would love for you to submit your fix as a Pull Request. It will be a great way to ensure your contribution gets properly reviewed and integrated into the project by our development team. You can find more details about contributing here.

I appreciate any help you can provide.

@josevarghese this notice generates an enormous amount of noise if tools like Sentry or similar are in place. We have a composer patch to patch this up in place, but would love it if it could make it upstream.

Hi @aanduque

Thanks for the comment and for taking the time to create a Pull Request! We appreciate your contribution and will look it over as soon as possible. Your efforts to improve the project mean a lot to us.

Please inform the customer of conversation # 1226199 when this conversation has been closed.