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

Removing .owncloud extension deletes content of a file #7338

Closed
HanaGemela opened this issue Jul 24, 2019 · 1 comment
Closed

Removing .owncloud extension deletes content of a file #7338

HanaGemela opened this issue Jul 24, 2019 · 1 comment
Assignees
@ckamm
Labels
p2-high Escalation, on top of current planning, release blocker ReadyToTest QA, please validate the fix/enhancement type:bug
Milestone
2.6.0

Comments

@HanaGemela
Copy link
Contributor

Client: 2.6.0alpha2 (build 12128)
macOS: 10.14.5
Server: 10.2.1. (stable)

Steps to recreate:

  1. Turn on VFS
  2. Rename test.txt to test.txt.owncloud -> file has been replaced by virtual file OK
  3. Double click on the file -> not possible/doesn't do anything BAD
  4. Remove .owncloud extension
  5. Open the file

Actual result: File is empty, the content of the file is lost
Expected result: It has to be possible to replace virtual and physical files just by renaming as described here #6876 (comment)
@HanaGemela HanaGemela added type:bug p2-high Escalation, on top of current planning, release blocker labels Jul 24, 2019
@HanaGemela HanaGemela added this to the 2.6.1 milestone Jul 24, 2019
@HanaGemela HanaGemela mentioned this issue Jul 24, 2019
Closed
@ckamm ckamm self-assigned this Jul 24, 2019
@ckamm ckamm modified the milestones: 2.6.1, 2.6.0 Jul 24, 2019
@HanaGemela HanaGemela mentioned this issue Jul 24, 2019
Closed
ckamm added a commit that referenced this issue Jul 24, 2019
@ckamm
Vfs: Remove old db record when dehydrating via rename
a581130 
For #7338
@ckamm ckamm mentioned this issue Jul 24, 2019
Merged
ckamm added a commit that referenced this issue Jul 25, 2019
@ckamm
Vfs: Remove old db record when dehydrating via rename
4fd9bdc 
For #7338
@ckamm ckamm added ReadyToTest QA, please validate the fix/enhancement and removed PR available labels Jul 25, 2019
@HanaGemela
Copy link
Contributor Author

Works as expected on macOS 10.14.6, client 2.6.0beta1 (build 12241)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ckamm ckamm

Labels
p2-high Escalation, on top of current planning, release blocker ReadyToTest QA, please validate the fix/enhancement type:bug
Projects
None yet
Milestone
2.6.0
Development

No branches or pull requests
2 participants
@ckamm @HanaGemela