Walkthrough

The pull request removes dependency injection of the Authorization class from multiple router callback functions in app/controllers/general.php. Three callback blocks have been updated: the main router callback, the automatic certificate generation App::init() block, and the main App::init() router group callback. Concurrent with removing the injected parameter, instance method calls to Authorization have been replaced with static method calls (disable() and reset()).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Single file affected with consistent, repetitive refactoring pattern (dependency injection removal)
• Three similar code blocks updated using the same pattern
• No new business logic introduced; purely structural changes in dependency access pattern
• Consider verifying that Authorization::disable() and Authorization::reset() static methods exist and function equivalently to their instance counterparts

• 📝 Generate docstrings

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

• X
• Mastodon
• Reddit
• LinkedIn