When checking to see if a file should be moved to public storage, an array of fields is being created on line 394 of HandlerService.php, the array loops through and checks each fields uri_scheme. If just one of those fields has a private uri_scheme then the whole array is being unset preventing the fields from proceeding through the method.
For example, on a current project, my when publishing a document media entity, there are two fields being set in that array. A thumbnail field and the actual file field. The thumbnail field has a private uri_scheme and the actual file field has a public uri_scheme. The thumbnail field causes the fields array to become unset so the actual file field never gets processed and the file doesn't move to the public directory as I would expect.
Needs work
1.0
Code
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Thanks for the bug report! It would be great to have a test for this!
created MR with a cleaner unset based on the array key instead of the loop variable.
attached is a static patch that can be used for composer builds.