Skip to content

dont try to update storage mtime if we can't get the mtime - #19743

Merged
rullzer merged 1 commit into
masterfrom
cache-updater-storage-mtime-false
Mar 5, 2020
Merged

dont try to update storage mtime if we can't get the mtime#19743
rullzer merged 1 commit into
masterfrom
cache-updater-storage-mtime-false

Conversation

@icewind1991

Copy link
Copy Markdown
Member

same check as is already in place for correctParentStorageMtime

@icewind1991icewind1991 added the 3. to review Waiting for reviews label Mar 2, 2020
@icewind1991icewind1991 added this to the Nextcloud 19 milestone Mar 2, 2020
@icewind1991
icewind1991 requested a review from rullzerMarch 2, 2020 13:28
Comment threadlib/private/Files/Cache/Updater.php Outdated
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991
icewind1991force-pushed the cache-updater-storage-mtime-false branch from 5d89fde to f972990CompareMarch 2, 2020 14:57
@rullzer
rullzer merged commit 672b7ad into masterMar 5, 2020
@rullzer
rullzer deleted the cache-updater-storage-mtime-false branch March 5, 2020 10:22
@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable18

@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable17

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@icewind1991@rullzer@nickvergessen