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

Please review the php var definition code in QgisProject.php and WMSRequest.php in 3.6.4 as in 3.7.0-alpha2 #3809

Closed
1 task done
Antoviscomi opened this issue Jul 28, 2023 · 5 comments · Fixed by #3810 or #3811
Closed
1 task done
Assignees
Labels
bug php Pull requests that update Php code

Comments

@Antoviscomi
Copy link
Contributor

Antoviscomi commented Jul 28, 2023

What is the bug?

php ./console.php wmts:capabilities cetraro Cetraro
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Project/QgisProject.php on line 1924
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Request/WMSRequest.php on line 798
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Request/WMSRequest.php on line 801
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Request/WMSRequest.php on line 801
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Request/WMSRequest.php on line 967
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Request/WMSRequest.php on line 967
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Request/WMSRequest.php on line 967
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/lizmap/lizmap/modules/lizmap/lib/Request/WMSRequest.php on line 967

Steps to reproduce the issue

substitute ${var} with {$var} to fix that errors

Versions

Versions :

  • Lizmap Web Client : 3.6.4
  • Lizmap plugin : 3.15.1
  • QGIS Desktop : 3.32.1
  • QGIS Server : 3.32.1
  • Py-QGIS-Server : not used
  • QGIS Server plugin DataPlotly : 4.0.3
  • QGIS Server plugin atlasprint : 3.3.2
  • QGIS Server plugin lizmap_server : 2.8.0
  • QGIS Server plugin wfsOutputExtension : 1.7.1

Check Lizmap plugin

  • I have done the step just before in the Lizmap QGIS desktop plugin before opening this ticket. Otherwise, my ticket is not considered valid and might get closed.

QGIS server version, only if the section above doesn't mention the QGIS Server version

3.32.0

Operating system

Ubuntu 22.04

Browsers

Chrome

Browsers version

Chrome 115.0.5790.102

Relevant log output

No response

@Antoviscomi Antoviscomi changed the title Please review the php vas definition code in QgisProject.php WMSRequest.php in 3.6.4 as in 3.7.0-alpha2 Please review the php var definition code in QgisProject.php WMSRequest.php in 3.6.4 as in 3.7.0-alpha2 Jul 28, 2023
@Gustry
Copy link
Member

Gustry commented Jul 28, 2023

Can you make the PR ? You can make the change straight on GitHub.

@Gustry Gustry added the php Pull requests that update Php code label Jul 28, 2023
@Antoviscomi
Copy link
Contributor Author

@Gustry I'm not able to edit the code on Github, I need to fork the repository?

@Gustry
Copy link
Member

Gustry commented Jul 28, 2023

You can, if you don't have enough permissions, it will create the fork for you automatically. It's what you did #3810 now, thanks

@Antoviscomi
Copy link
Contributor Author

@Gustry i hope i did well the PR

@Antoviscomi Antoviscomi changed the title Please review the php var definition code in QgisProject.php WMSRequest.php in 3.6.4 as in 3.7.0-alpha2 Please review the php var definition code in QgisProject.php and WMSRequest.php in 3.6.4 as in 3.7.0-alpha2 Jul 28, 2023
@laurentj laurentj reopened this Jul 30, 2023
@laurentj laurentj self-assigned this Jul 30, 2023
@laurentj
Copy link
Collaborator

All patchs backported manually into Lizmap 3.5 and 3.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug php Pull requests that update Php code
Projects
None yet
3 participants