Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(autoload): autoload needs to happen before use of use. #1546

Merged
merged 4 commits into from
Feb 26, 2025

Conversation

noone-silent
Copy link

Hello!

  • Type: bug fix
  • Link to issue:

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR

Small description of change:

Before the usage of vendor/bin/phalcon raised an exception:

www-data@ad3f5c1317e5:/app$ vendor/bin/phalcon
PHP Fatal error:  Uncaught Error: Class "Phalcon\DevTools\Script\Color" not found in /app/vendor/phalcon/devtools/phalcon:72
Stack trace:
#0 {main}
thrown in /app/vendor/phalcon/devtools/phalcon on line 72

Because the autoload happened after the use of use statements.

Thanks

@niden niden merged commit 345be5b into phalcon:5.0.x Feb 26, 2025
0 of 4 checks passed
@niden
Copy link
Member

niden commented Feb 26, 2025

Thank you @noone-silent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants