-
Notifications
You must be signed in to change notification settings - Fork 12
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
PHP8.2: Deprecated: Using ${expr} (variable variables) in strings is deprecated, use {${expr}} instead #59
Comments
Hi, |
echo "{${'var'.$i}}"; // works if ({${'var'.$i}}) { // don't works |
ok with quote |
I dont know if I have the same issue? did your installation run on php 8.2? or did it stop? |
Would you please share your code changes so others can use it? I have upgraded my Ubuntu20.04 to Ubuntu 22.04 and now my 123solar wont work because my php is now version 8.x. |
Just for completeness, I reinstalled my software to latest and it seems to work with php 8.1. |
BTW i use PHP 8.3.2 and it runs. |
Hi I decide to try and check update for this problem with the deprecated variable structure. What I have done is in php.ini enabled the error to be logged on the production value, that is you get on a single pass on the script the line numbers of the code that the error exists...then edit those as required. The text with error is appearing in the upper section of the displayed page, I make notes of the line numbers. When you again enter that page you do not see those errors (until a change in script or the next start up etc. This is for those trying to find a simple way of finding the line numbers....just so many lines :) I have been using 123 since 2013 and meterN since 2014 I think still going strong ...thanks Jeanmarc |
It turned out to be easier than I thought by doing this method. I have both 123solar and meterN no longer having PHP report errors. |
Recently I installed another instance of 123solar in a host where is running apache and php 8.2.
In php 8.2 the use of ${expr} is deprecated, and 123solar uses frequently this syntax. In my installation I corrected it, but it would be nice if this change will be implemented also in the main branch.
The text was updated successfully, but these errors were encountered: