Update file.php post hook calls #26387

Merged
merged 1 commit into from Oct 19, 2016

Projects

None yet

3 participants

@karakayasemi
Contributor
karakayasemi commented Oct 16, 2016 edited

Description

Related Issue

Motivation and Context

How Has This Been Tested?

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.

Logicaly, postHooks should emit after touch. For chunking file it is already emitting after touch.

@karakayasemi karakayasemi Update file.php post hook calls
Logicaly, postHooks should emit after touch. For chunking file it is already emitting after touch.
e2b09e1
@mention-bot

@karakayasemi, thanks for your PR! By analyzing the history of the files in this pull request, we identified @icewind1991, @rullzer and @PVince81 to be potential reviewers.

@PVince81
Collaborator

Indeed, makes sense 👍

@PVince81 PVince81 added this to the 9.2 milestone Oct 17, 2016
@PVince81
Collaborator

Kicked CI to retest

@PVince81
Collaborator

Tests passed, unpublished due to API rate limit => merge

@PVince81 PVince81 merged commit fecde40 into owncloud:master Oct 19, 2016

3 of 4 checks passed

continuous-integration/jenkins/pr-head This commit is being built
Details
Scrutinizer No new issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details
@PVince81
Collaborator

@DeepDiver1975 I suggest backport to stable9.1 at least, any objection?

@PVince81
Collaborator
PVince81 commented Dec 7, 2016

stable9.1: #26793

@karakayasemi karakayasemi deleted the karakayasemi:patch-2 branch Dec 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment