owncloud / client

🖥️ Desktop Syncing Client for ownCloud
GNU General Public License v2.0
1.4k stars 662 forks source link

When server has external storage the folders to sync dialog times out waiting for the server reply #3524

Closed phil-davis closed 9 years ago

phil-davis commented 9 years ago

owncloud-too-big-folder-01

Expected behaviour

After the end-user of the client is notified that "There are new shared folders that were not synchronized because they are too big", it should be obvious to an end user what to do next and how to choose to synchronize the new folder/s.

Actual behaviour

I click the Apply button in the screenshot. Then I cannot find where to go to choose which folders to sync. I am an IT guy, if I can't work out where to find the button to edit which folders are synced then I am sure our users will not be able to find it :) In this case I am using multi account and have 2 accounts connected up to the same server - but I suspect this will not be part of the issue here.

Steps to reproduce

  1. On account A client set "Ask confirmation before downloading folders larger than" to some low value (e.g. I did 5MB)
  2. On account B on the server share some folder >5MB with account A
  3. On account A client the "too big" notification is received, good. Click Apply.
  4. Now you want that folder to be synced - try to work out how to get account A to sync it - I can't.

    Server configuration

Operating system:

Web server:

Database:

PHP version:

ownCloud version: 8.0

Storage backend:

Client configuration

Client version: 2.0.0-nightly 20150731 (build 5317)

Operating system: Windows7

OS language: English (US)

Installation path of client: default Windows path

Logs

guruz commented 9 years ago

@phil-davis Thanks for testing 2.0 :) I'll keep this open for now until someone also tested the 2.1 code..

Dianafg76 commented 9 years ago

I tested this issue and on Desktop v ownCloud-2.1.0.5586-nightly20151028-setup.exe it is working the same that v 2.0.2

But whe the account is sync, on the Folder Dropbox you can see the message "Fetching folder list from server ...."

screen shot 2015-10-28 at 15 49 32

screen shot 2015-10-28 at 15 49 44

phil-davis commented 9 years ago

For me, it shows "Fetching folder list from server..." when opening the whole folder sync tree view, and when opening sub-folders in the tree. But each time, when the server reply is received, the message "Fetching folder list from server..." goes away and the tree is displayed nicely. That is the expected behavior. @Dianafg76 How did you get the message to be "stuck" on the display?

guruz commented 9 years ago

@Dianafg76 Can you re-test and wait longer?

Dianafg76 commented 9 years ago

But now when you sync the folder Dropbox, the total B is 0 and is not true screen shot 2015-11-04 at 09 23 48

Desktop v ownCloud-2.1.0.2851-nightly20151104.pkg Server v {"installed":true,"maintenance":false,"version":"8.1.4.0","versionstring":"8.1.4 RC1","edition":""}

guruz commented 9 years ago

@Dianafg76 Can you create an issue in core for the wrong dropbox size and assign to @icewind1991 ? There is possibly one already

phil-davis commented 9 years ago

As far as I am concerned, the issue here about the folders to sync tree view not opening has been resolved by: 1) Adding a progress message on the client UI. 2) Increasing the timeout so the client waits longer. 3) Apparent improvements in the server response time between server 8.0.* (which my hosted server was on when I first reported) and server 8.1.* (which it is running now) to the PROPFIND when there are external storages.

As @guruz says, the (0 B) folder sizes on the display are a separate issue.

guruz commented 9 years ago

:+1:

Dianafg76 commented 9 years ago

OK