(Tests needed) Some stream wrappers need to be hidden or read-only

Created on 14 January 2010, about 15 years ago
Updated 28 March 2025, 6 days ago

Currently the file module looks at all stream wrappers and presents everyone except 'temporary' as a possible file upload destination.

This seems terribly foolish to hard-code this way.

Wee should expand the data returned by hook_stream_wrappers() to include those as flags.

πŸ“Œ Task
Status

Fixed

Version

11.0 πŸ”₯

Component

file system

Created by

πŸ‡ΊπŸ‡ΈUnited States pwolanin

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡¦πŸ‡ΊAustralia kim.pepper πŸ„β€β™‚οΈπŸ‡¦πŸ‡ΊSydney, Australia

    This was committed but never closed.

  • πŸ‡¦πŸ‡ΊAustralia kim.pepper πŸ„β€β™‚οΈπŸ‡¦πŸ‡ΊSydney, Australia

    If there are tests required still we should open a new issue.

  • πŸ‡³πŸ‡ΏNew Zealand quietone

    @kim.pepper, thanks for closing this issue.

    I am updating the meta data and credit to the time of the commit.

    commit d53f3e39d5da63f20a90f3a221da139b7afa7673
    Author: Dries Buytaert <dries@buytaert.net>
    Date:   Tue Jan 26 08:29:25 2010 +0000
    
        - Patch #685074 by pwolanin: some stream wrappers need to be hidden or read-only.
    
Production build 0.71.5 2024