-
Notifications
You must be signed in to change notification settings - Fork 119
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
Uncaught RuntimeException #1224
Comments
FYI this is on a site that does not run woocommerce but I don't know much about the action scheduler so let me know if this is the wrong place to share this |
Hi @jeckman. This sounds like a problem in which the alter table wp_actionscheduler_actions
add priority tinyint default 10 not null; Or, if you have access to WP CLI, you could try running: wp action-scheduler fix-schema The latter may be easier/safer, but it also depends on the WP database user having the necessary privileges to make the required change, which may not be the case. Normally this isn't required but, again, it seems like it was not possible for Action Scheduler to perform the necessary update itself. |
Thanks Barry - will do. I guess my question was even if a table ends up without the right columns, shouldn't an exception be thrown and caught? |
Adding the column worked |
In this case, that probably would make sense, yes, and we have a separate issue that outlines the need to review how these situations are handled: |
Also opened a thread in all-in-one-seo-pack support forum, but I think at least the exception catching ought to be happening in this action scheduler?
PHP Fatal error: Uncaught RuntimeException: Error saving action: Error saving action: Unknown column 'priority' in 'field list' in /wp-content/plugins/all-in-one-seo-pack/vendor/woocommerce/action-scheduler/classes/migration/ActionScheduler_DBStoreMigrator.php:44
Not sure what version of action-scheduler is included in AIOSEO but the AIOSEO version is 4.7.6
The text was updated successfully, but these errors were encountered: