yii2-authclient
yii2-authclient copied to clipboard
Get rid of deprecation warnings for rawurlencode() and strpos() (#380)
| Q | A |
|---|---|
| Is bugfix? | ✔️ |
| New feature? | ❌ |
| Breaks BC? | ❌ |
| Fixed issues | #380 |
"Deprecated" warning now showed for trim(), but it should be fixed in UrlManager.php (another repo)
devenv:
- PhpStorm 2023.3.1
- PHP 8.3.1
- Xdebug v3.3.1
PR Summary
- Improved Null Handling in 'OAuth1.php'
In the file 'OAuth1.php', a series of changes have been made to ensure the system does not encounter unexpected problems if some values are missing or undefined. This involves using a null coalescing operator to flag certain items as empty strings if they're not present. The improved handling includes:
- The URL
- The method
- The result of
http_build_query - The consumer secret
- The token secret
This results in a more robust behavior when these values aren't provided, reducing the risk of system errors and promoting smoother system operation overall.
@g41797 do you want to fix the rest of compatibility issues? See tests failing.
"Deprecated" warning (showed for trim()) are fixed by PR#20090
Merged that one.