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

Add missing DB rollback functionality #11767

Merged
merged 1 commit into from Oct 24, 2014

Conversation

Projects
None yet
5 participants
@danbartram
Copy link
Contributor

commented Oct 24, 2014

This is hopefully a fix for Issue #9802.
This is also an updated version of PR #11765.


Changes

  • Add a missing method to rollback database changes in a transaction.

This contribution is MIT licensed.

@ghost

This comment has been minimized.

Copy link

commented Oct 24, 2014

Thanks a lot for your contribution! Contributions to the core repo require a signed contributors agreement http://owncloud.org/contribute/agreement/ Alternatively you can add a comment here stating that this contribution is MIT licensed. Some more details about out pull request workflow can be found here: http://owncloud.org/code-reviews-on-github/

@ghost

This comment has been minimized.

Copy link

commented Oct 24, 2014

Thanks a lot for your contribution! Contributions to the core repo require a signed contributors agreement http://owncloud.org/contribute/agreement/ Alternatively you can add a comment here stating that this contribution is MIT licensed. Some more details about out pull request workflow can be found here: http://owncloud.org/code-reviews-on-github/

@LukasReschke

This comment has been minimized.

Copy link
Member

commented Oct 24, 2014

@owncloud-bot This is okay to test

@LukasReschke

This comment has been minimized.

Copy link
Member

commented Oct 24, 2014

@owncloud-bot Retest this please

@LukasReschke

This comment has been minimized.

Copy link
Member

commented Oct 24, 2014

@scrutinizer-notifier

This comment has been minimized.

Copy link

commented Oct 24, 2014

The inspection completed: 2 updated code elements

@DeepDiver1975

This comment has been minimized.

Copy link
Member

commented Oct 24, 2014

👍

@ghost

This comment has been minimized.

Copy link

commented Oct 24, 2014

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/1228/
🚀 Test PASSed. 🚀

@MorrisJobke

This comment has been minimized.

Copy link
Member

commented Oct 24, 2014

👍

MorrisJobke added a commit that referenced this pull request Oct 24, 2014

Merge pull request #11767 from danbartram/add_db_rollback
Add missing DB rollback functionality

@MorrisJobke MorrisJobke merged commit 63f2b1f into owncloud:master Oct 24, 2014

1 check passed

default Merged build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.