[stable9] Fix archive file name when downloading public share #22922

Merged
merged 2 commits into from Mar 9, 2016

Projects

None yet

5 participants

@PVince81
Collaborator
PVince81 commented Mar 7, 2016

Backport of #22902 to stable9

Please review @oparoz @MorrisJobke @schiesbn @rullzer

PVince81 added some commits Mar 7, 2016
@PVince81 PVince81 Fix archive file name when downloading public share
When download a public link share folder using the button on the top
right, it doesn't provide a list of files.

This fix makes sure to trigger the correct logic when no file list was
given.
f9e1d4d
@PVince81 PVince81 Remove "files" arg from download URL in public link page
When no files were specified for download, it means folder download.
In this case, no need to pass an empty "files" argument.
ebfc8b6
@PVince81 PVince81 added this to the 9.0.1-next-maintenance milestone Mar 7, 2016
@mention-bot

By analyzing the blame information on this pull request, we identified @LukasReschke, @nickvergessen and @rullzer to be potential reviewers

@oparoz
Contributor
oparoz commented Mar 7, 2016

👍

@MorrisJobke
Member

Tested and works 👍

@DeepDiver1975 DeepDiver1975 merged commit c2a16e0 into stable9 Mar 9, 2016

20 of 21 checks passed

core-ci-linux/database=oci,label=SLAVE Build #27707 failed in 1 hr 31 min
Details
cla-bot-core Build #2000 succeeded in 10 sec
Details
core-ci-linux-jsunit/database=sqlite,label=SLAVE Build #59026 succeeded in 40 sec
Details
core-ci-linux-swift-primary-storage/database=mysql,label=SLAVE Build #52990 succeeded in 16 min
Details
core-ci-linux/database=mysql,label=SLAVE Build #27707 succeeded in 37 min
Details
core-ci-linux/database=pgsql,label=SLAVE Build #27707 succeeded in 35 min
Details
core-ci-linux/database=sqlite,label=SLAVE Build #27707 succeeded in 19 min
Details
ocs-api-integration-tests-ci Build #8141 succeeded in 16 min
Details
server-master-linux-externals-ci/database=sqlite,external=smb-silvershell,label=SLAVE Build #7958 succeeded in 1 min 8 sec
Details
server-master-linux-externals-ci/database=sqlite,external=swift-ceph,label=SLAVE Build #7958 succeeded in 3 min 27 sec
Details
server-master-linux-externals-ci/database=sqlite,external=webdav-ownCloud,label=SLAVE Build #7958 succeeded in 5 min 28 sec
Details
server-master-linux-externals-smb-windows-ext-ci/database=sqlite,external=smb-windows,label=master Build #8319 succeeded in 2 min 27 sec
Details
server-master-linux-php5.4-ci/database=sqlite,label=SLAVE Build #312 succeeded in 4 min 45 sec
Details
server-master-linux-php7-ci/database=sqlite,label=SLAVE Build #35775 succeeded in 4 min 18 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=litmus,mirallBranch=v2.0.2,slave=SMASH Build #12323 succeeded in 3 min 0 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@0,mirallBranch=v2.0.2,slave=SMASH Build #12323 succeeded in 18 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@1,mirallBranch=v2.0.2,slave=SMASH Build #12323 succeeded in 26 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_shareLink,mirallBranch=v2.0.2,slave=SMASH Build #12323 succeeded in 6 min 46 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePermissions,mirallBranch=v2.0.2,slave=SMASH Build #12323 succeeded in 25 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationGroups,mirallBranch=v2.0.2,slave=SMASH Build #12323 succeeded in 16 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationInsideGroups,mirallBranch=v2.0.2,slave=SMASH Build #12323 succeeded in 8 min 49 sec
Details
@DeepDiver1975 DeepDiver1975 deleted the stable9-share-archivenamedownloadall branch Mar 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment