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

Db connection query failure params #890

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

Chepheus
Copy link

Hello, sorry for my English
I need this params to display whats wrong was in query.

@svivian
Copy link
Collaborator

svivian commented Mar 30, 2021

Thanks for the PR. Could you explain a little more what this is for? The $params variable is not being used in your PR.

@Chepheus
Copy link
Author

It is need for more detailed logs if inserting was failed.
I create plugin for import array of data and I don't want to stop importing if one of element is broken (by default script stops).
I need this params to understand whats wrong with my data without deep debugging.

@Chepheus
Copy link
Author

Chepheus commented Mar 30, 2021

About $params - I override default function to function qa_page_db_fail_handler($type, $errno = null, $error = null, $query = null, array $params = null)

@Chepheus
Copy link
Author

Chepheus commented Apr 4, 2021

What do you think about this PR? =)

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.

3 participants