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

Piwik\Plugins\CoreUpdater\SystemSettings::updateToUtf8mb4 can be null #16002

Closed
diosmosis opened this issue May 29, 2020 · 0 comments · Fixed by #16003
Closed

Piwik\Plugins\CoreUpdater\SystemSettings::updateToUtf8mb4 can be null #16002

diosmosis opened this issue May 29, 2020 · 0 comments · Fixed by #16003
Assignees
Labels
not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Milestone

Comments

@diosmosis
Copy link
Member

Piwik\Plugins\CoreUpdater\SystemSettings::updateToUtf8mb4 can be null in some cases, but it is used directly in CoreUpdater\Tasks::schedule causing an error in some cases.

@diosmosis diosmosis added the Regression Indicates a feature used to work in a certain way but it no longer does even though it should. label May 29, 2020
@diosmosis diosmosis added this to the 4.0.0 milestone May 29, 2020
@sgiehl sgiehl self-assigned this May 29, 2020
sgiehl added a commit that referenced this issue May 29, 2020
@sgiehl sgiehl linked a pull request May 29, 2020 that will close this issue
@tsteur tsteur added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. and removed Regression Indicates a feature used to work in a certain way but it no longer does even though it should. labels May 30, 2020
@sgiehl sgiehl mentioned this issue Jun 3, 2020
tsteur pushed a commit that referenced this issue Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants