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

Updater's should stop iterating through ALL files in deleteOldFiles() #397

Closed
kreativf opened this issue Nov 30, 2021 · 2 comments
Closed

Comments

@kreativf
Copy link

This isn't so much of a bug, as it is a QOL improvement that might fix some stuff along the way.
Right now the deleteOldFiles() function will iterate through all files inside of the nextcloud folder, even though the files might be inside a directory listed in $excludedElements. In my case it takes forever to iterate through ALL of the data folder during each update, including the painfully large cache, even though at the end of the day it doesn't do anything to those files.

@joshtrichards joshtrichards changed the title Updater's deleteOldFiles() function should stop iterating through ALL files Updater's should stop iterating through ALL files in deleteOldFiles() Nov 16, 2023
@joshtrichards
Copy link
Member

Related: #507 (same solution will be likely used for both)

@come-nc
Copy link
Collaborator

come-nc commented Feb 10, 2025

Fixed by #516

@come-nc come-nc closed this as completed Feb 10, 2025
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

3 participants