-
-
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
Argument should be array, boolean given #141
Comments
That's a new one @bzneil good catch. Can you give me a little more information as to how to reproduce this? Magento version, php version, steps to reproduce, etc. Are you running this locally? |
Running on a hosted server, |
I bet Magento switched their API. In here: https://github.com/Ethan3600/magento2-CronjobManager/blob/1.x/Ui/Component/Listing/DataProviders/Cronjobmanager/Config/Grid.php#L156 we rely on a $job to be an associative array, but it gave us a boolean (based on the error). So, it looks like [
'group_name' => [
'job_name' => false // or true
]
] Instead of something like this: [
'group_name' => [
'job_name' => [
'name' => 'magento_cron_name',
'group' => 'default',
...
]
]
] We probably need to figure out why we're seeing a boolean there instead of an array and find a way to handle this situation to prevent it from causing this error. (As a side note, this would make for a great "first time contribution" issue. Maybe I'll make a label or something) |
When I try to load job configuration page I get this:
Argument 1 passed to EthanYehuda\CronjobManager\Helper\JobConfig::sanitizeJobConfig() must be of the type array, boolean given, called in vendor/ethanyehuda/magento2-cronjobmanager/Ui/Component/Listing/DataProviders/Cronjobmanager/Config/Grid.php on line 156
Any ideas?
The text was updated successfully, but these errors were encountered: