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

Reworked driver exceptions #3505

Merged
merged 1 commit into from
Apr 5, 2019
Merged

Conversation

morozov
Copy link
Member

@morozov morozov commented Apr 5, 2019

Q A
Type improvement
BC Break yes

Besides the breaking changes necessitated by enforcing strict scalar types and adding return type hints, named constructors were added to DB2Exception and MysqliException.

@morozov morozov changed the title Reworking driver exceptions Reworked driver exceptions Apr 5, 2019
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Ocramius Ocramius self-assigned this Apr 5, 2019
@Ocramius Ocramius modified the milestones: 2.10.0, 3.0.0 Apr 5, 2019
@Ocramius Ocramius merged commit 361a2b6 into doctrine:develop Apr 5, 2019
@morozov morozov deleted the driver-exceptions branch April 5, 2019 16:51
2. `Doctrine\DBAL\Driver\PDOException` no longer extends `PDOException`.
3. The value returned by `Doctrine\DBAL\Driver\PDOException::getSQLState()` no longer falls back to the driver error code.

The method was used internally and is no longer needed.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be here, looks like a copy&paste error.

Copy link
Member Author

@morozov morozov Apr 5, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, incorrect conflict resolution. I'll remove it later.

@morozov morozov added the PDO label Apr 6, 2019
morozov pushed a commit that referenced this pull request Apr 16, 2019
morozov pushed a commit that referenced this pull request May 6, 2019
morozov pushed a commit that referenced this pull request May 23, 2019
morozov pushed a commit to morozov/dbal that referenced this pull request May 31, 2019
morozov pushed a commit to morozov/dbal that referenced this pull request May 31, 2019
morozov pushed a commit that referenced this pull request Jun 13, 2019
morozov pushed a commit that referenced this pull request Jun 27, 2019
morozov pushed a commit that referenced this pull request Jun 27, 2019
morozov pushed a commit that referenced this pull request Jun 27, 2019
morozov pushed a commit to morozov/dbal that referenced this pull request Aug 26, 2019
morozov pushed a commit that referenced this pull request Nov 2, 2019
@morozov morozov modified the milestones: 4.0.0, 3.0.0 Nov 17, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants