Closed warcooft closed 4 months ago
The recent updates enhance the allOAuth
method in the ShieldOAuth
class by directly instantiating a new Autoload
object, shifting from a pre-configured instance. This change could impact dependency management depending on the state of Autoload
during calls. Additionally, adjustments for static analysis and coding standards have been implemented, promoting a smoother development experience.
Files | Change Summary |
---|---|
.../Basic/ShieldOAuth.php | Updated allOAuth method to instantiate Autoload using new Autoload() instead of retrieving it via config(Autoload::class) , altering instance management and control flow. |
.../Generators/NewShieldOauthGenerator.php | Added a comment to the run method to instruct PHPStan to ignore the following line, clarifying static analysis without affecting the logic. |
.php-cs-fixer.dist.php | Added 'modernize_strpos' => false to the $overrides array, preventing automatic modernization of strpos usage in the codebase. |
🐇 In the code where changes bloom,
A new instance brings fresh room.
Autoload hops in with glee,
Each call anew, just wait and see!
Onward we code, let logic flow,
With every tweak, our skills will grow! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@warcooft thanks!
Description
See https://github.com/codeigniter4/CodeIgniter4/pull/9052?notification_referrer_id=NT_kwDOAV4NwrQxMTU5MDI1MjU3MzoyMjk0MTEyMg#issuecomment-2241101064
Summary by CodeRabbit
Bug Fixes
Enhancements
strpos
function, allowing for consistent usage patterns.