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

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

Closed
TheOneRing opened this issue Jan 14, 2021 · 16 comments · Fixed by #10983 or #11028
Closed

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

TheOneRing opened this issue Jan 14, 2021 · 16 comments · Fixed by #10983 or #11028
Assignees
Labels
p4-low Low priority
Milestone

Comments

@TheOneRing
Copy link
Contributor

TheOneRing commented Jan 14, 2021

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.

@github-actions
Copy link

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 github-actions bot added the Stale label Feb 16, 2021
@gabi18
Copy link
Contributor

gabi18 commented Feb 16, 2021

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
Copy link
Contributor

gabi18 commented Feb 16, 2021

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

@github-actions github-actions bot removed the Stale label Feb 17, 2021
@github-actions
Copy link

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.

@gabi18
Copy link
Contributor

gabi18 commented Apr 6, 2021

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)
  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

@github-actions
Copy link

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 github-actions bot added the Stale label May 27, 2021
@TheOneRing TheOneRing added p4-low Low priority and removed Stale labels May 27, 2021
@gabi18
Copy link
Contributor

gabi18 commented Jul 4, 2022

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
Copy link
Contributor

gabi18 commented Jul 4, 2022

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

@TheOneRing TheOneRing added this to the 4.0 milestone Jul 5, 2022
@fmoc
Copy link
Contributor

fmoc commented Jun 23, 2023

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

TheOneRing added a commit that referenced this issue Jun 28, 2023
Instead of pausing the current sync we now abort the sync, ask the user and start a second sync

Fixes: #8360
TheOneRing added a commit that referenced this issue Jun 28, 2023
Instead of pausing the current sync we now abort the sync, ask the user and start a second sync

Fixes: #8360
@TheOneRing TheOneRing changed the title Remove all dialog: Keep files will make the dialog pop up again and again "Remove all dialog": Keep files will make the dialog pop up again and again Jun 29, 2023
TheOneRing added a commit that referenced this issue Jun 29, 2023
Instead of pausing the current sync we now abort the sync, ask the user and start a second sync

Fixes: #8360
TheOneRing added a commit that referenced this issue Jul 5, 2023
Instead of pausing the current sync we now abort the sync, ask the user and start a second sync

Fixes: #8360
TheOneRing added a commit that referenced this issue Jul 5, 2023
Instead of pausing the current sync we now abort the sync, ask the user and start a second sync

Fixes: #8360
@nabim777
Copy link
Member

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

@Salipa-Gurung
Copy link
Contributor

Salipa-Gurung commented Jul 10, 2023

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. ✔️

  • 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. ✔️

      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 ✔️
  • Remove All Files dialog appears only once ✔️

@Salipa-Gurung Salipa-Gurung reopened this Jul 11, 2023
@TheOneRing
Copy link
Contributor Author

It makes sense @michaelstingl

TheOneRing added a commit that referenced this issue Jul 11, 2023
@TheOneRing TheOneRing linked a pull request Jul 11, 2023 that will close this issue
TheOneRing added a commit that referenced this issue Jul 11, 2023
@Salipa-Gurung
Copy link
Contributor

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. ✔️

  • In case of dehydrated file,

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

    • Remove all files button removes the files ✔️

    • 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 ✔️
  • Remove All Files dialog appears only once ✔️

@Salipa-Gurung Salipa-Gurung reopened this Jul 12, 2023
@TheOneRing
Copy link
Contributor Author

@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.

@Salipa-Gurung
Copy link
Contributor

so, can it be closed then?

@michaelstingl
Copy link
Contributor

so, can it be closed then?

yes, close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p4-low Low priority
Projects
None yet
7 participants