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

Better exception models #29

Open
g105b opened this issue Feb 3, 2017 · 1 comment
Open

Better exception models #29

g105b opened this issue Feb 3, 2017 · 1 comment

Comments

@g105b
Copy link
Member

g105b commented Feb 3, 2017

Rather than throwing the uncaught PDOException, make a better model which relates to the actual problem in the SQL.

We already have QueryCollectionNotFoundException and QueryNotFoundException, when dealing with the actual filesystem, but it would be better to have the PDOExceptions wrapped in a more readable manner.

@g105b g105b added this to the v2 milestone Feb 3, 2017
@g105b
Copy link
Member Author

g105b commented Nov 6, 2021

An example of one exception that still needs to be improved:

PHP Fatal error: Uncaught PDOException: SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in /path/to/script.php

What variables are in the script? What variables have been given? Why not tell the developer which ones are missing?

@g105b g105b removed this from the v2 milestone Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant