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.

Sign up for GitHub

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

Jump to bottom

[stable19] Fix total upload size overwritten by next upload #24855

Merged
merged 1 commit into from
Dec 28, 2020
Merged

[stable19] Fix total upload size overwritten by next upload #24855

merged 1 commit into from
Dec 28, 2020

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #24833

@danxuliu @backportbot
Fix total upload size overwritten by next upload
acdf895 
The upload progress is based on the "totalToUpload" variable. However,
as the variable is set when an upload is submitted, if another upload is
submitted before the previous one finished the upload progress only took
into account the size of the new upload (although the upload itself
worked fine; the files of the new submitted upload are added to the
active one). Now "totalToUpload" is either increased or set depending on
whether an upload is active or not.

Note that although "data.total" holds the total size of the files being
uploaded "totalToUpload" needs to be used in "fileuploadprogressall"
instead; "totalToUpload" is calculated when the upload is submitted, but
since 7c4c5fe the actual upload of the files, and thus updating the
value of "data.total", may be deferred until the parent folders were
created.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 19.0.7 milestone Dec 28, 2020
danxuliu
danxuliu approved these changes Dec 28, 2020
View reviewed changes
Copy link
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@rullzer rullzer merged commit e1a1854 into stable19 Dec 28, 2020
@rullzer rullzer deleted the backport/24833/stable19 branch December 28, 2020 13:17
@rullzer rullzer mentioned this pull request Jan 5, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danxuliu danxuliu danxuliu approved these changes

@juliusknorr juliusknorr juliusknorr approved these changes

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

@rullzer rullzer Awaiting requested review from rullzer

@skjnldsv skjnldsv Awaiting requested review from skjnldsv

Assignees
No one assigned
Labels
bug feature: files regression
Projects
None yet
Milestone
Nextcloud 19.0.7
Development

Successfully merging this pull request may close these issues.
3 participants
@juliusknorr @danxuliu @rullzer