-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
Error showing in commands #128
Comments
@pushpendra07 Thanks for submitting this issue! I can't reproduce this on the latest version of the Cron Job Manager. Can you try upgrading to the latest version (v1.9.0). |
I can reproduce this on PHP 7.1, Magento 2.2.6, Cronjobmanager 1.9.0 I had a short look at the code, and it actually seems ok to me... so no idea where this error comes from. To reproduce:
All running in developer mode. |
@Ethan3600, It's showing same error for latest version |
Ok, I was on Magento version 2.3.1. That's why I didn't see it. Sorry for the mistake. Good News: I was able to reproduce on Magento version 2.2.6 Bad News: This seems to be a bug in some versions of Magento 2.2.x: magento/magento2#6106 There were more issues opened up for this, and I can't tell which release the fix was merged into. Anyway, the problem was this: The CJM's code had a method signature with a return type-hint like this: But, the generated code was spitting out something like this: Which causes php to throw an error because the signature doesn't match. I can confirm that this is fixed in the latest 2.2.x release (at the time of writing, it's v2.2.9). I'm going to close this due to it being a Magento bug. Feel free to comment to discuss further! |
Describe the bug
Showing error in Magento CLI commands
To Reproduce
Steps to reproduce the behavior:
Expected behavior

Commands should run without error
Screenshots
Environment (please complete the following information):
The text was updated successfully, but these errors were encountered: