-
Notifications
You must be signed in to change notification settings - Fork 2
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
Stream DB tables cannot be trusted to exist #128
Comments
This has been added to the |
@roborourke I've observed it on a local Altis 8 environment after restoring a production database, from an environment where |
Looks like @tomjn's PR xwp/stream#1286 (which creates these tables if they don't exist when a user visits wp-admin) has been merged, so this should be a non-issue after the next time we upgrade our |
Hm, it's possible it was run on v8 before the 8.0.1 patch was released... It shouldn't be possible now but guess I'm still missing something 🤔 |
v3.8.1 is out which contains my change xwp/stream#1284 (comment), it looks like this issue was introduced in either 3.7 or 3.8 to fix an issue wth excessive |
Altis v8+ tracks XWP stream ^3.7.0 so a composer update can get 3.8.2. |
Closing out as Altis v7 is back on Stream 3.4 on the HM fork. |
The Stream plugin only creates its tables on activation, and the way it is used in this project, that step will never necessarily be hit. This means that (especially in local environments) you can end up in a situation where you have no DB tables for stream, and your errors are littered with warnings about it.
Steps to reproduce:
composer server cli -- db clean
and reinstall your local install, or import a databaseI would expect to see no errors of this type, because I would expect the Altis Security module to orchestrate ensuring the tables are present if they did not previously exist.
As a stopgap for now, you can manually recreate these tables by invoking the stream plugin activation directly:
Acceptance criteria:
Shout-out to @tomjn for figuring this one out, this issue documents discussion we just had in Slack. Tom has opened a related issue on the upstream Stream repo (two of them, actually), but this is something we could fix at this level, too.
The text was updated successfully, but these errors were encountered: