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

Add support for Phalcon 5 #128

Closed
destyk opened this issue Dec 27, 2021 · 1 comment
Closed

Add support for Phalcon 5 #128

destyk opened this issue Dec 27, 2021 · 1 comment
Assignees
Labels
NFR New Feature Request

Comments

@destyk
Copy link

destyk commented Dec 27, 2021

Currently I have PHP 8.0, Phalcon 5.0.0-alpha7 and phalcon-migration.phar version 2.2.4 installed on my server.

My config file looks like this:

use Phalcon\Config;

return new Config([
    //...
]);

When I start generating the migration, I get the response:

Phalcon Migrations                                                                                                                                                     
                                                                                                                                                                       
Fatal Error: Class "Phalcon\Config" not found

But if I change the Phalcon\Config class to Phalcon\Config\Config (which is supported in Phalcon 5), I get this response:

Phalcon Migrations                                                                                                                                                     
                                                                                                                                                                       
Fatal Error: Phalcon\Migrations\Console\Commands\Migration::loadConfig(): Return value must be of type Phalcon\Config, Phalcon\Config\Config returned

Hopefully Phalcon 5 support will be added soon!

@Jeckerson Jeckerson added the NFR New Feature Request label Dec 27, 2021
@Jeckerson Jeckerson self-assigned this Jun 11, 2022
@Jeckerson
Copy link
Member

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

No branches or pull requests

3 participants