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
Make Matomo PHP 7.3 compatible #13418
Comments
Not a PHP error, it tries to write -5 into an db field that is unsigned mostlikely. Why it tries to write -5 is a interesting question nethertheless. Also note that your request contains 2 token_auths, that is quite unusual I would say.. |
Okay, I have now updated the server to the latest release candidate and tried around and couldn't really find more issues. For the first two I'll create a pull request soon once tedious/JShrink#80 is fixed. @fdellwing I agree that this is probably a really obscure visitor generator bug, so I have removed it here. Interestingly everything around PHP generation still works fine (where I expected most bugs). As PHP7.3 just migrated to Debian testing, I'll expect it to be the default version in the next Debian stable (will be released in about a year) and ubuntu versions. Therefore probably a lot of people will be using it in a year. |
Okay, I mixed a bit up regarding JShirk. We are using a more than 4 year old beta version of it. Does anyone have an idea what's the easiest way to fix this? |
If the current version is actually still compatible with 5.5, but the dependencies were just changed in the composer.json we could overwrite it with a custom package in composer.json to overwrite the php requirement. e.g.:
|
@sgiehl Thanks for the tip. |
Got another one:
Need to check if that will be fixed with #13566 |
#11936 is still open. Shouldn't this be resolved first? |
Please also clarify this in the docs. Currently it just says "PHP version 5.5.9 or greater". |
@sgiehl could you do any hot fix for the tcpdf issue? |
@sebash1992 this should be fixed in the upcoming 3.8.0 |
Replace continue with break resolved the issue. |
@Findus23 since JShrink was updated, is this issue done? |
@diosmosis If no one else reports any more issues, I'd say this is done. |
In 3 months PHP 7.3 will be released, so let's start making Matomo compatible!
I tried setting up a new Matomo instance using PHP 7.3 RC2 (so all bugs found now will probably also be in the final release) and most things are working fine. These errors occurred:
Stack Trace
from https://github.com/tedious/JShrink
also from https://github.com/tedious/JShrink
matomo/plugins/LanguagesManager/Commands/CreatePull.php
Lines 214 to 217 in b8e4d27
Stack Trace
Stack Trace
To be continued...
If you want to, I can create test-admin accounts on the php 7.3 instance.
The text was updated successfully, but these errors were encountered: