Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • I ISPConfig 3
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 579
    • Issues 579
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 48
    • Merge requests 48
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ISPConfig
  • ISPConfig 3
  • Issues
  • #4750
Closed
Open
Issue created Aug 05, 2017 by Ghost User@ghostContributor

Php7.1 Warning: Declaration of db::query($sQuery = '') should be compatible with mysqli::query($query, $resultmode = NULL)

Php7.1 Warning: Declaration of db::query($sQuery = '') should be compatible with mysqli::query($query, $resultmode = NULL) in /usr/local/ispconfig/server/lib/classes/db_mysql.inc.php

The above error has been reported at least twice for php7.1 in the forum (here and there).

They are both using php7.1 for ISPConfig "unknowingly" when the error was found but those who are testing php7.1 for ISPConfig may have already noticed this earlier.

I am testing a fix since I am also testing ISPConfig with php7.1 which is basically replacing all occurrences (11 altogether) of ($sQuery ='') to ($sQuery, $resultmode = MYSQLI_STORE_RESULT) in the said db_mysql.inc.php file. So far it manages to resolve the above issue but I am not so sure whether it won't cause others.

The modified file of db_mysql.inc.php is attached for reference and testing.

Do try and report as well.

Signed Ahmad Rasyid Ismail "ahrasis" ahrasis-at-gmail.com

Assignee
Assign to
Time tracking