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]: v27.1.6 - Deleted files view (files_trashbin) does not show deleted files #43134

Closed
5 of 8 tasks
Gummikavalier opened this issue Jan 25, 2024 · 11 comments · Fixed by #43136
Closed
5 of 8 tasks
Labels

Comments

@Gummikavalier
Copy link

⚠️ This issue respects the following points: ⚠️

Bug description

When you delete any file, all previously deleted files disappear from the Deleted files view.
The files remain on the disk but it is not possible to make them visible.

This issue started at Nextcloud 27.1.6. It did not exist yet in 27.1.5.

Steps to reproduce

  1. Delete any file in Nextcloud
  2. Check the Deleted files view

Expected behavior

The deleted file should be visible under Deleted files.

Installation method

Community Manual installation with Archive

Nextcloud Server version

27

Operating system

RHEL/CentOS

PHP engine version

PHP 8.1

Web server

Apache (supported)

Database engine version

MariaDB

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

Updated from a MINOR version (ex. 22.1 to 22.2)

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

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

Configuration report

{
    "system": {
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "dbtype": "mysql",
        "version": "27.1.6.2",
        "dbname": "***REMOVED SENSITIVE VALUE***",
        "dbhost": "***REMOVED SENSITIVE VALUE***",
        "dbtableprefix": "oc_",
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "installed": true,
        "theme": "",
        "maintenance": false,
        "mail_smtpmode": "smtp",
        "mail_smtphost": "***REMOVED SENSITIVE VALUE***",
        "loglevel": 2,
        "logtimezone": "Europe\/Helsinki",
        "trusted_domains": [
            "redacted from the output"
        ],
        "secret": "***REMOVED SENSITIVE VALUE***",
        "share_folder": "\/Shared",
        "trashbin_retention_obligation": "auto",
        "mail_domain": "***REMOVED SENSITIVE VALUE***",
        "overwrite.cli.url": "https:\/\/redacted from the output\/",
        "htaccess.RewriteBase": "\/",
        "auth.bruteforce.protection.enabled": true,
        "mail_smtpauthtype": "LOGIN",
        "mail_smtpport": "25",
        "default_phone_region": "FI",
        "preview_max_memory": 256,
        "mail_from_address": "***REMOVED SENSITIVE VALUE***",
        "mail_sendmailmode": "smtp",
        "mysql.utf8mb4": true
    }
}

List of activated Apps

Enabled:
  - activity: 2.19.0
  - admin_audit: 1.17.0
  - bruteforcesettings: 2.7.0
  - cloud_federation_api: 1.10.0
  - comments: 1.17.0
  - contactsinteraction: 1.8.0
  - dav: 1.27.0
  - federatedfilesharing: 1.17.0
  - federation: 1.17.0
  - files: 1.22.0
  - files_pdfviewer: 2.8.0
  - files_reminders: 1.0.0
  - files_rightclick: 1.6.0
  - files_sharing: 1.19.0
  - files_trashbin: 1.17.0
  - files_versions: 1.20.0
  - logreader: 2.12.0
  - lookup_server_connector: 1.15.0
  - notifications: 2.15.0
  - oauth2: 1.15.2
  - password_policy: 1.17.0
  - photos: 2.3.0
  - privacy: 1.11.0
  - provisioning_api: 1.17.0
  - recommendations: 1.6.0
  - serverinfo: 1.17.0
  - settings: 1.9.0
  - sharebymail: 1.17.0
  - systemtags: 1.17.0
  - text: 3.8.0
  - theming: 2.2.0
  - twofactor_backupcodes: 1.16.0
  - twofactor_totp: 9.0.0
  - viewer: 2.1.0
  - workflowengine: 2.9.0
Disabled:
  - circles: 27.0.1 (installed 22.1.1)
  - dashboard: 7.7.0 (installed 7.0.0)
  - encryption: 2.15.0
  - files_external: 1.19.0
  - firstrunwizard: 2.16.0 (installed 1.0)
  - nextcloud_announcements: 1.16.0 (installed 1.13.0)
  - related_resources: 1.2.0 (installed 1.1.0-alpha1)
  - support: 1.10.0 (installed 1.0.1)
  - survey_client: 1.15.0 (installed 1.10.0)
  - suspicious_login: 5.0.0
  - updatenotification: 1.17.0 (installed 1.0.1)
  - user_ldap: 1.17.0
  - user_status: 1.7.0 (installed 1.0.1)
  - weather_status: 1.7.0 (installed 1.2.0)

