client icon indicating copy to clipboard operation
client copied to clipboard

Remove all dialog: Keep files will make the dialog pop up again and again

Open TheOneRing opened this issue 4 years ago • 8 comments

After clicking on keep files, a new dialog pops up which asks to keep all files or remove all.

Might have been a special scenario, local db was deleted and all files on the remote where deleted.

TheOneRing avatar Jan 14 '21 13:01 TheOneRing

This issue was marked stale because it has been open for 30 days with no activity. Remove the stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Feb 16 '21 00:02 github-actions[bot]

Couldn't reproduce the issue that the 'Remove all' dialog popups again and again.

  • deleted all folder and files on server, clicked on 'Keep files' -> the popup didn't reappear
  • deleted all folder and files locally, same as above

gabi18 avatar Feb 16 '21 15:02 gabi18

Probably related to #8442 :

When testing to delete all both on server and locally the popup also appears once. Clicking on 'Keep files' cannot restore the files but they are still visible in the client.

20210216_1612_owncloud.log.0.gz

20210216_1611_owncloud.log.1.gz

all_deleted

gabi18 avatar Feb 16 '21 15:02 gabi18

This issue was marked stale because it has been open for 30 days with no activity. Remove the stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Mar 21 '21 00:03 github-actions[bot]

I could reproduce the issue on Windows 10 with VFS enabled.

Steps done:

  1. Enable VFS support in client
  2. Delete all folder/files on server
  3. The 'Keep files / Remove files' popup appears
  4. Click 'Keep files' Result: virtual folder/files are kept locally - no attempt to resync (see #8442)
  5. Quit owncloud-client and restart
  6. 'Keep files / Remove files' popup appears Result: client tries to resync after clicking 'Keep files' but cannot succeed (because of VFS) and the popup appears again and again

gabi18 avatar Apr 06 '21 12:04 gabi18

This issue was marked stale because it has been open for 30 days with no activity. Remove the stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar May 27 '21 00:05 github-actions[bot]

I could reproduce the issue on Windows 10 with VFS enabled.

Steps done:

1. Enable VFS support in client

2. Delete all folder/files on server

3. The 'Keep files / Remove files' popup appears

4. Click 'Keep files'
   Result: virtual folder/files are kept locally - no attempt to resync (see [[QA] No re-sync from client to server after 'Keep files'  #8442](https://github.com/owncloud/client/issues/8442))

NOT relevant here (for Windows VFS On), the issue above was reported for Linux (VFS Off)

5. Quit owncloud-client and restart

No need to restart on Windows, the popup appears again and again after clicking 'Keep files'. If the files which should be kept are virtual there is no chance to keep them.

6. 'Keep files / Remove files' popup appears
   Result: client tries to resync after clicking 'Keep files' but cannot succeed (because of VFS) and the popup appears again and again

gabi18 avatar Jul 04 '22 08:07 gabi18

Seen with ownCloud-3.0.0-daily20220622.7851.x64.msi on Windows 10 21H2:

  • the popup appears again and again when clicking "Keep files"

keep-files

  • "Remove all files" stops the endless loop but only removes the dehydrated files and keeps the hydrated

hydrated-files

keep-files.log.gz

gabi18 avatar Jul 04 '22 08:07 gabi18

TODO:

  • abort current sync and pause sync root before displaying dialog
  • if keep files: keep current behavior (removing all hacks from the code), start new sync to upload the files again
  • if remove all files: remove all files and start a new sync cycle

fmoc avatar Jun 23 '23 13:06 fmoc

Seen with ownCloud-5.0.0.11398--daily20230707 b0734d (download link) on Windows 10 Enterprise Evaluation :

  • the popup appears again and again when clicking "Keep files"

Screenshot from 2023-07-10 12-10-04

nabim777 avatar Jul 10 '23 07:07 nabim777

Tested on: Windows10 ownCloud version: ownCloud 5.0.0.11398-daily20230707

Deleting all the files only on the server

  • In case of hydrated and Always keep on this device files, both Keep files and Remove all files option works and Remove All Files dialog appears only once. :heavy_check_mark:

  • In case of dehydrated files:

    • Keep files option does not work (VFS on) ❌

    • Remove All Files dialog appears again and again. ❌

    • Remove all files button removes all the files. :heavy_check_mark:

      ownCloud.log

Screenshot from 2023-07-10 17-02-18 Screenshot from 2023-07-10 17-02-36

Deleting all the folders (empty folders) on the server

  • In case of dehydrated, hydrated and Always keep on this device folders both Keep files and Remove all files option works accordingly :heavy_check_mark:
  • Remove All Files dialog appears only once :heavy_check_mark:

Salipa-Gurung avatar Jul 10 '23 10:07 Salipa-Gurung

It makes sense @michaelstingl

TheOneRing avatar Jul 11 '23 07:07 TheOneRing

Client version

ownCloud 5.0.0.11438-daily20230712 0f63d2 Libraries Qt 6.4.3, OpenSSL 3.1.1 30 May 2023 Using virtual files plugin: wincfapi windows-10.0.19045

Deleting all the files only on the server

  • In case of hydrated and Always keep on this device files, both Keep files and Remove all files option works and Remove All Files dialog appears only once. :heavy_check_mark:

  • In case of dehydrated file,

    • Remove All Files dialog does not appear again and again.

    • Remove all files button removes the files :heavy_check_mark:

    • Keep files button does not keep the file ❌ [ instead it removes the dehydrated file ]

      ownCloud.log

Dehydrated files inside folder also gets deleted when keep files button clicked. ❌

Deleting all the folders (empty folders) on the server

  • In case of dehydrated, hydrated and Always keep on this device folders both Keep files and Remove all files option works accordingly :heavy_check_mark:
  • Remove All Files dialog appears only once :heavy_check_mark:

Salipa-Gurung avatar Jul 12 '23 10:07 Salipa-Gurung

@Salipa-Gurung @michaelstingl this is expected behavior. At least if I understand the scenario correctly. You have dehydrated placeholders locally and all files are removed on the server, this means the virtual files do no longer exist. We have to remove the local placeholders as they are already gone.

TheOneRing avatar Jul 12 '23 10:07 TheOneRing

so, can it be closed then?

Salipa-Gurung avatar Jul 17 '23 07:07 Salipa-Gurung

so, can it be closed then?

yes, close

michaelstingl avatar Jul 18 '23 10:07 michaelstingl