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

Locked files: not enough infos for end user, too much for admin #49875

Open
5 of 8 tasks
Ornanovitch opened this issue Dec 15, 2024 · 0 comments
Open
5 of 8 tasks

Locked files: not enough infos for end user, too much for admin #49875

Ornanovitch opened this issue Dec 15, 2024 · 0 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 30-feedback enhancement feature: locking

Comments

@Ornanovitch
Copy link
Contributor

Ornanovitch commented Dec 15, 2024

⚠️ This issue respects the following points: ⚠️

Bug description

When someone tries to move/delete/edit a locked file, he/she gets an incomplete notification while admin get errors as if something were abnormal.

Steps to reproduce

  1. try to delete a locked file
  2. see the error popup "action failed: deleting file" (or something like that, my interface is in french)
  3. go to the logs and see the Locked Exception thrown: Sabre\DAV\Exception\Locked error

Expected behavior

  • End user should see a complete warning : "action failed: file is locked by [...]" (and it could be a simple warning like "this file is locked, you cannot {action} it", rather than an error)
  • Logs should not be spammed with this message which is not an error in the admin pov but a simple information about a user trying to edit a locked file, which is ok

Nextcloud Server version

30

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.3

Web server

Nginx

Database engine version

MariaDB

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other
@Ornanovitch Ornanovitch added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Dec 15, 2024
@solracsf solracsf added enhancement and removed bug labels Dec 17, 2024
@solracsf solracsf changed the title [Bug]: locked files: not enough infos for end user, too much for admin Locked files: not enough infos for end user, too much for admin Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 30-feedback enhancement feature: locking
Projects
None yet
Development

No branches or pull requests

4 participants