Skip to content

Conversation

@sharidas
Copy link
Contributor

A custom wrapper to handle command transfer-ownership.
In this wrapper we can make changes for functions like
fopen or any other routines.

Signed-off-by: Sujith H sharidasan@owncloud.com

Description

A custom wrapper created along with a custom view so that changes required for common functions like fopen arguments or so can be handled here. Primarily the focus is for command transfer-ownership and later we would use for other command like recreate master key.

Related Issue

Motivation and Context

This change would prevent the modification to common routines which were made to arguments of functions like fopen or copy etc. And use this wrapper for custom changes.

How Has This Been Tested?

  • Scratch installation of oC and create 2 users
  • Verified transfer ownership command works with and without encryption between two users.
  • Verified Recreate command for master key works on the same setup.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@sharidas sharidas added this to the development milestone Aug 22, 2017
A custom wrapper to handle command transfer-ownership.
In this wrapper we can make changes for functions like
fopen or any other routines.

Signed-off-by: Sujith H <sharidasan@owncloud.com>
@sharidas sharidas force-pushed the customwrapper-rework branch from 943e733 to 0048a78 Compare August 22, 2017 12:59
@PVince81
Copy link
Contributor

Obsolete ? Please close and link to your new PR

@sharidas
Copy link
Contributor Author

Obsolete PR. The final PR is : #28774

@sharidas sharidas closed this Aug 28, 2017
@PVince81 PVince81 deleted the customwrapper-rework branch September 27, 2018 13:36
@lock lock bot locked as resolved and limited conversation to collaborators Sep 27, 2019
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.

4 participants