-
Notifications
You must be signed in to change notification settings - Fork 67
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
Compatibility issue with php7.4 #59
Comments
Thats really strange, i`m using also php7.4 without issues. |
Indeed, perhaps I have settings in php which do not allow for these types of error, or my specific install triggers $args[0] == null. |
It would be because the |
I reverted the change to retrigger the error, and now I'm not getting any error messages. I do not know what is happening. The original issue was
|
Making that a bit more readable:
Given the path this has followed, I can only assume that the error was caused by |
Hi Thomas, thank you for the answer. Yes it's possible, I would even say it's probable. I had not yet pulled this year's changes when the bug was triggered initially. I then implemented the fix above, and starting cherry picking commits - not noticing that the commit you linked might be fixing the issue. |
Hi, after upgrading to php7.4, I was having issues with php errors of the type
Uncaught ErrorException: Trying to access array offset on value of type null in
the file core.php at line 100.I therefore changed
to
Its not very clean, but I don't really know php !
Hope it helps others having the same issue. I can create a merge request if this is acceptable.
The text was updated successfully, but these errors were encountered: