fix: correct PohodaSer autoload path and add package dependency#45
Conversation
The autoload template referenced /usr/share/php/PohodaSer/autoload.php but php-vitexsoftware-pohodaser installs to /usr/share/php/Pohoda/ (matching its Pohoda\ namespace). This caused a fatal require_once failure on production. Also declares php-vitexsoftware-pohodaser as an explicit package dependency. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR updates Debian packaging configuration for the PHP Pohoda Connector. The PHP autoloader now requires the Pohoda autoloader instead of PohodaSer, and package dependencies are adjusted to include ChangesDebian Packaging Configuration Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
Summary
debian/autoload.phpline 9:PohodaSer/autoload.php→Pohoda/autoload.phpto match the actual install path ofphp-vitexsoftware-pohodaser(which uses thePohoda\namespace)php-vitexsoftware-pohodasertoDependsindebian/controlto declare the dependency explicitlyProblem
Production server (
se-zabbix-proxy) was throwing:The mServer autoload template referenced
PohodaSer/butphp-vitexsoftware-pohodaserinstalls its autoload toPohoda/(matching itsPohoda\PHP namespace per Debian PHP packaging conventions).Test plan
/usr/share/php/mServer/autoload.phpcontainsrequire_once '/usr/share/php/Pohoda/autoload.php'php-vitexsoftware-pohodaserpresent and confirm no fatal errorspohoda-raiffeisenbankscripts load cleanly🤖 Generated with Claude Code
Summary by CodeRabbit