StatementInvalid takes WrappedDatabaseException's place #10372

Merged
merged 1 commit into from Apr 29, 2013

Conversation

Projects
None yet
3 participants
@jeremy
Member

jeremy commented Apr 29, 2013

StatementInvalid would be much more useful if it gave us access to the originating exception. Hoist that up from WrappedDatabaseException without breaking compatibility.

@carlosantoniodasilva

This comment has been minimized.

Show comment
Hide comment
@carlosantoniodasilva

carlosantoniodasilva Apr 29, 2013

Member

Looks good to me 👍

Looks good to me 👍

@rafaelfranca

This comment has been minimized.

Show comment
Hide comment
@rafaelfranca

rafaelfranca Apr 29, 2013

Member

:shipit:

Member

rafaelfranca commented Apr 29, 2013

:shipit:

jeremy added a commit that referenced this pull request Apr 29, 2013

Merge pull request #10372 from jeremy/statement-invalid-wraps-underly…
…ing-exception

StatementInvalid takes WrappedDatabaseException's place

@jeremy jeremy merged commit a91bde3 into rails:master Apr 29, 2013

@jeremy jeremy deleted the jeremy:statement-invalid-wraps-underlying-exception branch Apr 29, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment