Fix empty files not being able to be shared

[mpivchev]

An empty .md or .txt is 0 bytes, and the check if the file exists in the document storage expects the size of the file to be > 0. This removes the check, which allows a 0 byte file to be shared.

@marinofaggiana do you think this is a dangerous solution? I haven't noticed any issues so far. I think all the other checks we do to make sure the file exists are enough.

[Copilot]

Pull request overview

This PR fixes an issue where empty files (0 bytes) could not be shared because the fileProviderStorageExists method incorrectly rejected files with zero size. The fix removes the && metadata.size > 0 condition from the file existence checks.

Key Changes:

• Removed the metadata.size > 0 check from two return statements in the fileProviderStorageExists method
• Now allows empty files (0 bytes) to be recognized as valid for sharing

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[marinofaggiana]

#if EXTENSION_FILE_PROVIDER_EXTENSION
return (fileNameViewSize == metadata.size) && metadata.size > 0
#else

Must exists because in FILE_PROVIDER_EXTENSION I'm not sure of the effects

[marinofaggiana]

EXTENSION_FILE_PROVIDER_EXTENSION

[mpivchev]

#if EXTENSION_FILE_PROVIDER_EXTENSION return (fileNameViewSize == metadata.size) && metadata.size > 0 #else

Must exists because in FILE_PROVIDER_EXTENSION I'm not sure of the effects

Done, please recheck

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[mpivchev]

#if EXTENSION_FILE_PROVIDER_EXTENSION return (fileNameViewSize == metadata.size) && metadata.size > 0 #else
Must exists because in FILE_PROVIDER_EXTENSION I'm not sure of the effects

Done, please recheck

@marinofaggiana