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

Update wrapper-mysql.php #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

afluegel9
Copy link

The order or arguments in the exception constructor is wrong. This is only visible, when mysqli is not available, thus the author quite likely never saw this for the user misleading error message:
PHP Fatal error: Uncaught TypeError: Exception::__construct(): Argument #2 ($code) must be of type int, string given in Standard input code:2
Stack trace:
#0 Standard input code(2): Exception->__construct()
#1 {main}
thrown in Standard input code on line 58

The order or arguments in the exception constructor is wrong. This is only visible, when mysqli is not available, thus the author quite likely never saw this for the user misleading error message:
PHP Fatal error:  Uncaught TypeError: Exception::__construct(): Argument kinghost#2 ($code) must be of type int, string given in Standard input code:2
Stack trace:
#0 Standard input code(2): Exception->__construct()
kinghost#1 {main}
  thrown in Standard input code on line 58
@fbraz3
Copy link
Contributor

fbraz3 commented Jul 6, 2022

Hi @afluegel9 i wrote this code when i was a kinghost employee and its abandonned as far as know.

In matter of fact this repo is a copy/paste from my personal project.
https://github.com/fbraz3/php5-compatibility-layer

Can you please PR to my repo?

Thanks in advance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants