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

[Bug]: oc_file_metadata preservation issues with trashbin #34424

Closed
6 of 9 tasks
PVince81 opened this issue Oct 4, 2022 · 0 comments · Fixed by #41634
Closed
6 of 9 tasks

[Bug]: oc_file_metadata preservation issues with trashbin #34424

PVince81 opened this issue Oct 4, 2022 · 0 comments · Fixed by #41634
Assignees
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap feature: trashbin technical debt

Comments

@PVince81
Copy link
Member

PVince81 commented Oct 4, 2022

⚠️ This issue respects the following points: ⚠️

  • This is a bug, not a question or a configuration/webserver/proxy issue.
  • This issue is not already reported on Github (I've searched it).
  • Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
  • Nextcloud Server is running on 64bit capable CPU, PHP and OS.
  • I agree to follow Nextcloud's Code of Conduct.

Bug description

Metadata is lost after deleting and restoring a picture.

Steps to reproduce

  1. Upload a picture
  2. Check oc_files_metadata and see that its metadata was retrieved (ex: size, exif, etc)
  3. Delete the file so it goes to trash
  4. Check metadata in database again
  5. Restore file from trash

Expected behavior

At step 3 the metadata should stay in the database.
At step 5 it should be back at least.

Installation method

No response

Operating system

No response

PHP engine version

No response

Web server

No response

Database engine version

No response

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

No response

Are you using the Nextcloud Server Encryption module?

No response

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

N/A

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

It is likely that the wrong event is used for metadata deletion.

We should use one of the "removed from cache" event to purge the metadata of a file instead of the "deleted" event.

cc @CAR

@PVince81 PVince81 added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Oct 4, 2022
@AndyScherzinger AndyScherzinger moved this to 📄 To do (~10 entries) in 📁 Files team Aug 3, 2023
@github-project-automation github-project-automation bot moved this from 📄 To do (~10 entries) to ☑️ Done in 📁 Files team Nov 21, 2023
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 feature: trashbin technical debt
Projects
Archived in project
4 participants