[Stable9] fix file not found #128

Merged
merged 10 commits into from Oct 17, 2016

Projects

None yet

6 participants

@VicDeo
Member
VicDeo commented Aug 24, 2016 edited

Backport of
#111 #121 #113 #118 to stable9

Needs manual testing

VicDeo added some commits May 4, 2016
@VicDeo VicDeo Move table initialization to install script 8ec7827
@VicDeo VicDeo Fix savig rules 7013ad0
@VicDeo VicDeo Please codechecker && migrate tests to PSR-4
Fixes
84f0598
@VicDeo VicDeo Fix file not found 68f38c0
@VicDeo VicDeo Count only successful scan results in batch 74806a7
@VicDeo VicDeo Make var type more obvious
70f354b
@VicDeo VicDeo added this to the 9.0.5 milestone Aug 24, 2016
@mention-bot

@VicDeo, thanks for your PR! By analyzing the annotation information on this pull request, we identified @DeepDiver1975, @artem-sidorenko and @fsbruva to be potential reviewers

VicDeo added some commits Aug 24, 2016
@VicDeo VicDeo Remove illegal merge lefover
28b5b17
@VicDeo VicDeo Backport test fix
9abeb3d
@VicDeo VicDeo Update appinfo
847c42e
@VicDeo VicDeo Fix assignment in condition
8eeb2ae
@PVince81
Contributor
PVince81 commented Sep 7, 2016

@ms270169 are you able to help testing this OC 9.0 backport ?

@owncloud/qa

@ms270169
ms270169 commented Sep 8, 2016

If you would brief me about correct test setup, I could do that.
Because it's backporting to OC9.0, it should be tested together with OC9.0 system. Right?
I would download latest OC9.0 from release channel (at the moment 9.0.4) and replace app/filese_antivirus with commit 8eeb2ae.
Do you suggest a better way?

@PVince81 PVince81 modified the milestone: 9.0.6, 9.0.5 Sep 21, 2016
@VicDeo VicDeo referenced this pull request Oct 10, 2016
Merged

Fix file size check #121

@PVince81
Contributor

@owncloud/qa can you help testing this ? Might take some time to setup.

@SergioBertolinSG
Member

👍 Works.

@PVince81
Contributor

Thanks a lot! Merging.

@PVince81 PVince81 merged commit a9e6e1b into stable9 Oct 17, 2016

3 checks passed

Scrutinizer 3 new issues, 4 updated code elements
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details
@PVince81 PVince81 deleted the stable9-fix-file-not-found branch Oct 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment