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

File Download: Changing the list of selected files for a multipage file list results in bad download all behavior. #3711

Closed
kcondon opened this issue Mar 20, 2017 · 2 comments
Assignees
Labels
Feature: File Upload & Handling Type: Bug a defect User Role: Guest Anyone using the system, even without an account

Comments

@kcondon
Copy link
Contributor

kcondon commented Mar 20, 2017

Tested in production with 42 and 108 files, See RT 24178 for details.

If I change the selected file list after downloading then choose download (all) the file downloaded is not a zip file but one of the files on the list. There are at least two ways to reproduce this:

  1. Select all without scrolling to see other files. Download. Now scroll to see all files, then select all again so that all are selected. Choose download to get all files. Notice the file downloaded is one of the files, not a zip of all the files.

  2. Scroll to see all files, select all, download. Now, deselect one file (2nd from top is what I chose). Now download all and a single file is downloaded, not a zip of all files.

I believe downloading in between changing selected files is essential to reproducing this issue.

@mheppler
Copy link
Contributor

mheppler commented Feb 6, 2018

Related to Edit File Metadata: Select All Files in Tab, Can't Edit Them All after clicking Edit Metadata #2859, as well as to Initial ADA check: Primeface hidden checkboxes fail compatibility #2634. Investigate as part of Spike: Files DataTable, Selection, Paginator vs Scroll #4437.

@mheppler
Copy link
Contributor

Issue #4656, which removed the scrolling feature of the files table, was recently merged as part of 4.9.2. I tried to replicate this issue on demo, with a dataset containing 59 files. I was able to select all 59 files, download a ZIP of all the files, then unselect one file, and download a ZIP of the remaining 58 files.

Also, as part of #4656 a new dynamic counter and "Select all ## files in this dataset" text link was added to the top of the files table. This will help you know exactly how files have selected currently. Which comes in very handy as it pertains to the use case reported in this issue.

I suggest that we can close this issue. Will assign to QA and tag @kcondon to confirm and close.

@mheppler mheppler added Status: QA and removed UX & UI: Design This issue needs input on the design of the UI and from the product owner labels Aug 27, 2018
@kcondon kcondon self-assigned this Aug 28, 2018
@kcondon kcondon closed this as completed Aug 28, 2018
@djbrooke djbrooke added this to the 4.9.2 - Stata Upgrades, etc. milestone Aug 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: File Upload & Handling Type: Bug a defect User Role: Guest Anyone using the system, even without an account
Projects
None yet
Development

No branches or pull requests

4 participants