Conversation
📝 WalkthroughWalkthroughThe changes add an LDAP extension availability check to the application. A new error handler method in PanicAttack displays a 500 error if LDAP is missing, and a pre-flight check in index.php invokes this handler before normal initialization. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
bootstrap/PanicAttack.php (1)
133-140: Inconsistent brace placement and extra blank line.The opening brace should be on a new line to match the rest of the file (PSR-12 style). Also, there's an extra blank line at the end.
🧹 Proposed style fix
- public function ldap() { + public function ldap() + { $this->title = 'Missing PHP Extension'; $this->code = 500; $this->message = 'The PHP extension "ldap" is required but not loaded. Please install and enable it to run Lychee.'; $this->displaySimpleError(); } - -As per coding guidelines: "Run
vendor/bin/php-cs-fixer fixto apply PHP code style fixes before committing."
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
Summary by CodeRabbit