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

Prevent the OpenDocument preview generator from trying to open empty files #37772

Merged
merged 3 commits into from
Apr 18, 2023

Conversation

szaimen
Copy link
Contributor

@szaimen szaimen commented Apr 17, 2023

Supersedes #30986

rotdrop and others added 2 commits April 17, 2023 16:19
…files.

Rationale: does not make sense, and triggers a deprecation error in
\ZipArchive.

Signed-off-by: Claus-Justus Heine <himself@claus-justus-heine.de>
Co-authored-by: Carl Schwan <carl@carlschwan.eu>
@szaimen szaimen added the 3. to review Waiting for reviews label Apr 17, 2023
@szaimen szaimen added this to the Nextcloud 27 milestone Apr 17, 2023
@szaimen szaimen requested review from juliushaertl, come-nc, a team, ArtificialOwl and icewind1991 and removed request for a team April 17, 2023 14:20
Signed-off-by: Simon L <szaimen@e.mail.de>
@szaimen szaimen merged commit 8508865 into master Apr 18, 2023
@szaimen szaimen deleted the enh/30986/follow-up branch April 18, 2023 10:18
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.

4 participants