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

Trashbin fails with unknown error when restoring #139

Closed
jnweiger opened this issue Jul 15, 2020 · 9 comments
Closed

Trashbin fails with unknown error when restoring #139

jnweiger opened this issue Jul 15, 2020 · 9 comments

Comments

@jnweiger
Copy link

On the web gui of owncloud/ocis:1.0.0-beta8 :

  • upload a file
  • delete the file
  • upload the same file again
  • go to trashbin, see the the copy of the file there
  • restore from the trashbin, an error occurs:

image

Expected behaviour:

  • the error message should give the reason "cannot restore in the original location XXX, a file with the same name is already there."
  • and/or (even better) suggest to rename things to resolve that conflict, then restore.
@jnweiger
Copy link
Author

Same as above, but do empty trash:

image

Expected behaviour: Empty trash works.

@individual-it
Copy link
Member

@micbar micbar transferred this issue from owncloud/ocis Jul 28, 2020
@micbar micbar added bug Something isn't working p3-medium C1-oCIS labels Jul 28, 2020
@exalate-issue-sync exalate-issue-sync bot changed the title Trashbin fails with unknown error Trashbin fails with unknown error when restoring Aug 17, 2020
@butonic
Copy link
Member

butonic commented Aug 19, 2020

fixed by cs3org/reva#1103

@PVince81
Copy link

it's part of owncloud/ocis-reva#447.

@individual-it @phil-davis any tests to enable on ocis-reva to validate this ?

@individual-it
Copy link
Member

tests added in owncloud/core#37831 & owncloud/ocis#482
not many passing, need more investigation

@individual-it
Copy link
Member

bug demonstration tests added in owncloud/ocis#489

@individual-it
Copy link
Member

phoenix tests added in owncloud/web#3982 , taking off QA-team tag

@exalate-issue-sync
Copy link

Artur Neumann commented: current behaiviour is changed, the original file is overwritten without comment. see #196

@jnweiger
Copy link
Author

retested with

Version details ocis-1.0.0-rc1
OCIS_VERSION:         v1.0.0-rc1
ocis --version:       ocis version 199267c^M
git log:              commit 199267c13e916aa8de027a81dbf6733af820369f (HEAD, tag: refs/tags/v1.0.0-rc1)
eos --version:        EOS 4.6.5 (2019)^M
xrootd -v:            v4.11.0^M
bin/ocis contains:
        owncloud/ocis-accounts@v0.4.2-0.20200901074457-6a27781a2741
        owncloud/ocis-glauth@v0.5.1-0.20200731165959-1081de7c60f1
        owncloud/ocis-graph-explorer@v0.0.0-20200210111049-017eeb40dc0c
        owncloud/ocis-graph@v0.0.0-20200318175820-9a5a6e029db7
        owncloud/ocis-hello@v0.1.0-alpha1.0.20200828085053-37fcf3c8f853
        owncloud/ocis-konnectd@v0.3.2
        owncloud/ocis-migration@v0.2.0
        owncloud/ocis-ocs@v0.3.1
        owncloud/ocis-phoenix@v0.13.0
        owncloud/ocis-pkg/v2@v2.4.1-0.20200828095914-d3b859484b2b
        owncloud/ocis-proxy@v0.7.1-0.20200904132806-fcceec602fcb
        owncloud/ocis-reva@v0.13.0
        owncloud/ocis-settings@v0.3.2-0.20200902094647-35dc3aeaba78
        owncloud/ocis-store@v0.1.1
        owncloud/ocis-thumbnails@v0.3.0
        owncloud/ocis-webdav@v0.1.1

Restore works fine in RC1. No error messages when the target name is free. Appropriate errors when the target file or directory exists. Nothing is overwritten.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants