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

Handle file contained inside the uploads folder #32956

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

CarlSchwan
Copy link
Member

Fix #32450

@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Jun 21, 2022
@CarlSchwan CarlSchwan added this to the Nextcloud 25 milestone Jun 21, 2022
@CarlSchwan CarlSchwan requested review from PVince81 and a team June 21, 2022 12:06
@CarlSchwan CarlSchwan self-assigned this Jun 21, 2022
@CarlSchwan CarlSchwan requested review from ArtificialOwl and come-nc and removed request for a team June 21, 2022 12:06
@CarlSchwan CarlSchwan force-pushed the fix/upload-folder-wrong-content branch from 0f322d3 to 37076b5 Compare June 21, 2022 12:29
Fix #32450

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan CarlSchwan force-pushed the fix/upload-folder-wrong-content branch from 37076b5 to 944675a Compare June 21, 2022 12:29
Copy link
Member

@ArtificialOwl ArtificialOwl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code looks good, but i am not psalm !

@PVince81 PVince81 merged commit a02dee7 into master Jul 27, 2022
@PVince81 PVince81 deleted the fix/upload-folder-wrong-content branch July 27, 2022 15:06
@skjnldsv skjnldsv mentioned this pull request Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Cron Error: Call to undefined method OC\Files\Node\File::getDirectoryListing()
5 participants