Nextcloud Signing status

No errors have been found.

Nextcloud Logs

No response

Additional info

Also any previous deleted file in the view disappear after deleting one file running Nextcloud version 27.1.6.

@Gummikavalier Gummikavalier added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Jan 25, 2024
@Dennis1993
Copy link
Contributor

I can reproduce it. After upgrade to 27.1.6 the trash bin is always empty.

@solracsf
Copy link
Member

Yes, there is a TypeError:

TypeError: (0 , d.parseWebdavPermissions) is not a function at v (https://nextcloud.tld/dist/files_trashbin-main.js?v=81952118-50:2:855)

@ostasevych
Copy link

ostasevych commented Jan 25, 2024

Yes, there is a TypeError:

TypeError: (0 , d.parseWebdavPermissions) is not a function at v (https://nextcloud.tld/dist/files_trashbin-main.js?v=81952118-50:2:855)

are there any quick fixes?

@solracsf solracsf changed the title [Bug]: Deleted files view does not show any deleted files [Bug]: v27 - Deleted files view (files_trashbin) does not show deleted files Jan 25, 2024
@solracsf solracsf changed the title [Bug]: v27 - Deleted files view (files_trashbin) does not show deleted files [Bug]: v27.1.6 - Deleted files view (files_trashbin) does not show deleted files Jan 25, 2024
joshtrichards added a commit that referenced this issue Jan 25, 2024
Fixes #43134 

Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards
Copy link
Member

joshtrichards commented Jan 25, 2024

At first I was confused because I thought this report was for v28 (which doesn't have this problem ironically).

Pretty certain it's supposed to be davParsePermissions though a quick look would lead me to believe it was an issue in v27.1.5 too.

As far as I can tell, parseWebdavPermissions was the legacy/interim API in nextcloud-files early on, but switched in nextcloud-libraries/nextcloud-files#706

My wild guess is this got overlooked in NC27 because so much of the F2V changes are in NC28.

Preliminary (untested) fix in #43136.

are there any quick fixes?

Not really. This code gets compiled so it's not as easy to monkey patch (unlike the PHP).

nextcloud-command pushed a commit that referenced this issue Jan 25, 2024
Fixes #43134

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@solracsf
Copy link
Member

solracsf commented Jan 26, 2024

Good catch!
Tested #43136 and works ✅

@solracsf solracsf added 3. to review Waiting for reviews and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jan 26, 2024
@ostasevych
Copy link

At first I was confused because I thought this report was for v28 (which doesn't have this problem ironically).

Pretty certain it's supposed to be davParsePermissions though a quick look would lead me to believe it was an issue in v27.1.5 too.

As far as I can tell, parseWebdavPermissions was the legacy/interim API in nextcloud-files early on, but switched in nextcloud-libraries/nextcloud-files#706

My wild guess is this got overlooked in NC27 because so much of the F2V changes are in NC28.

Preliminary (untested) fix in #43136.

are there any quick fixes?

Not really. This code gets compiled so it's not as easy to monkey patch (unlike the PHP).

While inspecting the code I also found reference to davParsePermissions function instead of unknown (or obsolete) parseWebdavPermissions.

Is that possible to share the compiled file(s) which fix the bug for quick and dirty patch? Not waiting until the new subrelease is issued?

@solracsf
Copy link
Member

solracsf commented Jan 26, 2024

Is that possible to share the compiled file(s) which fix the bug for quick and dirty patch?

All files are at #43136

@solracsf
Copy link
Member

Fixed by #43136

@solracsf solracsf added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 26, 2024
@Githopp192

This comment was marked as resolved.

@Gummikavalier
Copy link
Author

Confirmed working in Nextcloud 27.1.7. Thanks! ❤️

@Githopp192
Copy link

confirm fixed with 27.1.7

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

Successfully merging a pull request may close this issue.

8 participants