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
set etag and permission fields for trashbin entries #21544
Conversation
By analyzing the blame information on this pull request, we identified @PVince81, @schiesbn and @Xenopathic to be potential reviewers |
👍 as discussed |
@@ -91,6 +92,8 @@ public static function getTrashFiles($dir, $user, $sortAttribute = '', $sortDesc | |||
'type' => $view->is_dir($dir . '/' . $entryName) ? 'dir' : 'file', | |||
'directory' => ($dir === '/') ? '' : $dir, | |||
'size' => $size, | |||
'etag' => '', | |||
'permissions' => Constants::PERMISSION_ALL - Constants::PERMISSION_SHARE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Files can't have create permissions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So nothing bad really happens probably. But for cleanness.
|
... |
Yep, works indeed. |
👍 |
set etag and permission fields for trashbin entries
@karlitschek @icewind1991 Could we backport this to stable8.1 and stable8.2? |
i think a backport makes sense 👍 |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Properly set all the fields that are expected
Fixes #21378
cc @PVince81 @schiesbn