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
Several PHP 7.4 fixes #15222
Several PHP 7.4 fixes #15222
Conversation
On a fresh 3.13.0 install with php 7.4.0 following error occurs when tracker does track a page view, when using Mysqli as database driver. `Notice: Trying to access array offset on value of type int in /srv/0x.at/matomo/core/Tracker/Db/Mysqli.php on line 50` This fix this error
This error appears, this change fixes this issue `DataTable/Filter/ColumnDelete.php(164): Deprecated - array_key_exists(): Using array_key_exists() on objects is deprecated. Use isset() or property_exists() instead - Matomo 3.11.0`
@@ -161,7 +161,7 @@ protected function removeColumnsFromTable(&$table) | |||
} | |||
foreach ($this->columnsToRemove as $column) { | |||
|
|||
if (!array_key_exists($column, $row)) { | |||
if (!property_exists($column, $row)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you maybe undo this change? We will investigate this as part of #15210
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, done.
As suggested here https://github.com/matomo-org/matomo/pull/15222/files/cba4f22e12a23ec392f45decceedc49e5753eea5#diff-a5f249b6134f80011f165be56c5995b1 This wasn't a proper fix, i got other errors after applying that.
Awesome, thanks for quick change. Very appreciated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to wait for tests before merging though
On a fresh 3.13.0 install with php 7.4.0 following error occurs when tracker does track a page view, when using Mysqli as database driver.
Notice: Trying to access array offset on value of type int in /srv/0x.at/matomo/core/Tracker/Db/Mysqli.php on line 50
This fix this error