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

[2.1] Settings... and select the user, you see all the files with 0 B #3986

Closed
Dianafg76 opened this issue Oct 22, 2015 · 5 comments

Comments

@Dianafg76
Copy link

commented Oct 22, 2015

Steps to reproduce

  1. Install the version 2.1 with user 1
  2. Right click on oC icon
  3. Click on Settings...
  4. Go to the user1

Expected behaviour

You should see the folders with the size

Actual behaviour

You see all the files with 0 B
screen shot 2015-10-22 at 10 16 43

Server configuration

Desktop v ownCloud-2.1.0.2821-nightly20151022.pkg

Server v {"installed":true,"maintenance":false,"version":"8.1.4.0","versionstring":"8.1.4 RC1","edition":""}

@Dianafg76 Dianafg76 added this to the 2.1-next milestone Oct 22, 2015

@guruz guruz changed the title When you install the new version and go to Setting... and select the user, you see all the files with 0 B [2.1] Settings... and select the user, you see all the files with 0 B Oct 22, 2015

@guruz

This comment has been minimized.

Copy link
Collaborator

commented Oct 22, 2015

Does the old version (from other computer or same computer) with same server show other numbers?
I very much think this is a server issue

@Dianafg76

This comment has been minimized.

Copy link
Author

commented Oct 22, 2015

@guruz, with Desktop v 2.0.2 (build 2818) is working OK
screen shot 2015-10-22 at 10 56 50

@Dianafg76

This comment has been minimized.

Copy link
Author

commented Oct 22, 2015

Server v {"installed":true,"maintenance":false,"version":"8.1.4.0","versionstring":"8.1.4 RC1","edition":""}

@guruz

This comment has been minimized.

Copy link
Collaborator

commented Oct 22, 2015

I can confirm this is a regression in 2.1/master, independant of the server.

@guruz guruz assigned guruz and ogoffart and unassigned guruz Oct 22, 2015

ogoffart added a commit that referenced this issue Oct 23, 2015

FolderStatusModel: fix getting the size of the folder in the selectiv…
…e sync (#3986)

Regressed since d610693. The problem
is that the _size vector contains the pathToRemove and that it was removed
before.
Reorganize a bit the code so there is only one loop that has still all the
 information.
@Dianafg76

This comment has been minimized.

Copy link
Author

commented Oct 26, 2015

I tested this issue, and now is working OK
Desktop v ownCloud-2.1.0.2829-nightly20151026.pkg
Server v {"installed":true,"maintenance":false,"version":"8.1.4.0","versionstring":"8.1.4 RC1","edition":""}

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.